Closed Bug 1116747 Opened 7 years ago Closed 7 years ago

Always use multithreaded image decoding

Categories

(Core :: ImageLib, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla37

People

(Reporter: seth, Assigned: seth)

References

Details

Attachments

(1 file)

Multithreaded image decoding has been on by default on all platforms for quite some time. Continuing to support disabling it makes it much harder to support multiple decoders for a single RasterImage (bug 1079627). Let's commit to always have it on.
Here we go. This patch basically just removes the pref, then removes things in
DecodePool (the only class that cared about that pref) that checked the pref or
assumed that it might be set to false.
Attachment #8542945 - Flags: review?(tnikkel)
Blocks: 1118087
Attachment #8542945 - Flags: review?(tnikkel) → review+
Pretty sure all the failures here are from earlier bugs in my patch stack, which have now been fixed. Here's a new try job:

https://tbpl.mozilla.org/?tree=Try&rev=a7ca4f452ef4
All the failures are from older bugs and should now be fixed. Pushed:

https://hg.mozilla.org/integration/mozilla-inbound/rev/a49774cdd1b1
https://hg.mozilla.org/mozilla-central/rev/a49774cdd1b1
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
You need to log in before you can comment on or make changes to this bug.