1655239 - setPointerCapture does not work across Fission frames

Status: RESOLVED FIXED

(Core :: DOM: UI Events & Focus Handling, defect, P2)

Description

[robbendebiene]

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:78.0) Gecko/20100101 Firefox/78.0

Steps to reproduce:

Using element.setPointerCapture on an element inside an iframe normally re-targets all mouse events across frames to one particular element.

Best way to test this is using the example on MDN here: https://developer.mozilla.org/en-US/docs/Web/API/Element/setPointerCapture

This does not work when Fission is enabled.

Actual results:

The mouse movement is not detected outside of the iframe.

Expected results:

The mouse events should be redirected to the element inside the iframe on which setPointerCapture was called on.

Comment 1

[Tomislav Jovanovic :zombie]

Thank you for the report, I've confirmed this on Nightly 80 with Fission. Probably a blocker for Nightly or Beta rollout.

Comment 3

[Chris Peterson [:cpeterson]]

Fission M6c

Comment 4

[Neha Kochar [:neha]]

Olli reported the failure with pointer capture: http://mozilla.pettay.fi/moztests/pointercapture_parent.html

Comment 5

[Edgar Chen [:edgar]]

This seems like a similar issue as the pointer lock bug we had in bug 1662587, i.e. the mouse event doesn't dispatch to the right process. I will take a look.

Comment 6

[Edgar Chen [:edgar]]

Attachment: Bug 1655239 - Make pointer capture fission compatible;

Comment 7

[Edgar Chen [:edgar]]

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

Comment 8

[Edgar Chen [:edgar]]

I tried to write a test for implicit pointer capture for touch event, and just realized we didn't support dispatching touch event asynchronously.

Comment 9

[Edgar Chen [:edgar]]

(In reply to Edgar Chen [:edgar] from comment #8)

I tried to write a test for implicit pointer capture for touch event, and just realized we didn't support dispatching touch event asynchronously.

I did some quick try but failed to make it work, file bug 1672726 to add tests for implicit pointer capture for fission.

Comment 10

[Edgar Chen [:edgar]]

I have verified manually on Surface Pro that implicit pointer capture for touch events works on Fission.

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/82dc435b2c6d
Make pointer capture fission compatible; r=smaug

Comment 12

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/82dc435b2c6d