341431 - ###!!! ASSERTION: Event listener added to outer window!: 'win->IsInnerWindow()', file c:/mozilla.org/baseline/mozilla/content/events/src/nsEventListenerManager.cpp, line 1250

Status: RESOLVED FIXED

(Core :: DOM: Events, defect)

Description

[Alex Vincent [:WeirdAl]]

Steps to reproduce:
(1) seamonkey -inspector
(2) Make sure View -> Chrome is enabled.
(3) Inspect DOM Inspector
(4) Shut down SeaMonkey.

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060613 SeaMonkey/1.5a

This assertion was added by jst for bug 296639.

 	ntdll.dll!7c901230() 	
 	[Frames below may be incorrect and/or missing, no symbols loaded for ntdll.dll]	
 	xpcom_core.dll!Break(const char * aMsg=0x0012daac)  Line 471	C++
 	xpcom_core.dll!NS_DebugBreak_P(unsigned int aSeverity=1, const char * aStr=0x01e16178, const char * aExpr=0x01e16160, const char * aFile=0x01e16110, int aLine=1250)  Line 350 + 0xc bytes	C++
>	gklayout.dll!nsEventListenerManager::AddScriptEventListener(nsISupports * aObject=0x0215f278, nsIAtom * aName=0x009fba80, const nsAString_internal & aBody={...}, unsigned int aLanguage=2, int aDeferCompilation=0, int aPermitUntrustedEvents=0)  Line 1250 + 0x2f bytes	C++
 	gklayout.dll!nsGenericElement::AddScriptEventListener(nsIAtom * aEventName=0x009fba80, const nsAString_internal & aValue={...}, int aDefer=1)  Line 3192 + 0x46 bytes	C++
 	gklayout.dll!nsXULElement::AfterSetAttr(int aNamespaceID=0, nsIAtom * aName=0x009fba80, const nsAString_internal * aValue=0x0012e278, int aNotify=0)  Line 1211	C++
 	gklayout.dll!nsGenericElement::SetAttr(int aNamespaceID=0, nsIAtom * aName=0x009fba80, nsIAtom * aPrefix=0x00000000, const nsAString_internal & aValue={...}, int aNotify=0)  Line 3355 + 0x26 bytes	C++
 	gklayout.dll!nsXULDocument::SynchronizeBroadcastListener(nsIDOMElement * aBroadcaster=0x02e622cc, nsIDOMElement * aListener=0x02e3315c, const nsAString_internal & aAttr={...})  Line 674	C++
 	gklayout.dll!nsXULDocument::RemoveBroadcastListenerFor(nsIDOMElement * aBroadcaster=0x02e622cc, nsIDOMElement * aListener=0x02e3315c, const nsAString_internal & aAttr={...})  Line 817	C++
 	gklayout.dll!nsXULDocument::RemoveSubtreeFromDocument(nsIContent * aElement=0x02e33140)  Line 1719	C++
 	gklayout.dll!nsXULDocument::RemoveSubtreeFromDocument(nsIContent * aElement=0x02e38ff0)  Line 1687 + 0x20 bytes	C++
 	gklayout.dll!nsXULDocument::RemoveSubtreeFromDocument(nsIContent * aElement=0x0305dc30)  Line 1687 + 0x20 bytes	C++
 	gklayout.dll!nsXULDocument::RemoveSubtreeFromDocument(nsIContent * aElement=0x0305d918)  Line 1687 + 0x20 bytes	C++
 	gklayout.dll!nsXULDocument::RemoveSubtreeFromDocument(nsIContent * aElement=0x0305d868)  Line 1687 + 0x20 bytes	C++
 	gklayout.dll!nsXULDocument::RemoveSubtreeFromDocument(nsIContent * aElement=0x03056240)  Line 1687 + 0x20 bytes	C++
 	gklayout.dll!nsXBLBinding::ChangeDocument(nsIDocument * aOldDocument=0x022e37d0, nsIDocument * aNewDocument=0x00000000)  Line 943	C++
 	gklayout.dll!nsBindingManager::ChangeDocumentFor(nsIContent * aContent=0x02e43cc0, nsIDocument * aOldDocument=0x022e37d0, nsIDocument * aNewDocument=0x00000000)  Line 429	C++
 	gklayout.dll!nsXULElement::UnbindFromTree(int aDeep=1, int aNullParent=0)  Line 954	C++
 	gklayout.dll!nsXULElement::UnbindFromTree(int aDeep=1, int aNullParent=0)  Line 997	C++
 	gklayout.dll!nsXULElement::UnbindFromTree(int aDeep=1, int aNullParent=0)  Line 997	C++
 	gklayout.dll!nsXULElement::UnbindFromTree(int aDeep=1, int aNullParent=1)  Line 997	C++
 	gklayout.dll!nsDocument::Destroy()  Line 4817	C++
 	gklayout.dll!DocumentViewerImpl::Close(nsISHEntry * aSHEntry=0x00000000)  Line 1410	C++
 	docshell.dll!nsDocShell::Destroy()  Line 3602	C++
 	appshell.dll!nsXULWindow::Destroy()  Line 507	C++
 	appshell.dll!nsWebShellWindow::Destroy()  Line 834 + 0x9 bytes	C++
 	appshell.dll!nsWebShellWindow::HandleEvent(nsGUIEvent * aEvent=0x0012f0ac)  Line 393	C++
 	gkwidget.dll!nsWindow::DispatchEvent(nsGUIEvent * event=0x0012f0ac, nsEventStatus & aStatus=nsEventStatus_eIgnore)  Line 1102 + 0xc bytes	C++
 	gkwidget.dll!nsWindow::DispatchWindowEvent(nsGUIEvent * event=0x0012f0ac)  Line 1123	C++
 	gkwidget.dll!nsWindow::DispatchStandardEvent(unsigned int aMsg=101)  Line 1142 + 0x11 bytes	C++
 	gkwidget.dll!nsWindow::ProcessMessage(unsigned int msg=16, unsigned int wParam=0, long lParam=0, long * aRetValue=0x0012f498)  Line 4226	C++
 	gkwidget.dll!nsWindow::WindowProc(HWND__ * hWnd=0x000e0142, unsigned int msg=16, unsigned int wParam=0, long lParam=0)  Line 1291 + 0x1d bytes	C++
 	user32.dll!77d48734() 	
 	user32.dll!77d48816() 	
 	user32.dll!77d4b4c0() 	
 	user32.dll!77d4b50c() 	
 	ntdll.dll!7c90eae3() 	
 	user32.dll!77d494be() 	
 	user32.dll!77d4b42d() 	
 	user32.dll!77d4baa4() 	
 	user32.dll!77d4b96b() 	
 	user32.dll!77d4b3f9() 	
 	uxtheme.dll!5ad73c20() 	
 	uxtheme.dll!5ad8e300() 	
 	uxtheme.dll!5ad71ac7() 	
 	uxtheme.dll!5ad71b3d() 	
 	uxtheme.dll!5ad8e2d5() 	
 	user32.dll!77d4bb15() 	
 	nspr4.dll!_MD_CURRENT_THREAD()  Line 296 + 0xc bytes	C
 	gkwidget.dll!nsWindow::DefaultWindowProc(HWND__ * hWnd=0x000e0142, unsigned int msg=274, unsigned int wParam=61536, long lParam=1245803)  Line 1312	C++
 	user32.dll!77d48734() 	
 	user32.dll!77d48816() 	
 	gkwidget.dll!nsBaseWidget::QueryInterface(const nsID & aIID={...}, void * * aInstancePtr=0x000e0142)  Line 64 + 0x84 bytes	C++
 	user32.dll!77d4c63f() 	
 	user32.dll!77d4c665() 	
 	gkwidget.dll!nsWindow::WindowProc(HWND__ * hWnd=0x0215e2e4, unsigned int msg=1243400, unsigned int wParam=38545384, long lParam=0)  Line 1298 + 0x1f bytes	C++
 	nspr4.dll!PR_GetCurrentThread()  Line 175	C
 	gkwidget.dll!nsWindow::`vftable'()  + 0x23 bytes	C++
 	gkwidget.dll!02469aaf() 	
 	4d890cec()

Comment 1

[:Gijs (he/him)]

Just closing DOMI is enough in a 1.8 branch build of firefox w/ DOMI. I get 4 of them whenever I close DOM Inspector.

Comment 2

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

This should be fixed on trunk with the patch from bug 348156

Comment 3

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Should be fixed by patch in bug 348156