Intent to Prototype and Ship: Don't clear adoptedStyleSheets on adoption to/from <template>

[Rakina Zata Amni]

Contact emails

rak...@chromium.org

Explainer

https://github.com/WICG/construct-stylesheets/issues/133 - no explainer as this is a small improvement, and spec change is pending due to the dependency on moving to the DOM & CSSOM specs (but should be simple).

Summary

When adopting a shadow root into a <template> document from a document that the <template> is in (or vice versa), we will no longer clear its adoptedStyleSheets.

Motivation

Currently we always clear adoptedStyleSheets when the shadow root containing it is adopted into a different document. The main reason for this is to ensure constructed stylesheets are not used across <iframe>s, but this also covers adopting into/from <template>s, causing some confusion to web developers (see https://crbug.com/1111864)

We recently assessed that there should be no problem with keeping adoptedStyleSheets on adoption for the <template> case, as the lifetime of a <template> is bounded by the lifetime of the document containing it. For more details, see https://github.com/WICG/construct-stylesheets/issues/133.

Risks

Interoperability and Compatibility

The risk for this change should be low as the behavior change is small (and more intuitive) and Chrome is the only browser that has shipped adoptedStyleSheets (Firefox implementation is pending).

Edge: No signals

Firefox: No signals

Safari: No signals

Web / Framework developers: Positive (see bug)

Will this feature be supported on all six Blink platforms (Windows, Mac, Linux, Chrome OS, Android, and Android WebView)?

Yes 

Is this feature fully tested by web-platform-tests?

Yes (tests added in https://chromium-review.googlesource.com/c/chromium/src/+/2464519) 

Link to entry on the feature dashboard

https://chromestatus.com/features/5980793119703040 

Requesting approval to ship?

Yes

[Alex Russell]

Hey Rakina!

It's great to see sharp edges being rounded off for constructable stylesheets. The current usage looks very high, however:

    https://chromestatus.com/metrics/feature/timeline/popularity/2846

Do we have a way to know if we'll cause breakage with this? Do we know if cross-document template cloning is rare?

Regards

[Rakina Zata Amni]

Thanks Alex,

I don't think we have data on cross-document template cloning + adoptedStyleSheets is used, we can try to gather some data if we need to. However, I think that the current behavior (clearing on adoption) is very much a footgun and is unexpected to web developers though (see comment), and I doubt many developers depend on the current behavior (and if they do, they can have a quick fix for this by clearing manually). Additionally, nothing other than adoptedStyleSheets is currently cleared on adoption (see adopt, adopting steps, adoptedStyleSheets adopting steps).

[Chris Harrelson]

LGTM1

--
You received this message because you are subscribed to the Google Groups "blink-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+...@chromium.org.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CACPC1r6bgbBHpXNseRmTqSzOEPf0A_ZZW%2BHwS2sXqzbK28A0ng%40mail.gmail.com.

[TAMURA, Kent]

LGTM2

To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOMQ%2Bw8CiMwnU0fTFsJnKmjWaFwOS5hjUq3T59LwC%2BRqsc6X6Q%40mail.gmail.com.

--

TAMURA Kent
Software Engineer, Google

[Yoav Weiss]

LGTM3

To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAGH7WqELvBAzaiCR453rDfp3fmUSjehWtQmZ-wGytJ%3DietZ3zA%40mail.gmail.com.