1185582 - Back out the hack to retry image decoding when memory is low, which is now useless

Status: RESOLVED FIXED

(Core :: Layout: Images, Video, and HTML Frames, defect)

Description

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

In bug 1171356 we landed a hack to retry decoding an image up to 10 times if memory was currently too low to allocate the image's first frame. There are two reasons why it no longer makes sense to keep this hack:

- This was motivated by the fact that we locked images during decoding, which meant that until the decoder got to the front of the decode queue and actually started running, the image's first frame would pointlessly consume image memory that could not be freed. This was fixed in bug 1117607 and bug 1176124; we no longer allocate an image's first frame until the decoder actually starts running.

- *Because* we no longer allocate an image's first frame until the decoder starts running, checking for decoder errors in RasterImage::CreateDecoder() to see if allocating the first frame of the image failed is now useless, because we don't allocate the first frame of the image at that point anymore.

Let's remove the code that landed in bug 1171356.

Comment 1

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

Attachment: Back out bug 1171356, a hack to retry image decoding which is now useless

Here's the patch. One less hack in the codebase.

Comment 2

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

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

Comment 3

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

Thanks for the quick review!

Comment 4

[Guilherme Lima]

This landed yesterday - https://hg.mozilla.org/mozilla-central/rev/df5f40088d22