Closed Bug 847630 Opened 7 years ago Closed 7 years ago

VectorImage doesn't need to defer OnStopRequest anymore

Categories

(Core :: ImageLib, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla22

People

(Reporter: seth, Assigned: seth)

References

Details

Attachments

(1 file)

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.
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
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+
https://hg.mozilla.org/mozilla-central/rev/e2cdf49fe873
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in before you can comment on or make changes to this bug.