Closed
Bug 750111
Opened 13 years ago
Closed 13 years ago
mozMovementX is transient and becomes zero later
Categories
(Core :: DOM: Events, defect)
Core
DOM: Events
Tracking
()
RESOLVED
FIXED
mozilla15
People
(Reporter: azakai, Assigned: humph)
References
(Blocks 1 open bug)
Details
Attachments
(2 files, 1 obsolete file)
(deleted),
text/html
|
Details | |
(deleted),
patch
|
humph
:
review+
akeybl
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
mousemove events have valid mozMovementX and Y values, but if that event object is saved somewhere and processed later, the value of mozMovementX is 0. This is quite surprising and a problem for web pages that want to process the event info later.
For example, if you receive such an event and do a setTimeout whose closure captures the event, the timeout will print 0 for mozMovementX.
The attached testcase shows this, search for 'zz' to see the relevant lines. Looking in the web console it is clear mozMovementX is valid when received, but zero'd out later.
Reporter | ||
Updated•13 years ago
|
Blocks: gecko-games, 633602
Comment 1•13 years ago
|
||
nsDOMUIEvent::DuplicatePrivateData() should handle also movementX/Y
Updated•13 years ago
|
tracking-firefox14:
--- → ?
Assignee | ||
Comment 3•13 years ago
|
||
As an aside, this test case is triggering bug 731350 for me when I go fullscreen.
Assignee | ||
Comment 4•13 years ago
|
||
Attachment #620296 -
Flags: review?(bugs)
Updated•13 years ago
|
Attachment #620296 -
Flags: review?(bugs) → review+
Comment 5•13 years ago
|
||
seems like there's a rash of tracking flags being reset going around. fixing. we're tracking this for 14.
Assignee | ||
Comment 6•13 years ago
|
||
I wanted to run this through try server before updating, but now that I've done that, this looks good.
https://tbpl.mozilla.org/?tree=Try&rev=6d128e6de11b
Attachment #620296 -
Attachment is obsolete: true
Attachment #622237 -
Flags: review+
Assignee | ||
Updated•13 years ago
|
Keywords: checkin-needed
Version: unspecified → Trunk
Comment 7•13 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/4d1a47d43ce5
Should this have an automated test?
Comment 8•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
status-firefox14:
--- → affected
Comment 9•13 years ago
|
||
Please nominate for Aurora approval so we can uplift.
Comment 10•12 years ago
|
||
Comment on attachment 622237 [details] [diff] [review]
Patch v2 (r=smaug)
[Approval Request Comment]
Bug caused by (feature/regressing bug #): NA
User impact if declined: buggy behavior in certain cases
Testing completed (on m-c, etc.): in m-c about a month
Risk to taking this patch (and alternatives if risky): very low risk
String or UUID changes made by this patch: NA
Attachment #622237 -
Flags: approval-mozilla-beta?
Comment 11•12 years ago
|
||
Comment on attachment 622237 [details] [diff] [review]
Patch v2 (r=smaug)
[Triage Comment]
Low risk fix for new feature. Approved for Beta 14.
Attachment #622237 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 12•12 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•