Closed
Bug 944847
Opened 12 years ago
Closed 12 years ago
"Assertion failure: !doc->IsLoadedAsData() (Should not get in here at all)" after adoptNode
Categories
(Core :: DOM: Events, defect)
Tracking
()
VERIFIED
FIXED
mozilla29
People
(Reporter: jruderman, Assigned: smaug)
References
Details
(Keywords: assertion, regression, testcase)
Attachments
(4 files)
|
314 bytes,
text/html
|
Details | |
|
14.21 KB,
text/plain
|
Details | |
|
3.02 KB,
patch
|
bzbarsky
:
review+
|
Details | Diff | Splinter Review |
|
3.03 KB,
patch
|
Details | Diff | Splinter Review |
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•12 years ago
|
||
|
Assignee | |
Comment 2•12 years ago
|
||
The assertion shows an existing bug, or inconsistency.
|
|
Assignee | |
Comment 3•12 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•12 years ago
|
||
I think we should do this.
Assignee: nobody → bugs
Attachment #8345372 -
Flags: review?(bzbarsky)
|
||
Comment 5•12 years ago
|
||
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•12 years ago
|
||
|
|
Assignee | |
Comment 7•12 years ago
|
||
|
||
Comment 8•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
|
||
Comment 9•12 years ago
|
||
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.
Description
•