LGTM2 On Tue, Jan 24, 2023 at 9:27 PM Rick Byers <rby...@chromium.org> wrote:
> Thanks Mason. LGTM1 > > On Tue, Jan 24, 2023 at 2:59 PM Mason Freed <mas...@chromium.org> wrote: > >> >> >> On Tue, Jan 24, 2023 at 8:41 AM Rick Byers <rby...@chromium.org> wrote: >> >>> >>>> *Gecko*: Positive ( >>>> https://github.com/whatwg/html/pull/5465#pullrequestreview-1132523065) >>>> >>> >>> Per our process, Mozilla has asked that we only consider positive >>> signals from their standards-position repo. I see they currently have >>> <https://mozilla.github.io/standards-positions/#declarative-shadow-dom> >>> a 'neutral' for declarative shadow DOM with concerns about the cost/benefit >>> tradeoff of the feature. >>> >>> Also this is a link to a comment by a (now) Apple employee. Did you have >>> a different link in mind perhaps? >>> >> >> Good catch - that was a link copy/paste mistake on my part. I've updated >> Chromestatus, but the link I was intending to use was this one for >> Mozilla >> <https://github.com/whatwg/dom/issues/831#issuecomment-1269026678>. >> Having said that, it isn't obviously supportive, so I've also asked to >> have the position issue re-opened and hopefully updated >> <https://github.com/mozilla/standards-positions/issues/335#issuecomment-1402437234> >> . >> >> >>> *WebKit*: Positive ( >>>> https://github.com/whatwg/html/pull/5465#pullrequestreview-1132523065) >>>> >>> >>> Based on your link >>> <https://github.com/WebKit/WebKit/commit/e7d5dde467956b24438d23f458d4d527f7fd108d> >>> above we can now call this "shipping", right? That's a very strong signal >>> IMHO. >>> >> >> Thanks - also updated. It is indeed shipping, and WebKit's official >> position is now also "support" >> <https://github.com/WebKit/standards-positions/issues/12>. >> >> >>> Is this feature fully tested by web-platform-tests >>>> <https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md> >>>> ?Yes >>>> >>> >>> I see the test link >>> <https://wpt.fyi/results/shadow-dom/declarative?label=master&label=experimental&aligned&view=subtest>, >>> but are tests written somewhere for this change in particular? In >>> particular do latest WebKit and chromium with this flag enabled pass the >>> exact same set of tests for declarative shadow DOM or are there still >>> notable behavior differences? >>> >> >> So I've already updated the tests on WPT to use the new `shadowrootmode` >> attribute, and they also now check for streaming behavior. (For Chromium, I >> have a virtual suite >> <https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/web_tests/VirtualTestSuites;l=927;drc=fba07e03f2517fdaf9cb8c45fa8f8db5073219e2> >> that also tests the old behavior, which I'll keep running until I'm able to >> deprecate and remove it.) >> >> WebKit (reportedly >> <https://github.com/whatwg/html/pull/5465#issuecomment-1377992970>) >> passes all of those tests except the ones related to the serialization >> behavior which was pulled out >> <https://github.com/whatwg/html/pull/5465#issuecomment-1266085972> of >> the current PR. There is still ongoing discussion around enabling >> imperative parsing (via DOMParser >> <https://github.com/whatwg/html/issues/8759>) of DSD content, and if >> they end up removing that functionality, they'll fail more tests. >> >> Thanks, >> Mason >> >> >> >> >>> Flag nameStreamingDeclarativeShadowDOM >>>> >>>> Requires code in //chrome?False >>>> >>>> Tracking bughttps://crbug.com/1379513 >>>> >>>> Estimated milestones >>>> >>>> M111 >>>> >>>> >>>> Anticipated spec changes >>>> >>>> Open questions about a feature may be a source of future web compat or >>>> interop issues. Please list open issues (e.g. links to known github issues >>>> in the project for the feature specification) whose resolution may >>>> introduce web compat/interop risk (e.g., changing to naming or structure of >>>> the API in a non-backward-compatible way). >>>> >>>> >>>> Link to entry on the Chrome Platform Status >>>> https://chromestatus.com/feature/5161240576393216 >>>> >>>> This intent message was generated by Chrome Platform Status >>>> <https://chromestatus.com/>. >>>> >>>> -- >>>> 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+unsubscr...@chromium.org. >>>> To view this discussion on the web visit >>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAM%3DNeDhGHqD6du83UKvRpX-P7ftaG_R8j1pXE-ofqwHGf-sysA%40mail.gmail.com >>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAM%3DNeDhGHqD6du83UKvRpX-P7ftaG_R8j1pXE-ofqwHGf-sysA%40mail.gmail.com?utm_medium=email&utm_source=footer> >>>> . >>>> >>> -- > 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+unsubscr...@chromium.org. > To view this discussion on the web visit > https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAFUtAY9pyLc1B%3Dnqu1KU3pAWnQWPcM5Ankd34p0jA7AuBWmETg%40mail.gmail.com > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAFUtAY9pyLc1B%3Dnqu1KU3pAWnQWPcM5Ankd34p0jA7AuBWmETg%40mail.gmail.com?utm_medium=email&utm_source=footer> > . > -- 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+unsubscr...@chromium.org. To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAL5BFfUyHD0u3jfXGT8-9gJ7Srapg3eEr%3DBo3hBAnGS8HQ8rPg%40mail.gmail.com.