Open
Bug 987016
Opened 10 years ago
Updated 2 years ago
Add size constraints from CSS background image
Categories
(Core :: Graphics: ImageLib, defect, P3)
Tracking
()
NEW
People
(Reporter: kanru, Unassigned)
References
Details
(Keywords: perf, Whiteboard: [c=memory p= s= u=tarako] [demo])
Attachments
(1 file)
975 bytes,
patch
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #945161 +++ The downsample-on-decode should be able to get constraints from CSS background-size.
Whiteboard: [c=memory p= s= u=tarako] [MemShrink:P2][demo] → [c=memory p= s= u=tarako] [demo]
Reporter | ||
Comment 1•10 years ago
|
||
Comment 2•10 years ago
|
||
Does this actually work? I would expect that function to be called fairly late, usually after decoding had already started.
Reporter | ||
Comment 3•10 years ago
|
||
(In reply to Timothy Nikkel (:tn) from comment #2) > Does this actually work? I would expect that function to be called fairly > late, usually after decoding had already started. It does work at least on my build. The first StartDecoding() called in PrepareImage() triggers a size decode I guess, and the full decode is done when we actually trying to draw the background. I'll analysis thoroughly and post an update.
Comment 4•10 years ago
|
||
(In reply to Kan-Ru Chen [:kanru] from comment #3) > (In reply to Timothy Nikkel (:tn) from comment #2) > > Does this actually work? I would expect that function to be called fairly > > late, usually after decoding had already started. > > It does work at least on my build. The first StartDecoding() called in > PrepareImage() triggers a size decode I guess, and the full decode is done > when we actually trying to draw the background. Size decodes are automatically done for images before and without any requests for any decoding of the image, so it must be more complicated then this.
Reporter | ||
Updated•10 years ago
|
Attachment #8398414 -
Flags: review?(seth)
Comment 5•6 years ago
|
||
Moving to p3 because no activity for at least 1 year(s). See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information
Priority: P2 → P3
Comment 6•2 years ago
|
||
The bug assignee is inactive on Bugzilla, so the assignee is being reset.
Assignee: kanru → nobody
Status: ASSIGNED → NEW
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•