Closed
Bug 1651587
Opened 5 years ago
Closed 5 years ago
Make image::Image releasing on main thread efficient
Categories
(Core :: Graphics: ImageLib, enhancement)
Core
Graphics: ImageLib
Tracking
()
RESOLVED
FIXED
mozilla80
| Tracking | Status | |
|---|---|---|
| firefox80 | --- | fixed |
People
(Reporter: sotaro, Assigned: sotaro)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
There are several places to release image::Images on MainThread async way. But they just post the Images to main thread. And there were cases that deletions of the Images took long time. It could increase peak memory usage like Bug 1639280.
| Assignee | ||
Updated•5 years ago
|
Assignee: nobody → sotaro.ikeda.g
| Assignee | ||
Comment 1•5 years ago
|
||
From the following link, it is intentional to release image async way on main thread.
https://searchfox.org/mozilla-central/rev/85ae3b911d5fcabd38ef315725df32e25edef83b/image/DecodedSurfaceProvider.cpp#39
| Assignee | ||
Comment 2•5 years ago
•
|
||
The following always release it on main thread async way.
https://searchfox.org/mozilla-central/rev/85ae3b911d5fcabd38ef315725df32e25edef83b/image/DecodedSurfaceProvider.cpp#46
| Assignee | ||
Comment 3•5 years ago
|
||
| Assignee | ||
Comment 4•5 years ago
|
||
| Assignee | ||
Updated•5 years ago
|
Summary: Make async image::Image releasing on main thread efficient → Make image::Image releasing on main thread quick
Updated•5 years ago
|
Attachment #9162413 -
Attachment description: Bug 1651587 - Make image::Image released quickly on main thread → Bug 1651587 - Make image::Image released efficient on main thread
| Assignee | ||
Updated•5 years ago
|
Summary: Make image::Image releasing on main thread quick → Make image::Image releasing on main thread efficient
| Assignee | ||
Comment 5•5 years ago
|
||
Pushed by sikeda.birchill@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f7a6478d206e
Make image::Image released efficient on main thread r=aosmond
Comment 7•5 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox80:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
You need to log in
before you can comment on or make changes to this bug.
Description
•