Intermittent PROCESS-CRASH | Main app process exited normally | application crashed [@ nsImageFrame::DisplayAltFeedback] after Assertion failure: imgCon (Load complete, but no image container?)

RESOLVED FIXED in Firefox 58

Status

()

defect
P5
critical
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: intermittent-bug-filer, Assigned: ethlin)

Tracking

({crash, intermittent-failure})

unspecified
mozilla58
Points:
---

Firefox Tracking Flags

(firefox-esr52 unaffected, firefox56 unaffected, firefox57 unaffected, firefox58 fixed)

Details

(Whiteboard: [gfx-noted], crash signature)

Attachments

(1 attachment)

It looks like we should handle the image error status.
Bullet frame[1] checks both STATUS_LOAD_COMPLETE and STATUS_ERROR before getting the image container. I think we should do the same thing here.

[1] https://dxr.mozilla.org/mozilla-central/rev/083a9c84fbd09a6ff9bfecabbf773650842fe1c0/layout/generic/nsBulletFrame.cpp#737
Assignee: nobody → ethlin
Whiteboard: [gfx-noted]
Comment on attachment 8924074 [details]
Bug 1413082 - Check both STATUS_LOAD_COMPLETE and STATUS_ERROR before getting the image container in nsImageFrame.

https://reviewboard.mozilla.org/r/195290/#review200378
Attachment #8924074 - Flags: review?(matt.woodrow) → review+
Pushed by ethlin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a94bd2a1b543
Check both STATUS_LOAD_COMPLETE and STATUS_ERROR before getting the image container in nsImageFrame. r=mattwoodrow
https://hg.mozilla.org/mozilla-central/rev/a94bd2a1b543
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.