"ASSERTION: no owner document" involving createDocumentType

VERIFIED FIXED

Status

()

VERIFIED FIXED
13 years ago
10 years ago

People

(Reporter: jruderman, Assigned: dbaron)

Tracking

(Blocks: 1 bug, {assertion, fixed1.8.1, testcase})

Trunk
PowerPC
Mac OS X
assertion, fixed1.8.1, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [patch])

Attachments

(2 attachments)

(Reporter)

Description

13 years ago
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

13 years ago
Created attachment 211375 [details]
testcase
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.
Created attachment 211398 [details] [diff] [review]
patch
Assignee: general → dbaron
Status: NEW → ASSIGNED
Attachment #211398 - Flags: superreview?(peterv)
Attachment #211398 - Flags: review?(bugmail)
Whiteboard: [patch]
Attachment #211398 - Flags: superreview?(peterv) → superreview+
Fix checked in to trunk.
Blocks: 241518
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
(Reporter)

Updated

13 years ago
Status: RESOLVED → VERIFIED
Fixed on MOZILLA_1_8_BRANCH by checkin of bug 336791.
Keywords: fixed1.8.1
(Reporter)

Comment 7

11 years ago
Crashtest checked in.
Flags: in-testsuite+

Updated

10 years ago
Component: DOM: Core → DOM: Core & HTML
QA Contact: ian → general
You need to log in before you can comment on or make changes to this bug.