Closed Bug 1413082 Opened 3 years ago Closed 3 years ago

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

Categories

(Core :: Graphics, defect, P5)

defect

Tracking

()

RESOLVED FIXED
mozilla58
Tracking Status
firefox-esr52 --- unaffected
firefox56 --- unaffected
firefox57 --- unaffected
firefox58 --- fixed

People

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

Details

(Keywords: crash, intermittent-failure, Whiteboard: [gfx-noted])

Crash Data

Attachments

(1 file)

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: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.