Closed Bug 942995 Opened 6 years ago Closed 6 years ago
APZC for overflow:hidden element should not allow panning
The RecordFrameMetrics call here: http://mxr.mozilla.org/mozilla-central/source/layout/base/nsDisplayList.cpp#1186 unconditionally assigns a non-null scroll id to the root scroll frame of the root content document. With APZ enabled, this causes an APZC to be created for this frame. If the element corresponding to the root scroll frame is overflow:hidden, this allows the user to scroll the overflow:hidden element by panning. This is undesirable. Since there is no use in having APZC for an overflow:hidden element, we can solve this problem by getting rid of it in such a case (by assigning NULL_SCROLL_ID to the frame).
6 years ago
blocking-b2g: --- → 1.3+
Attachment #8338094 - Flags: review?(tnikkel) → review+
After further discussion (see bug 940691), we don't want to go the route of not having an APZC for an overflow:hidden element, because we always want to have a root APZC. Instead, an APZC for an overflow:hidden element should disallow panning.
Summary: APZC wrongly created for overflow:hidden root scroll frame → APZC for overflow:hidden element should not allow panning
Reposting Kats' patch from bug 940691. Timothy, can you give it another round of feedback in light of our IRC conversation?
Comment on attachment 8339494 [details] [diff] [review] bug942995.patch This seems fine based on what I know now.
Attachment #8339494 - Flags: feedback?(tnikkel) → feedback+
Try push: https://tbpl.mozilla.org/?tree=Try&rev=6c920312c805 (also covers bug 942799).
Comment on attachment 8339494 [details] [diff] [review] bug942995.patch (In reply to Botond Ballo [:botond] from comment #5) > Try push: https://tbpl.mozilla.org/?tree=Try&rev=6c920312c805 (also covers > bug 942799). Looks clean. Flagging for review.
Attachment #8339494 - Flags: review?(tnikkel)
6 years ago
This one, together with the patch for bug 942799 takes care of most of the first wave of problems with APZC that I've seen. The only one that I still see is in the gallery app (bug 944511)
Attachment #8339494 - Flags: review?(tnikkel) → review+
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
This broke touch scrolling in metro for top level frames. If there isn't an easy fix, we should back this out. Unfortunately it made it into this morning's nightly.
Duplicate of this bug: 940691
You need to log in before you can comment on or make changes to this bug.