42418 - [CRASH]Closing a alert dialog generated by onmousedown event causes a crash.

Status: VERIFIED FIXED

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

Description

[Chris Petersen]

Build: 2000061210
Platforms: All
Expect results: The dialog should close and application should remain open.
What I got: Closing this dialog causes a crash.

Steps to reproduce:
1) Open either test case attached: select_onmousedown or option_onmousedown.

2) Mouse down on list box (select) or menu item (option) which have been assigned 
a onmousedown event.

3) When this event is processed, a javascript alert dialog should appear. Click 
OK to close this dialog.

4) The crash should occur.

Comment 1

[Chris Petersen]

Attachment: A select element with a onmousedown event.

Comment 2

[Chris Petersen]

Attachment: A option element assigned with a onmousedown event.

Comment 3

[Chris Petersen]

Changing QA contact to myself.

Comment 4

[Chris Petersen]

This bug is related to 31891 (onmousedown event) which was fixed several months 
ago. It appears to have come back.

Comment 5

[leger]

Adding crash keyword

Comment 6

[joki (gone)]

Weird.  Not crashing in my working tree but is crashing in my verification tree.  
Trying to figure out why.

Comment 7

[joki (gone)]

Okay, I'm not entirely sure why but my latest unmodified tip build works for 
this now on WinNT.  Build before was about a day old and it crashed.  Go figure.  
Marking WORKSFORME.

Comment 8

[Chris Petersen]

The application no longer crashes after closing the alert dialog. Marking 
verified works for me in the June 21 build.

Comment 9

[joki (gone)]

This bug is not crashing due to an unintentional change of script alerts from 
modal to non-modal.  They will be changing back at which point this crashes 
again.

Reopening.

Comment 10

[Chris Petersen]

Nominating to nsbeta2. The problem is that closing a alert dialog that was 

generated by a onmousedown event causes a crash. Closing a alert dialog shouldn't 

cause the application to crash.

Comment 11

[Jay Patel [:jay]]

Putting on [nsbeta2+] radar for beta2 fix. 

Comment 12

[joki (gone)]

Okay, this is fixed now.

Comment 13

[Chris Petersen]

Fixed in the June 30th builds.