We trigger a decode for basically every image we download because imgStatusTracker::SyncNotify calls FrameRect on the image, which asks for decoded frames, which triggers a decode. This is the only caller of FrameRect, so it shouldn't be a problem to make FrameRect not ask for decoded frames.
This causes some of our async_load_tests.js tests to fail for a very simple reason: without this inadvertent decode we don't trigger a decode at all in these tests. But checkAllCallbacks wants us to be decoded fully. Should we insert a requestDecode in this test or just not check if the image is decoded?
(In reply to Timothy Nikkel (:tn) from comment #1) > But checkAllCallbacks wants us to be decoded fully. We only check this on a static request. So maybe we only have these expectations of being fully decoded on static requests? Before bug 843895 that was always true because static requests used ExtractFrame with sync decode, but after bug 843895 we just wrap them in a wrapper that always returns the first frame.
Talked to Joe on irc about this.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in before you can comment on or make changes to this bug.