Closed
Bug 1763643
Opened 3 years ago
Closed 3 years ago
Turn on blob recordings for SVG images
Categories
(Core :: Graphics: ImageLib, task, P3)
Core
Graphics: ImageLib
Tracking
()
RESOLVED
FIXED
101 Branch
Tracking | Status | |
---|---|---|
firefox101 | --- | fixed |
People
(Reporter: aosmond, Assigned: aosmond)
References
(Regressed 2 open bugs)
Details
(Keywords: perf-alert)
Attachments
(1 file)
No description provided.
Assignee | ||
Comment 1•3 years ago
|
||
This should reduce our memory footprint by no longer requiring us to
fully rasterize SVG images when used as an image map. It should also
move the cost of rasterization off the main thread during display list
building to worker threads during scene building.
Pushed by aosmond@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4a5f91ff2e40
Turn on SVG image blob recordings. r=gfx-reviewers,lsalzman
Comment 3•3 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox101:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 101 Branch
Comment 4•3 years ago
•
|
||
== Change summary for alert #33810 (as of Tue, 12 Apr 2022 18:46:13 GMT) ==
Improvements:
Ratio | Test | Platform | Options | Absolute values (old vs new) |
---|---|---|---|---|
25% | Heap Unclassified | linux1804-64-shippable-qr | tp6 | 276,925,392.30 -> 208,088,787.52 |
22% | Heap Unclassified | linux1804-64-shippable-qr | tp6 | 269,623,775.42 -> 210,195,635.40 |
10% | Explicit Memory | linux1804-64-shippable-qr | tp6 | 920,393,593.93 -> 828,522,707.84 |
8% | Resident Memory | linux1804-64-shippable-qr | tp6 | 1,306,076,910.21 -> 1,200,050,686.73 |
5% | Explicit Memory | linux1804-64-shippable-qr | tp6 | 886,963,101.49 -> 842,351,634.63 |
For up to date results, see: https://treeherder.mozilla.org/perfherder/alerts?id=33810
Keywords: perf-alert
Comment 5•3 years ago
|
||
There are a lot of regressions from this. Do we have a plan to fix them or back this out before the merge? Soft freeze is less than a week away.
Flags: needinfo?(aosmond)
Updated•3 years ago
|
See Also: → deferrable-blobs
You need to log in
before you can comment on or make changes to this bug.
Description
•