9738 - Crash reordering panels in Configure dialog

Status: VERIFIED FIXED

(Core Graveyard :: Tracking, defect, P3)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Tested on:
 * 1999-07-12-18-M8 apprunner, Linux, from mozilla.org

Steps to duplicate:
 * open apprunner
 * if sidebar is already open (which is stored in prefs, I think), hit configure
at top of sidebar
 * hit "Move Down"

apprunner crashes with the following stack:

#0  0x40a1fb2b in nsCSSFrameConstructor::ContentInserted ()
#1  0x40aa3b22 in StyleSetImpl::ContentInserted ()
#2  0x40972bf6 in PresShell::ContentInserted ()
#3  0x40733b68 in XULDocumentImpl::ContentInserted ()
#4  0x40981ee1 in nsGenericHTMLContainerElement::InsertChildAt ()
#5  0x4098180f in nsGenericHTMLContainerElement::InsertBefore ()
#6  0x409c8e51 in nsHTMLSelectElement::InsertBefore ()
#7  0x402a1fed in NS_NewScriptNamedNodeMap ()
#8  0x40316272 in js_Invoke ()
#9  0x4031bf0e in js_Interpret ()
#10 0x403162c0 in js_Invoke ()
#11 0x4031bf0e in js_Interpret ()
#12 0x403162c0 in js_Invoke ()
#13 0x40316495 in js_InternalCall ()
#14 0x4030131d in JS_CallFunctionValue ()
#15 0x402a4adc in nsJSEventListener::HandleEvent ()
#16 0x4094a269 in nsEventListenerManager::HandleEvent ()
#17 0x40a8ea44 in nsGenericElement::HandleDOMEvent ()
#18 0x4098e4da in nsHTMLButtonElement::HandleDOMEvent ()
#19 0x4094bf3c in nsEventStateManager::CheckForAndDispatchClick ()
#20 0x4094b31c in nsEventStateManager::PostHandleEvent ()
#21 0x40973416 in PresShell::HandleEvent ()
#22 0x40b647bd in nsView::HandleEvent ()
#23 0x40b6bfad in nsViewManager::DispatchEvent ()
etc...

Comment 1

[chris hofmann]

-> slamm.   can you have a look?

Comment 2

[Steve Lamm]

This does not surprise me. The customize dialog needs a lot of work.

Comment 3

[Chris Waterson]

slamm, no matter what, XUL shouldn't be crashing on content. Feel free to
reassign to me.

Comment 4

[Steve Lamm]

I will take a closer look at this today.

Comment 5

[Steve Lamm]

*** Bug 9760 has been marked as a duplicate of this bug. ***

Comment 6

[Chris Waterson]

Okay, from the stack trace, I'm guessing that we've either got some bogus style
rules or we're setting up an incorrect content model. Can you make sure that
we're including xul.css, and that we're not doing anything funky like putting
TREECELLs in the wrong place, etc.? cc'ing hyatt, too.

Comment 7

[Steve Lamm]

Looks like its a bug for troy. I filled another one on him.

Comment 8

[Steve Lamm]

*** Bug 9904 has been marked as a duplicate of this bug. ***

Comment 9

[Steve Lamm]

Fixed.

Comment 10

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Verified fixed.