Closed Bug 1148581 Opened 6 years ago Closed 6 years ago

APZ ignores outer clip in this testcase

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla41

People

(Reporter: mstange, Unassigned)

References

Details

(Whiteboard: [gfx-noted])

Attachments

(1 file)

Attached file testcase
In this testcase, the red box should never leave the white box.
Whiteboard: [gfx-noted]
Going to take this because I need it fixed to be able to test bug 1148582, and I want to make sure that my fix for bug 1145730 fixes bug 1148582 as well.
Assignee: nobody → botond
This looks like a Layout issue - Layout is setting the clip for the container layer enclosing the red image layer differently with APZ enabled than with APZ disabled.
Component: Panning and Zooming → Layout
I bet we're failing to apply the clip of the outer overflow: hidden element because we clear the clipstate when we build the display list for the actually scrolling element, and then we add back in the clip for actively scrolling scroll frames that are parents of the scrolled content. Maybe we should save the current clip state on the scroll frame (so that we include overflow: hidden clips, but also CSS clip properties and any other clipping) and then apply that in ScrollFrameHelper::ComputeFrameMetrics.
OS: Mac OS X → All
Hardware: x86 → All
The plan in bug 1148581 does comment 3 and more.
(In reply to Timothy Nikkel (:tn) from comment #4)
> The plan in bug 1148581 does comment 3 and more.

The plan in bug 1148582 does comment 3 and more.
Updating dependencies to reflect the fact that the plan in bug 1148582 will fix this as well. Let's keep this open until bug 1148582 lands, and verify that the STR here is indeed fixed before closing.
Assignee: botond → nobody
No longer blocks: 1148582
Depends on: 1148582
Fixed by bug 1148582.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in before you can comment on or make changes to this bug.