21192 - Window should close, but get assert.

Status: VERIFIED WORKSFORME

(Core :: DOM: UI Events & Focus Handling, defect, P3)

Description

[kevinyen]

Well, we changed the content a bit, but the bug still exists.  No longer needed
for the Demo, but needed for browser fxnality.

1. Load the link above
2. Click on "MotherSaito" text in middle of page to bring up mail window.
3. In mail window, click on the "send mail" gif in the upper right.  Window
should close, but instead, the browser crashes.

thx,
kevin

Comment 1

[kevinyen]

Sensitive content.  So changing to "netscape confidential".

Comment 2

[Alex Musil]

Stack trace:

nsVoidArray::Count() line 43 + 3 bytes
nsEventListenerManager::HandleEvent(nsIPresContext * 0x036df710, nsEvent *
0x0012f654, nsIDOMEvent * * 0x0012f600, unsigned int 7, nsEventStatus *
0x0012fa74) line 694 + 33 bytes
nsGenericElement::HandleDOMEvent(nsIPresContext * 0x036df710, nsEvent *
0x0012f654, nsIDOMEvent * * 0x0012f600, unsigned int 1, nsEventStatus *
0x0012fa74) line 792
nsHTMLImageElement::HandleDOMEvent(nsHTMLImageElement * const 0x037e02ac,
nsIPresContext * 0x036df710, nsEvent * 0x0012f654, nsIDOMEvent * * 0x00000000,
unsigned int 1, nsEventStatus * 0x0012fa74) line 338
nsEventStateManager::CheckForAndDispatchClick(nsEventStateManager * const
0x037c6e30, nsIPresContext * 0x036df710, nsMouseEvent * 0x0012fb68,
nsEventStatus * 0x0012fa74) line 1338 + 42 bytes
nsEventStateManager::PostHandleEvent(nsEventStateManager * const 0x037c6e30,
nsIPresContext * 0x036df710, nsGUIEvent * 0x0012fb68, nsIFrame * 0x0271ec6c,
nsEventStatus * 0x0012fa74, nsIView * 0x03851e70) line 564 + 24 bytes
PresShell::HandleEvent(PresShell * const 0x037810b4, nsIView * 0x03851e70,
nsGUIEvent * 0x0012fb68, nsEventStatus * 0x0012fa74) line 2577 + 43 bytes
nsView::HandleEvent(nsView * const 0x03851e70, nsGUIEvent * 0x0012fb68, unsigned
int 8, nsEventStatus * 0x0012fa74, int & 0) line 841
nsView::HandleEvent(nsView * const 0x03851f90, nsGUIEvent * 0x0012fb68, unsigned
int 8, nsEventStatus * 0x0012fa74, int & 0) line 826
nsView::HandleEvent(nsView * const 0x038500a0, nsGUIEvent * 0x0012fb68, unsigned
int 8, nsEventStatus * 0x0012fa74, int & 0) line 826
nsView::HandleEvent(nsView * const 0x037cb9e0, nsGUIEvent * 0x0012fb68, unsigned
int 8, nsEventStatus * 0x0012fa74, int & 0) line 826
nsView::HandleEvent(nsView * const 0x037ca120, nsGUIEvent * 0x0012fb68, unsigned
int 8, nsEventStatus * 0x0012fa74, int & 0) line 826
nsView::HandleEvent(nsView * const 0x037808b0, nsGUIEvent * 0x0012fb68, unsigned
int 28, nsEventStatus * 0x0012fa74, int & 0) line 826
nsViewManager::DispatchEvent(nsViewManager * const 0x03783260, nsGUIEvent *
0x0012fb68, nsEventStatus * 0x0012fa74) line 1678
HandleEvent(nsGUIEvent * 0x0012fb68) line 69
nsWindow::DispatchEvent(nsWindow * const 0x037cbef4, nsGUIEvent * 0x0012fb68,
nsEventStatus & nsEventStatus_eIgnore) line 421 + 10 bytes
nsWindow::DispatchWindowEvent(nsGUIEvent * 0x0012fb68) line 442
nsWindow::DispatchMouseEvent(unsigned int 301, nsPoint * 0x00000000) line 3332 +
21 bytes
ChildWindow::DispatchMouseEvent(unsigned int 301, nsPoint * 0x00000000) line
3550
nsWindow::ProcessMessage(unsigned int 514, unsigned int 0, long 1573463, long *
0x0012fdc8) line 2632 + 24 bytes
nsWindow::WindowProc(HWND__ * 0x074d0896, unsigned int 514, unsigned int 0, long
1573463) line 608 + 27 bytes
USER32! 77e71820()

Reassigning to joki who owns events.

Comment 3

[rods (gone)]

This test no longer crashes but there is an assert when bringing up the "send
mail" window.

Comment 4

[ekrock's old account (dead)]

Updating summary to remove mention of crash.

Comment 5

[Kevin McCluskey (gone)]

Here is the  assertion:

###!!! ASSERTION: no script global object: 'mScriptGlobalObject != nsnull", file 
s:\mozilla\rdf\content\src\nsXULDocument.cpp 4977

I get the  same assertion when doing a view source within mozilla

Comment 6

[Kevin McCluskey (gone)]

Chris, could you look at this. It isn't crashing in the HandleDOMEvent anymore. 
It is asserting in s:\mozilla\rdf\content\src\nsXULDocument.cpp 4977.

Comment 7

[leger]

Updating QA Contact.

Comment 8

[Chris Waterson]

no crash, no assert. not sure what changed...

Comment 9

[ckritzer (gone)]

Mass update:  changing qacontact to [e-mail]

Comment 10

[Loco]

Updating QA Contact.

Comment 11

[Hixie (not reading bugmail)]

Reassigning QA Contact for all open and unverified bugs previously under Lorca's
care to Gerardo as per phone conversation this morning.

Comment 12

[Madhur Bhatia]

QA contact updated

Comment 13

[luis rincon]

verified on build 2001-08-13-trunk

Comment 14

[luis rincon]

verified on build 2001-08-13-trunk