Closed Bug 321643 Opened 19 years ago Closed 19 years ago

Spurious mousemove events when mouse is over appearing/disappearing fixed positioned div

Categories

(Core :: Widget: Win32, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: martijn.martijn, Assigned: emaijala+moz)

References

Details

(Keywords: regression, testcase)

Attachments

(2 files)

See upcoming testcase.
Move the mouse over the green blinking div and leave it there.
While the mouse is blinking, you should not get any new mousemove events (if you don't move the mouse).
That's not happening in current trunk builds, though.

Probably a regression from bug 312566.
Ria, could you check this? Thanks!
Attached file testcase
Oh my, when am I going to get this right?
Attached patch Patch v1Splinter Review
This patch fixes the detection of move messages caused by widget creation. After my previous changes the detection went wrong because two widget-based coordinates were compared. This patch changes is so that gLastMouseMovePoint is again in screen coordinates and appropriate conversions are used everywhere.
Assignee: win32 → emaijala
Status: NEW → ASSIGNED
Attachment #206996 - Flags: superreview?(roc)
Attachment #206996 - Flags: review?(roc)
Attachment #206996 - Flags: superreview?(roc)
Attachment #206996 - Flags: superreview+
Attachment #206996 - Flags: review?(roc)
Attachment #206996 - Flags: review+
Fix checked in to trunk. Martijn, could you file another bug against Gtk? Linux builds seem to show the same problem although it's milder there (single mousemove only when the div appears).
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Sure, I filed bug 321794 for it.
(In reply to comment #3)
> Oh my, when am I going to get this right?
> 
I hardly dare to ask,
but could this have caused bug 321897 ?
Depends on: 321816
Depends on: 321794
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: