LGTM3
On Tue, May 14, 2024 at 8:49 AM Chris Harrelson <chris...@chromium.org> wrote: > LGTM2 > > On Mon, May 13, 2024 at 4:36 PM Mike Taylor <miketa...@chromium.org> > wrote: > >> Hi Morten - >> >> Appreciate the response. I spent some time looking at >> https://github.com/search?q=%22instanceof+CSSGroupingRule%22+language%3AJavaScript&type=code&l=JavaScript >> as well as "instanceof CSSRule" earlier, but didn't see anything >> immediately worrying (granted, it's hard to predict perfectly how the >> tooling-related scripts might handle sites in the wild that might ship the >> problematic patterns). >> >> LGTM1 to ship. >> On 5/9/24 6:16 AM, Morten Stenshorne wrote: >> >> On Wed, 8 May 2024 at 17:18, Mike Taylor <miketa...@chromium.org> wrote: >> >>> On 5/8/24 3:37 AM, Morten Stenshorne wrote: >>> >>> Interoperability and Compatibility >>> >>> Low risk. The one possible issue is if an author uses "instanceof >>> CSSGroupingRule" or "instanceof CSSRule" with a page rule object and makes >>> incorrect assumptions based on that. However, given that this is already >>> shipping in Firefox (and also that in Firefox, even CSSStyleRule inherits >>> from CSSGroupingRule, as the spec says - whereas Blink still doesn't), the >>> risk should be very low. >>> >>> I agree the risk is likely very low, or contained to just a few >>> sites/applications - have you done any investigation to try to find any >>> problematic code examples that you mention (via GitHub search or >>> HTTPArchive)? >>> >> >> There's a use counter - >> https://chromestatus.com/metrics/feature/timeline/popularity/4862 - for >> CSSPageRule, but I'm not sure how useful it is, since it will count as long >> as someone e.g. iterates over the rules in a style sheet and there happens >> to be an @page rule in there. The use is currently at 0.07, but the number >> isn't that useful, apart from being an absolute worst-case number. >> >> Here's a constructed problematic case if CSSPageRule suddenly inherits >> from CSSGroupingRule: >> >> ``` >> <style> >> @media screen { >> div { >> color: hotpink; >> } >> } >> @page { >> margin: 1in; >> @top-center { >> content: "Header"; >> } >> } >> </style> >> <script> >> for (const rule of document.styleSheets[0].cssRules) { >> if (rule instanceof CSSGroupingRule) { >> // Aha! You're a media rule! (d'oh!) >> // ... >> } >> } >> </script> >> ``` >> >> How likely is that, though? >> >> >>> *Gecko*: Shipped/Shipping ( >>> https://bugzilla.mozilla.org/show_bug.cgi?id=1868215) >>> >>> Noting that the patch should ship to Firefox release channel (126) in >>> about a week, per https://whattrainisitnow.com/calendar/. >>> >> >> Ah, right. Pretty recent change. >> >> On that note, earlier (some time in 2023), Firefox changed CSSStyleRule >> to inherit from CSSGroupingRule - >> https://github.com/w3c/csswg-drafts/issues/8940#issuecomment-1747351690 >> That sounds like a much riskyisher change, but allegedly it went just >> fine. >> >>> >>> *WebKit*: No signal >>> >>> Can we request a signal? Or do we have any other indications about their >>> intentions here? >>> >> >> Done. https://github.com/WebKit/standards-positions/issues/346 >> >> -- >> 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/ee67bf7d-78f9-4309-b831-00942ae69936%40chromium.org >> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/ee67bf7d-78f9-4309-b831-00942ae69936%40chromium.org?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/CAOMQ%2Bw-mNDO7Z2A0-KaB5tz4WAaDfp1%2B5WFpqCNrdaZZLZbPzA%40mail.gmail.com > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOMQ%2Bw-mNDO7Z2A0-KaB5tz4WAaDfp1%2B5WFpqCNrdaZZLZbPzA%40mail.gmail.com?utm_medium=email&utm_source=footer> > . > -- TAMURA Kent Software Engineer, Google -- 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/CAGH7WqHzPDdSpagps3hpTjYLNSuEAfLpJXap0HiboH5Fm%2BRMPA%40mail.gmail.com.