Closed
Bug 1696447
Opened 3 years ago
Closed 3 years ago
Consider sharing `CascadeData` objects more aggressively across shadow roots.
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
88 Branch
Tracking | Status | |
---|---|---|
firefox88 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
Details
(Keywords: perf-alert)
Attachments
(2 files)
For shadow roots in the same document, with the same stylesheet list, we should be able to do something like this.
We have something like this already for UA sheets, so most of the code should be written already, I think.
Flags: needinfo?(emilio)
Assignee | ||
Comment 1•3 years ago
|
||
We're going to use it both for UA sheets and for author styles in Shadow
DOM.
Updated•3 years ago
|
Assignee: nobody → emilio
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•3 years ago
|
||
This should be both a memory and speed win for pages using a lot of
Shadow DOM.
Depends on D107265
Assignee | ||
Updated•3 years ago
|
Flags: needinfo?(emilio)
Updated•3 years ago
|
Severity: -- → S3
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/5f595b1443a0 Make the cascade data cache generic. r=boris https://hg.mozilla.org/integration/autoland/rev/f088cf41fc03 Share CascadeData instances across ShadowRoots. r=boris
Comment 4•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/5f595b1443a0
https://hg.mozilla.org/mozilla-central/rev/f088cf41fc03
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox88:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch
Comment 5•3 years ago
|
||
== Change summary for alert #29227 (as of Thu, 11 Mar 2021 22:45:52 GMT) ==
Improvements:
Ratio | Suite | Test | Platform | Options | Absolute values (old vs new) |
---|---|---|---|---|---|
38% | perf_reftest_singletons | link-style-cache-1.html | macosx1015-64-shippable-qr | e10s stylo webrender | 600.88 -> 372.65 |
31% | perf_reftest_singletons | link-style-cache-1.html | windows10-64-shippable | e10s stylo | 709.48 -> 489.06 |
30% | perf_reftest_singletons | link-style-cache-1.html | windows10-64-shippable-qr | e10s stylo webrender | 701.73 -> 494.38 |
15% | perf_reftest_singletons | inline-style-cache-1.html | windows10-64-shippable | e10s stylo | 1,744.20 -> 1,477.40 |
15% | perf_reftest_singletons | inline-style-cache-1.html | macosx1015-64-shippable-qr | e10s stylo webrender | 1,283.24 -> 1,090.57 |
For up to date results, see: https://treeherder.mozilla.org/perfherder/alerts?id=29227
Updated•3 years ago
|
Keywords: perf-alert
You need to log in
before you can comment on or make changes to this bug.
Description
•