touch action regions are ignored inside opacity:0 divs

RESOLVED FIXED in Firefox 66

Status

()

P3
normal
RESOLVED FIXED
5 months ago
4 days ago

People

(Reporter: kats, Assigned: mattwoodrow)

Tracking

(Depends on: 1 bug, {regression})

64 Branch
mozilla66
regression
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], URL)

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.
status-firefox63: --- → wontfix
status-firefox64: --- → affected
status-firefox65: --- → affected
status-firefox-esr60: --- → affected
Keywords: regression
Priority: -- → P3
Whiteboard: [gfx-noted]
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.
status-firefox64: affected → fix-optional
(Assignee)

Updated

3 months ago
Assignee: nobody → matt.woodrow
(Assignee)

Updated

3 months ago
Blocks: 1500864

Comment 5

3 months ago
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
https://hg.mozilla.org/mozilla-central/rev/c62d53bdab79
https://hg.mozilla.org/mozilla-central/rev/0d3d0fa680d8
https://hg.mozilla.org/mozilla-central/rev/8c491aa06a8e
Status: NEW → RESOLVED
Last Resolved: 3 months ago
status-firefox66: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
Is this something you wanted to consider for Beta uplift or should it ride the trains?
status-firefox64: fix-optional → wontfix
status-firefox-esr60: affected → wontfix
Flags: needinfo?(matt.woodrow)
Flags: in-testsuite+
(Assignee)

Comment 9

3 months ago
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)
status-firefox65: affected → wontfix
Depends on: 1535585
You need to log in before you can comment on or make changes to this bug.