Touch event isn't fired on Fission oop iframe
Categories
(Core :: DOM: UI Events & Focus Handling, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox80 | --- | fixed |
People
(Reporter: edgar, Assigned: edgar)
References
(Blocks 1 open bug)
Details
Attachments
(3 files)
STR:
- Enable fission
- Load https://codepen.io/edgarchen-the-decoder/pen/eYJWqRe
- Touch green area with finger on touch screen
Expected Result:
touch event logged
Actual Result:
No touch event logged
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 1•4 years ago
|
||
Assignee | ||
Comment 2•4 years ago
|
||
This was used in B2G and no one uses it now.
Assignee | ||
Comment 3•4 years ago
|
||
Assignee | ||
Comment 4•4 years ago
|
||
Updated•4 years ago
|
Assignee | ||
Comment 5•4 years ago
|
||
Other than the layers id isn't set properly for touch events, I also found that maybe we have some other issues for multiple touches.
For multiple touches, per https://w3c.github.io/touch-events/#touchevent-implementer-s-note, all touch events should be dispatched to the same document that first touch event associated with. So for example, if we initialize the first touch inside iframe, and then initialize the second touch outside the iframe, the touch event of the second touch should be dispatch to iframe too.
Assignee | ||
Comment 6•4 years ago
|
||
(In reply to Edgar Chen [:edgar] from comment #5)
Other than the layers id isn't set properly for touch events, I also found that maybe we have some other issues for multiple touches.
For multiple touches, per https://w3c.github.io/touch-events/#touchevent-implementer-s-note, all touch events should be dispatched to the same document that first touch event associated with. So for example, if we initialize the first touch inside iframe, and then initialize the second touch outside the iframe, the touch event of the second touch should be dispatch to iframe too.
Okay, we suppress the touch that is outside the frame and we behave the same without Fission, so I think this is just what we behave now.
Updated•4 years ago
|
Assignee | ||
Comment 7•4 years ago
|
||
Pushed by echen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/70f5cc27a0c4 Part 1: Remove MultiTouchInput::ToWidgetMouseEvent; r=kats https://hg.mozilla.org/integration/autoland/rev/e5c2bc278f07 Part 2: Use EvictTouches to evict old touches; r=kats https://hg.mozilla.org/integration/autoland/rev/d23fbb1a4fd8 Part 3: Propagate layers id to touch event properly for Fission; r=kats
Comment 9•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/70f5cc27a0c4
https://hg.mozilla.org/mozilla-central/rev/e5c2bc278f07
https://hg.mozilla.org/mozilla-central/rev/d23fbb1a4fd8
Description
•