Always use multithreaded image decoding

RESOLVED FIXED in mozilla37

Status

()

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: seth, Assigned: seth)

Tracking

unspecified
mozilla37
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Assignee

Description

5 years ago
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.
Assignee

Comment 1

5 years ago
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)
Assignee

Updated

5 years ago
Blocks: 1118087
Attachment #8542945 - Flags: review?(tnikkel) → review+
Assignee

Comment 3

5 years ago
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
Assignee

Comment 4

5 years ago
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: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
You need to log in before you can comment on or make changes to this bug.