Closed Bug 379105 Opened 18 years ago Closed 17 years ago

"ASSERTION: DOM window should always have a document" with XBL and <frame>

Categories

(Core :: XBL, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: jruderman, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: testcase)

Attachments

(2 files)

###!!! ASSERTION: DOM window should always have a document!: 'domDoc', file /Users/jruderman/trunk/mozilla/toolkit/components/passwordmgr/base/nsPasswordManager.cpp, line 840 Steps to reproduce: unknown This might be a regression from within the last week or two.
I have a testcase now. I no longer think this is a regression. The testcase triggers two assertions: one while loading, and one upon GC: ###!!! ASSERTION: DOM window should always have a document!: 'domDoc', file /Users/jruderman/trunk/mozilla/toolkit/components/passwordmgr/base/nsPasswordManager.cpp, line 840 ###!!! ASSERTION: window not set for document!: 'Error', file /Users/jruderman/trunk/mozilla/layout/base/nsDocumentViewer.cpp, line 1245 It's interesting that I first noticed this because of an assertion in password manager rather than an assertion in core. Should there be a similar assertion somewhere in core?
Assignee: nobody → general
Blocks: 348483
Component: DOM → XBL
Keywords: testcase
Summary: "ASSERTION: DOM window should always have a document" → "ASSERTION: DOM window should always have a document" with XBL and <frame>
Attached file testcase (deleted) —
I really can't tell how we're getting into this state. And since imo the appendChild call in question should be throwing, I'm disinclined to try to figure out exactly what's going on; we should just fix the real issue, which is that we can stick this frame in a stupid place.
This testcase creates the <frame> using createElement, and still triggers both assertions. So enforcing a rule "you may not move an element from the main document into an XBL shadow tree" would not take care of the problem reported here.
I'm not getting any of these assertions in my mingw debug build.
Blocks: 379374
WFM on trunk.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → WORKSFORME
Flags: in-testsuite?
I checked in the second testcase as a crashtest.
Flags: in-testsuite? → in-testsuite+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: