Closed
Bug 1169512
Opened 9 years ago
Closed 9 years ago
control quirks mode UA style sheet rules without needing to clone quirk.css
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla41
Tracking | Status | |
---|---|---|
firefox41 | --- | fixed |
People
(Reporter: heycam, Assigned: heycam)
References
Details
Attachments
(2 files)
31.45 KB,
patch
|
Details | Diff | Splinter Review | |
10.64 KB,
patch
|
jwatt
:
review+
|
Details | Diff | Splinter Review |
Instead of cloning the quirks.css UA style sheet for each document that might need it and toggling its enabled bit, let's always keep it enabled but use a media query to control whether the rules apply. This will let us avoid cloning the style sheet for basically every HTML document and help us in bug 77999 with rule processor caching.
Assignee | ||
Comment 1•9 years ago
|
||
I had a patch to add this media query, but I think it's easier just to add and remove the (non-cloned) quirks sheet from the style set as needed.
Assignee | ||
Comment 2•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Summary: control quirks mode UA style sheet rules with a media query → control quirks mode UA style sheet rules without needing to clone quirk.css
Assignee | ||
Comment 3•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=1c45d3c5a917 - getting a random test failure.
Assignee | ||
Comment 4•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e49ffc6b6ad2
Updated•9 years ago
|
Attachment #8614478 -
Flags: review?(jwatt) → review+
Comment 6•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/368c057b5b00
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in
before you can comment on or make changes to this bug.
Description
•