Since bug 841579 landed, it should now be the case that things don't blow up if all decoding isn't complete by the time LOAD_COMPLETE is sent to nsImageLoadingContent. That means the code added for bug 704059 which deferred the arrival of OnStopRequest for VectorImages until the underlying document was ready is probably redundant now. Let's rip that out and keep things as simple as possible.
Created attachment 723153 [details] [diff] [review] Stop deferring OnStopRequest in VectorImage. Proposed patch. Try job here: https://tbpl.mozilla.org/?tree=Try&rev=b920b9632166
OK, nothing appears to have broken. Now for a broader test to see if we start hitting test_animSVGImage mochitest failures frequently again. Try job here: https://tbpl.mozilla.org/?tree=Try&rev=ee66def6285b
Alright, I'm happy. Pushed in: https://hg.mozilla.org/integration/mozilla-inbound/rev/e2cdf49fe873
Ack, just realized that I never actually requested review for this, even though the patch says r=joe. Joe, I'm gonna go ahead and request review anyway; if you find any issues I'll just create a followup patch to fix them or back the patch out. Sorry for the screwup.
Attachment #723153 - Flags: review?(joe)
Attachment #723153 - Flags: review?(joe) → review+
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in before you can comment on or make changes to this bug.