Jesse Ruderman Reporter
	Description • 19 years ago

Steps to reproduce: Load testcase in a (trunk) debug build.

<window xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
  <grid>
    <popupgroup>
    </popupgroup>
  </grid>
</window>

Result: two assertion failures, with the same text but different line numbers.

###!!! ASSERTION: grandparent should be root box: 'NS_SUCCEEDED(res)', file /Users/admin/trunk/mozilla/layout/xul/base/src/nsPopupSetFrame.cpp, line 155

###!!! ASSERTION: grandparent should be root box: 'NS_SUCCEEDED(res)', file /Users/admin/trunk/mozilla/layout/xul/base/src/nsPopupSetFrame.cpp, line 194

	Jesse Ruderman Reporter
	Comment 2 • 17 years ago

Still happens on trunk, post bug 279703 landing.

	Jesse Ruderman Reporter
	Comment 3 • 17 years ago

Now it also triggers:

###!!! ASSERTION: Unexpected PopupSetFrame: 'rootBox->GetPopupSetFrame() == newFrame', file /Users/jruderman/trunk/mozilla/layout/base/nsCSSFrameConstructor.cpp, line 6167

(That assertion also appears in bug 321058.)

	Mats Palmgren (inactive)
	Comment 4 • 17 years ago

The first testcase doesn't trigger the assertion for me (on Linux and Mac).
This testcase does (and also triggers bug 385845).

	Jesse Ruderman Reporter
	Comment 5 • 17 years ago

Same here.

	Neil Deakin
	Comment 6 • 16 years ago

This shouldn't be happening any more now that 400185 is fixed.

	Jesse Ruderman Reporter
	Comment 7 • 16 years ago

Neil, I still see this bug using "Testcase #2".

	Jesse Ruderman Reporter
	Comment 8 • 15 years ago

The "grandparent should be root box" assertion was removed in bug 421203:
http://hg.mozilla.org/mozilla-central/rev/4a73efb88edc

When I load these testcases now I only get the assertion in bug 385845.

	Jesse Ruderman Reporter
	Comment 9 • 15 years ago

Crashtest: http://hg.mozilla.org/mozilla-central/rev/b64a7e4a35e2