Closed Bug 617718 Opened 9 years ago Closed 9 years ago

Fennec crash [@ nsPresContext::IsRootContentDocument ] when loading GrooveShark (plugin.disable=false, dom.ipc.plugins.enabled=true)

Categories

(Core :: Layout, defect)

ARM
Maemo
defect
Not set

Tracking

()

VERIFIED FIXED
mozilla2.0b9

People

(Reporter: dholbert, Assigned: roc)

Details

(Keywords: crash, crashreportid)

Crash Data

Attachments

(1 file)

STEPS TO REPRODUCE:
On a Nokia n900 with flashplayer installed:
 1. Set pref "plugin.disable" to false and "dom.ipc.plugins.enabled" to true, & restart Fennec.
(NOTE: If you don't set the second pref above, you'll hit Bug 617703 instead.)
 2. Visit http://listen.grooveshark.com
 3. Wait for a minute -- if you haven't crashed yet, try to interact with the page. (Search for a song, try to play it, etc.)

ACTUAL RESULTS: Crash [@ nsPresContext::IsRootContentDocument ]
Sample crash reports:
 bp-0520609d-0692-4e5e-90dd-04e172101208
 bp-e6263632-5147-4668-9218-30e262101208
Keywords: crash, crashreportid
|vm| is null?
Ah, yes.  We're under PresShell:Destroy, so it is.

We shouldn't be doing that invalidate during frame destruction, imho.
Attached patch fixSplinter Review
Completely untested, but trivial fix.
Assignee: nobody → roc
Attachment #496278 - Flags: review?(benjamin)
Attachment #496278 - Flags: review?(benjamin)
Attachment #496278 - Flags: review+
Attachment #496278 - Flags: approval2.0+
Keywords: checkin-needed
Whiteboard: [needs landing]
Landed: http://hg.mozilla.org/mozilla-central/rev/6b98f6cf4589
Status: NEW → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [needs landing]
Target Milestone: --- → mozilla2.0b9
I've verified that this no longer crashes, with the latest mozilla-central Fennec nightly on my n900.
Status: RESOLVED → VERIFIED
Crash Signature: [@ nsPresContext::IsRootContentDocument ]
You need to log in before you can comment on or make changes to this bug.