touch action regions are ignored inside opacity:0 divs

RESOLVED FIXED in Firefox 66

Status

()

defect
P3
normal
RESOLVED FIXED
9 months ago
3 months ago

People

(Reporter: kats, Assigned: mattwoodrow)

Tracking

({regression})

64 Branch
mozilla66
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox-esr60 wontfix, firefox63 wontfix, firefox64 wontfix, firefox65 wontfix, firefox66 fixed)

Details

(Whiteboard: [gfx-noted], )

Attachments

(3 attachments)

Spinoff from bug 1500864 comment 5.

See the URL for the testcase. The STR are to (1) Load this page in Fennec and (2) try to scroll by starting your finger inside the black box. If you can scroll, that's bad. If you can't scroll, that means the touch-action:none is successfully preventing the scroll, which is good.

I ran mozregression on Android and found that this was broken by bug 1434243.
Marking fix-optional for 64. We could still take a patch for 65, and if it's verified and doesn't seem risky, could still take fixes for 64 as well.
Assignee: nobody → matt.woodrow
Blocks: 1500864
Pushed by mwoodrow@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c62d53bdab79
Build display items within opacity:0 if we need them for APZ hit testing. r=kats
https://hg.mozilla.org/integration/autoland/rev/0d3d0fa680d8
Cull items within an opacity:0 container when using WebRender. r=kats
Is this something you wanted to consider for Beta uplift or should it ride the trains?
Flags: needinfo?(matt.woodrow)
Flags: in-testsuite+
This has been broken for a while without anyone complaining, so I don't think there's a huge rush to uplift.
Flags: needinfo?(matt.woodrow)
Depends on: 1535585
No longer depends on: 1535585
Regressions: 1535585
You need to log in before you can comment on or make changes to this bug.