From bug 1346501, comment 16. (In reply to Mats Palmgren (:mats) from comment #16) > BTW, I was looking through the nsImageLoadingContent code and found a > potential issue > in nsImageLoadingContent::MakePendingRequestCurrent: > http://searchfox.org/mozilla-central/rev/ > 557f236c19730116d3bf53c0deef36362cafafcd/dom/base/nsImageLoadingContent. > cpp#1309 > Shouldn't mPendingRequestRegistered be copied to mCurrentRequestRegistered > there > and then reset?
Created attachment 8849274 [details] [diff] [review] patch
Attachment #8849274 - Flags: review?(mats)
Comment on attachment 8849274 [details] [diff] [review] patch LGTM ;-)
Attachment #8849274 - Flags: review?(mats) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/a058d871d52f nsImageLoadingContent::MakePendingRequestCurrent should transfer mPendingRequestRegistered flag to mCurrentRequestRegistered. r=mats
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox55: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.