1411467 - Keeping PointerEvent enabled in release build

Status: RESOLVED FIXED

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

Comment 1

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

Stone, the merge day is Nov. 13, let's have this in by then for smooth train riding and get to more users.

Comment 2

[Ming-Chou Shih [:stone]]

Attachment: Bug 1411467 - Keeping PointerEvent enabled in early beta

Comment 3

[Ming-Chou Shih [:stone]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=d9ac2c6a5a630ae1e145a4a52065dbba33bf6277

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2ed0ff57330d
Keeping PointerEvent enabled in early beta. r=smaug.

Comment 5

[Eliza Balazs [:ebalazs_]]

https://hg.mozilla.org/mozilla-central/rev/2ed0ff57330d

Comment 6

[Ryan VanderMeulen [:RyanVM]]

This patch didn't take test_interfaces into account at all and unsurprisingly, it's burning Beta badly. Backing out because we can't have burning Beta builds with b3 gtb scheduled for Monday.
https://treeherder.mozilla.org/logviewer.html#?job_id=143859638&repo=mozilla-beta

https://hg.mozilla.org/mozilla-central/rev/933f9cd9b3b9030399a11c19cb4e5117b29e2772

Comment 7

[Ming-Chou Shih [:stone]]

Can't only enable some interfaces in test_interfaces.js for early beta. So we plan to enable it in release build and ride the next train (FF59).

Comment 8

[Ming-Chou Shih [:stone]]

Attachment: Keeping PointerEvent enabled in release build

Comment 9

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

Comment on attachment 8932040 [details] [diff] [review]
Keeping PointerEvent enabled in release build

Hmm, why we can't enable on Android too? At least file a bug to enable there too.

Comment 10

[Ming-Chou Shih [:stone]]

Created bug 1426786 for pointer event on android. We still have some works (e.g. fill the pointer event related attributes, fix failed test cases) before enabling it on android.

Comment 11

[Ming-Chou Shih [:stone]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b01aae26d03e6f8be3fa281ccf61fa96f397cd07

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0d6b0f5ffc8f
Keeping PointerEvent enabled in release build. r=smaug.

Comment 13

[Eliza Balazs [:ebalazs_]]

https://hg.mozilla.org/mozilla-central/rev/0d6b0f5ffc8f

Comment 14

[Gerry Chang [:gchang]]

Ride the train on 59, won't fix for 58.

Comment 15

[Chris Mills [:cmills]]

I've documented this:

https://developer.mozilla.org/en-US/Firefox/Releases/59#New_APIs
https://github.com/mdn/browser-compat-data/pull/1066