1550462 - Implement pointerrawupdate event

Status: RESOLVED FIXED

(Core :: DOM: Events, enhancement, P3)

Description

[Hsin-Yi Tsai (she/her)[:hsinyi]]

This is for implementing https://w3c.github.io/pointerevents/#the-pointerrawupdate-event

Comment 1

[Jon S]

Also requested on Connect: https://connect.mozilla.org/t5/ideas/support-for-pointerrawupdate/idi-p/89299

Comment 2

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Bug 1550462 - part 1: Define pointerawupdate event r=smaug

This patch just defines the event message, an atom for event listener attribute
and managing whether the window may have a listener.

Depends on D243402

Comment 3

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Bug 1550462 - part 2: Make PresShell::HandleEvent dispatch preceding pointerrawupdate event r=smaug,edgar,#dom-core

Depends on D243403

Comment 4

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Bug 1550462 - part 3: Enable `pointerrawupdate` event r=smaug!

Depends on D243404

Comment 5

[ashley]

Please take care to avoid the following Chrome bugs with pointerrawupdate which so far have remained unfixed:

https://issues.chromium.org/issues/353249342
https://issues.chromium.org/issues/40817651
https://issues.chromium.org/issues/40683756

Comment 6

[Olli Pettay [:smaug][[e-mail]]]

Would it be possible to get a test for https://github.com/w3c/pointerevents/issues/509 in some bug? Maybe a followup

Comment 7

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

(In reply to Olli Pettay [:smaug][[e-mail]] from comment #6)

Would it be possible to get a test for https://github.com/w3c/pointerevents/issues/509 in some bug? Maybe a followup

Filed bug 1965118. I'll work on that once I finished working on current Interop work which should be done by end of June.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/aae164cff6aa
part 1: Define `pointerawupdate` event r=smaug
https://hg.mozilla.org/integration/autoland/rev/5718f7f3067e
part 2: Make `PresShell::HandleEvent` dispatch preceding `pointerrawupdate` event r=smaug,dom-core,edgar

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/93856bf75c44
part 3: Enable `pointerrawupdate` event r=smaug

Comment 10

[agoloman]

https://hg.mozilla.org/mozilla-central/rev/aae164cff6aa
https://hg.mozilla.org/mozilla-central/rev/5718f7f3067e
https://hg.mozilla.org/mozilla-central/rev/93856bf75c44

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f8a2a169b296
Revert "Bug 1550462 - part 3: Enable `pointerrawupdate` event r=smaug" for causing mochitest failures @ test_pointerrawupdate_event_count.html

Comment 12

[amarc]

Backed out for causing mochitest failures @ test_pointerrawupdate_event_count.html

• Backout link
• Push with failures
• Failure Log

Comment 13

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Hmm, xorig test framework is really unstable for pointer events...

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/32ba02fa32ec
part 1: Define pointerawupdate event r=smaug
https://hg.mozilla.org/integration/autoland/rev/37f6acd499b1
part 2: Make PresShell::HandleEvent dispatch preceding pointerrawupdate event r=smaug,dom-core,edgar

Comment 15

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ccd6d2c97e41
part 3: Enable `pointerrawupdate` event r=smaug

Comment 16

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/32ba02fa32ec
https://hg.mozilla.org/mozilla-central/rev/37f6acd499b1
https://hg.mozilla.org/mozilla-central/rev/ccd6d2c97e41

Comment 17

[Donal Meehan [:dmeehan]]

:masayuki could you consider nominating this for a release note? (Process info)

Comment 18

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Release Note Request (optional, but appreciated)
[Why is this notable]: Introducing a new API.
[Affects Firefox for Android]: Yes.
[Suggested wording]: pointerrawupdate event is now supported. The event allows the web apps to handle pointermove immediately. However, this event is fired more frequently than pointermove with another hit test for considering the target. Therefore, even only adding an event listener of pointerrawupdate makes the damage to the performance. So, this is intended to improve apps which cannot handle user input smoothly only with coalesced pointermove events.
[Links (documentation, blog post, etc)]: https://developer.mozilla.org/en-US/docs/Web/API/Element/pointerrawupdate_event

Comment 19

[Donal Meehan [:dmeehan]]

Thanks, added to the Fx140 nightly release notes, please allow 30 minutes for the site to update.
Keeping the relnote-firefox flag as ? to keep it on the radar for inclusion in the final Fx140 release notes.

Comment 20

[Hamish Willee]

FF140 MDN docs work for this done in https://github.com/mdn/content/issues/39614