Open Bug 987016 Opened 10 years ago Updated 2 years ago

Add size constraints from CSS background image

Categories

(Core :: Graphics: ImageLib, defect, P3)

All
Gonk (Firefox OS)
defect

Tracking

()

People

(Reporter: kanru, Unassigned)

References

Details

(Keywords: perf, Whiteboard: [c=memory p= s= u=tarako] [demo])

Attachments

(1 file)

+++ 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]
Assignee: nobody → kchen
Status: NEW → ASSIGNED
Attachment #8398414 - Flags: review?(seth)
Does this actually work? I would expect that function to be called fairly late, usually after decoding had already started.
(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.
(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.
Attachment #8398414 - Flags: review?(seth)
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

The bug assignee is inactive on Bugzilla, so the assignee is being reset.

Assignee: kanru → nobody
Status: ASSIGNED → NEW
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: