Closed Bug 704423 Opened 13 years ago Closed 10 years ago

mousemove event should be cancelable

Categories

(Core :: DOM: Events, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla31

People

(Reporter: masayuki, Assigned: masayuki)

References

(Blocks 2 open bugs, )

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Patch (obsolete) — Splinter Review
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)
Btw, I'm not sure if this is just a bug in D3E.
DOM 2 mousemove is not cancelable.
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)
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
Attached patch PatchSplinter Review
Okay, let's take this into 31 which is next ESR.
Attachment #576366 - Attachment is obsolete: true
Attachment #8411624 - Flags: review?(bugs)
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/mozilla-central/rev/36f02c47f4be
Status: ASSIGNED → RESOLVED
Closed: 10 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.