Closed Bug 1311779 Opened 3 years ago Closed 3 years ago

Image decoders should use B8G8R8X8 if possible

Categories

(Core :: ImageLib, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla52
Tracking Status
firefox52 --- fixed

People

(Reporter: aosmond, Assigned: aosmond)

References

(Blocks 1 open bug)

Details

Attachments

(2 files, 1 obsolete file)

JPEG and BMP decoders do not start in B8G8R8X8 and rely upon an optimization later to use the correct format. We should also add telemetry to track how frequently this optimization would be used (even if disabled at present due to issues bug 1290293 is fixing) to decide if it is even necessary with the changes in this bug.
Assignee: nobody → aosmond
Status: NEW → ASSIGNED
Blocks: 1290293
Change it to expire in 53 (we shouldn't need to run the collection for long).
Attachment #8803046 - Attachment is obsolete: true
Attachment #8803040 - Flags: review?(tnikkel)
Attachment #8806015 - Flags: review?(tnikkel)
Attachment #8803040 - Flags: review?(tnikkel) → review+
Attachment #8806015 - Flags: review?(tnikkel) → review+
Pushed by aosmond@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1cea19368e06
Part 1. Ensure JPEG and BMP image decoders use B8G8R8X8 surfaces when possible. r=tnikkel
https://hg.mozilla.org/integration/mozilla-inbound/rev/36b7546597dc
Part 2. Add telemetry to track optimizable opaque images decoded into B8G8R8A8 surfaces. r=tnikkel
https://hg.mozilla.org/mozilla-central/rev/1cea19368e06
https://hg.mozilla.org/mozilla-central/rev/36b7546597dc
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.