Closed
Bug 326646
Opened 18 years ago
Closed 18 years ago
"ASSERTION: no owner document" involving createDocumentType
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: jruderman, Assigned: dbaron)
References
Details
(Keywords: assertion, fixed1.8.1, testcase, Whiteboard: [patch])
Attachments
(2 files)
233 bytes,
text/html
|
Details | |
2.42 KB,
patch
|
sicking
:
review+
peterv
:
superreview+
|
Details | Diff | Splinter Review |
After loading the testcase, I get two of these at each GC: ###!!! ASSERTION: no owner document: 'GetOwnerDoc()', file /Users/admin/trunk/mozilla/content/base/src/nsGenericDOMDataNode.cpp, line 632
Reporter | ||
Comment 1•18 years ago
|
||
Comment 2•18 years ago
|
||
In Gecko a node is allowed to not have an ownerDocument (that's why it's GetOwnerDoc() and not OwnerDoc()). We can try to give the DOMImplementation a document (not sure which), but the assertion is bogus and nsGenericDOMDataNode::AppendReachableList should be able to deal with a null ownerDocument.
The doctype *should not* have an owner document there according to the DOM spec, so the solution here is definitely to make AppendReachableList be able to deal.
Assignee | ||
Comment 4•18 years ago
|
||
Assignee: general → dbaron
Status: NEW → ASSIGNED
Attachment #211398 -
Flags: superreview?(peterv)
Attachment #211398 -
Flags: review?(bugmail)
Assignee | ||
Updated•18 years ago
|
Whiteboard: [patch]
Attachment #211398 -
Flags: review?(bugmail) → review+
Updated•18 years ago
|
Attachment #211398 -
Flags: superreview?(peterv) → superreview+
Assignee | ||
Comment 5•18 years ago
|
||
Fix checked in to trunk.
Reporter | ||
Updated•18 years ago
|
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 6•18 years ago
|
||
Fixed on MOZILLA_1_8_BRANCH by checkin of bug 336791.
Keywords: fixed1.8.1
You need to log in
before you can comment on or make changes to this bug.
Description
•