1116747 - Always use multithreaded image decoding

Status: RESOLVED FIXED

(Core :: Graphics: ImageLib, defect)

Description

[Seth Fowler [:seth] [:s2h]]

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.

Comment 1

[Seth Fowler [:seth] [:s2h]]

Attachment: Always use multithreaded image decoding

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.

Comment 2

[Seth Fowler [:seth] [:s2h]]

Try job here:

https://tbpl.mozilla.org/?tree=Try&rev=edb318af3e7c

Comment 3

[Seth Fowler [:seth] [:s2h]]

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

Comment 4

[Seth Fowler [:seth] [:s2h]]

All the failures are from older bugs and should now be fixed. Pushed:

https://hg.mozilla.org/integration/mozilla-inbound/rev/a49774cdd1b1

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/a49774cdd1b1