"Assertion failure: !doc->IsLoadedAsData() (Should not get in here at all)" after adoptNode

VERIFIED FIXED in mozilla29

Status

()

Core
DOM: Events
--
critical
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: Jesse Ruderman, Assigned: smaug)

Tracking

(Blocks: 3 bugs, {assertion, regression, testcase})

Trunk
mozilla29
x86_64
Mac OS X
assertion, regression, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments)

(Reporter)

Description

4 years ago
Created attachment 8340591 [details]
testcase

Assertion failure: !doc->IsLoadedAsData() (Should not get in here at all), at /Users/jruderman/trees/mozilla-central/content/events/src/nsEventListenerManager.cpp:1346

Regression from bug 942432?
(Reporter)

Comment 1

4 years ago
Created attachment 8340592 [details]
stack
(Assignee)

Comment 2

4 years ago
The assertion shows an existing bug, or inconsistency.
(Assignee)

Comment 3

4 years ago
Problem is that the spec doesn't define what should happen in this case
Hixie filed https://www.w3.org/Bugs/Public/show_bug.cgi?id=23956
(Assignee)

Comment 4

4 years ago
Created attachment 8345372 [details] [diff] [review]
patch

I think we should do this.
Assignee: nobody → bugs
Attachment #8345372 - Flags: review?(bzbarsky)
Comment on attachment 8345372 [details] [diff] [review]
patch

I don't think we should NS_ENSURE_TRUE, with its attendant warning, here.

r=me other than that.
Attachment #8345372 - Flags: review?(bzbarsky) → review+
(Assignee)

Comment 6

4 years ago
Created attachment 8347241 [details] [diff] [review]
v2
(Assignee)

Comment 7

4 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/65167826b460
https://hg.mozilla.org/mozilla-central/rev/65167826b460
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Reproduced on Nightly 2013-11-29-mozilla-central-debug.
Verified fixed on Nightly 29.0a1 2013-12-18-mozilla-central-debug, Win 7 x64.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.