mousemove event should be cancelable

RESOLVED FIXED in mozilla31

Status

()

Core
DOM: Events
RESOLVED FIXED
6 years ago
3 years ago

People

(Reporter: masayuki, Assigned: masayuki)

Tracking

(Blocks: 2 bugs)

Trunk
mozilla31
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

mousemove event should be cancelable, see D3E spec:
http://dev.w3.org/2006/webapi/DOM-Level-3-Events/html/DOM3-Events.html#event-type-mousemove
Created attachment 576366 [details] [diff] [review]
Patch

This patch doesn't change actual behavior, just changes as cancelable.

IE9 and Opera support cancelable mousemove event. However, on both of them, :hover state isn't prevented even if I prevents default of mousemove event.

And also, text selection isn't blocked by the default prevented mousemove event.

WebKit's mousemove event is not cancelable.
Attachment #576366 - Flags: review?(bugs)

Comment 2

6 years ago
Btw, I'm not sure if this is just a bug in D3E.
DOM 2 mousemove is not cancelable.

Comment 3

5 years ago
Comment on attachment 576366 [details] [diff] [review]
Patch

You did send email to the mailing list, but did you perhaps file a 
spec bug?
Clearing r? until we know what to do with this.
Attachment #576366 - Flags: review?(bugs)
I filed a spec bug:
https://www.w3.org/Bugs/Public/show_bug.cgi?id=17626

Comment 5

4 years ago
In Blink, it seems that this bug is fixed.
https://src.chromium.org/viewvc/blink?revision=157207&view=revision
https://codereview.chromium.org/23463012
https://code.google.com/p/chromium/issues/detail?id=261449

I confimed this in Chromium 31.0.1624.0 (221710)(Blink 537.36 (@157356)).
Also, the spec bug has been resolved from way back.
Any progress? Chrome and IE are consistent with D3E, and Firefox still with D2E. Hmm... now I see that Firefox have correct behaviour a long time ago:
https://bugzilla.mozilla.org/show_bug.cgi?id=76929 << contains a good testcase
Created attachment 8411624 [details] [diff] [review]
Patch

Okay, let's take this into 31 which is next ESR.
Attachment #576366 - Attachment is obsolete: true
Attachment #8411624 - Flags: review?(bugs)

Comment 9

3 years ago
Why should mousemove be cancelable?
Comment on attachment 8411624 [details] [diff] [review]
Patch

Ah, hmm, selection handling might in theory need this.
Attachment #8411624 - Flags: review?(bugs) → review+
(In reply to Olli Pettay [:smaug] from comment #9)
> Why should mousemove be cancelable?

For the odd spec change... We discussed about this in the last D3E meeting, though.
https://hg.mozilla.org/integration/mozilla-inbound/rev/36f02c47f4be
https://hg.mozilla.org/mozilla-central/rev/36f02c47f4be
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
You need to log in before you can comment on or make changes to this bug.