1399740 - [Pointer Event] Handle the case that invoke setPointerCapture while the page has a locked element

Status: RESOLVED FIXED

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

Description

[Ming-Chou Shih [:stone]]

Spec [1] says 'If this method is invoked while the page has a locked element ([PointerLock]), throw an exception with the name InvalidStateError.'

More discussions in [2].

[1] https://w3c.github.io/pointerevents/#setting-pointer-capture
[2] https://github.com/w3c/pointerevents/issues/176

Comment 1

[Ming-Chou Shih [:stone]]

Spec [3] also says 'When a pointer lock ([PointerLock]) is successfully applied on an element, a user agent must run the steps as if the releasePointerCapture() method has been called if any element is set to be captured or pending to be captured.'

[3] https://w3c.github.io/pointerevents/#implicit-release-of-pointer-capture

Comment 2

[Ming-Chou Shih [:stone]]

Attachment: Handle the case that invoke setPointerCapture while the page has a locked element

Comment 3

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

I filed
https://github.com/w3c/pointerevents/issues/219
and
https://github.com/w3c/pointerevents/issues/220

Comment 4

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

And https://github.com/w3c/pointerevents/issues/221

Comment 5

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

Comment on attachment 8914033 [details] [diff] [review]
Handle the case that invoke setPointerCapture while the page has a locked element

But we can still take this.
We may need to tweak the implementation once https://github.com/w3c/pointerevents/issues/220 gets resolved somehow.

Comment 6

[Ming-Chou Shih [:stone]]

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

Comment 7

[Ming-Chou Shih [:stone]]

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

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/fa3f9e9479ae
[Pointer Event] Handle the case that invokes setPointerCapture while the page has a locked element. r=smaug.

Comment 9

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/fa3f9e9479ae