Closed Bug 321058 Opened 19 years ago Closed 17 years ago

ASSERTION: Popup set is already defined! Only 1 allowed. (in nsRootBoxFrame::SetPopupSetFrame)

Categories

(Core :: XUL, defect)

PowerPC
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: jruderman, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: assertion, testcase)

Attachments

(2 files)

(deleted), application/vnd.mozilla.xul+xml
Details
(deleted), application/vnd.mozilla.xul+xml
Details
Steps to reproduce: Load the testcase.

Result:

###!!! ASSERTION: Popup set is already defined! Only 1 allowed.: 'Not Reached', file /Users/admin/trunk/mozilla/layout/xul/base/src/nsRootBoxFrame.cpp, line 270

Testcase source:

<window xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
  <popupgroup>
  </popupgroup>
</window>
Summary: ASSERTION: Popup set is already defined! Only 1 allowed. → ASSERTION: Popup set is already defined! Only 1 allowed. (in nsRootBoxFrame::SetPopupSetFrame)
Attached file testcase (deleted) —
Also triggers:

###!!! ASSERTION: Unexpected PopupSetFrame: 'rootBox->GetPopupSetFrame() == newFrame', file /Users/jruderman/trunk/mozilla/layout/base/nsCSSFrameConstructor.cpp, line 6114
This triggers the above assertions, plus:

###!!! ASSERTION: Popup containing block is missing: '!aState.mRootBox', file /Users/jruderman/trunk/mozilla/layout/base/nsCSSFrameConstructor.cpp, line 5993
Still happens on trunk.
I think for this, and bug 321056 we should remove the special <popupgroup> element and just create an anonymous nsPopupSetFrame inside the root instead, or something  like it.

Blocks: 404120
Blocks: 404077
This should have been fixed by bug 400185.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Crashtests checked in.
Flags: in-testsuite+
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: xptoolkit.xul → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: