Bug 243540 – Stylesheets from other stylesheet sets should be disabled

• Status: NEW
• Component: CSS

[Ahmad Saleem]

Hi Team,

Another test case from Chrome bugs, where Safari 15.6 is failing test case similar to Chrome and I think we should fix and align it with web-spec.

Test Case - https://bug1708045.bmoattachments.org/attachment.cgi?id=9220199

Chrome Bug - https://bugs.chromium.org/p/chromium/issues/detail?id=1205488

Web-Spec Reference (from Chrome Bug Report) - The sheet should be disabled, per https://drafts.csswg.org/cssom/#add-a-css-style-sheet

*** STEPS TO REPRODUCE ***

1) Run the test case and review dialog box

<> EXPECTED RESULT:

PASS (The red stylesheet's 'disabled' flag is true)

<> ACTUAL RESULT:

FAIL (The red stylesheet's 'disabled' flag is false)

____

NOTE - Safari 15.6 and Chrome Canary 106 both fail this test case.

If this has been fixed upstream, please mark this as DUPLICATE.

Thanks!

[Ryosuke Niwa]

Created attachment 461403 [details]
WIP

[Ryosuke Niwa]

Created attachment 461404 [details]
Test

[Ryosuke Niwa]

Comment on attachment 461403 [details]
WIP

View in context: https://bugs.webkit.org/attachment.cgi?id=461403&action=review

> Source/WebCore/style/StyleScope.cpp:408
> +                sheet->setDisabled(title != m_preferredStylesheetSetName);

This isn't quite right. Firefox allows unpreferred / disabled script to be enabled again by script.

[Radar WebKit Bug Importer]

<rdar://problem/98504974>

[Ahmad Saleem]

Web-Spec: Setting the disabled flag in step 5 of https://drafts.csswg.org/cssom/#add-a-css-style-sheet

Source - Mozilla bug (https://bugzilla.mozilla.org/show_bug.cgi?id=1708045)