321098 - onmouseover alert() causing multiple stacked alert dialogs

Status: RESOLVED FIXED

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

Description

[Clifford Barney]

User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1; .NET CLR 1.1.4322)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8) Gecko/20051111 Firefox/1.5

If a form button with an onMouseover alert() function is positioned near the center of the page (horizontally and vertically), when the alert pop-up appears, it partially overlaps the form button. This overlapping causes the onMouseOver event to continuously fire, creating multiple stacked alert windows. Because they are stacked, the user only sees a single alert window that doesn't close no matter how many times he clicks the 'close' button.

Reproducible: Always

Steps to Reproduce:
1. Go to http://www.dawginc.com/docs/mozilla_bug.php
2. Follow instructions
Actual Results:  
Depending upon where the form button is positioned using the vertical scrollbar, I can make the problem appear or not appear.

Expected Results:  
A single alert window should appear, with focus, requiring the user to close to proceed with any new action on the page.

Comment 1

[:Gavin Sharp [email: [e-mail]]]

Indeed, something is going wrong with that testcase.

Comment 2

[:Gavin Sharp [email: [e-mail]]]

Works in 1.0.7, I suspect bug 297561.

Comment 3

[Ere Maijala (slow)]

Attachment: Win32 patch v1

In the continuation of Win32 mouse handling fixes, this patch makes the behavior match that of Linux, I believe. I suspect the order of mouse messages there is EXIT -> ENTER -> MOVE instead of what we had on Windows (MOVE -> EXIT -> ENTER), but haven't got a Linux build yet to verify (build underway). Anyway, having messages in this order makes sense. Otherwise we're processing move in a widget that hasn't been entered yet. 

This doesn't fix the problem completely however. Both Linux and Win32 show two dialogs, but it's not in my domain of knowledge to fix that.

Comment 4

[Ere Maijala (slow)]

Oh, and the patch is for trunk. 

Comment 5

[Ere Maijala (slow)]

I verified that GTK has it as I suspected (EXIT -> ENTER -> MOVE).

Comment 6

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Ere, are you going to land this?

Comment 7

[Ere Maijala (slow)]

Oops, I've somehow lost this one. Anyway, the patch was checked in to trunk but branch needs its own version. Need to test because there are some differences.

Comment 8

[Ere Maijala (slow)]

I don't think it's feasible to try to get this into branch anymore. Anyone disagree?