Closed Bug 1526486 Opened 8 months ago Closed 8 months ago

Green up Android tests with containerless scrolling enabled (part 1)

Categories

(Core :: Panning and Zooming, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: botond, Assigned: botond)

References

Details

Attachments

(4 files)

I haven't gotten all Android tests to pass with containerless scrolling enabled, but I have fixes that green up some of the tests, and I'd like to start getting them reviewed and landed without waiting for everything to be green.

I'm going to post the fixes I have so far in this bug.

In bug 1521644, we changed the clip rect computation so we don't divide by
the resolution when using containerless scrolling. The rationale for that
change was that this clip becomes a clip rect on the async zoom container
layer, which is applied after the resolution in the compositor.

However, when building a display list for event delivery, this rationale
does not apply as there are no layers and no compositor involved.

Depends on D19219

Blocks: 1527516
Pushed by bballo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fe6aac2cec49
Hook up reftest-async-{scroll,zoom} for containerless scrolling. r=kats
https://hg.mozilla.org/integration/autoland/rev/4009b930fbf1
Build an async zoom container for event delivery as well. r=mstange
https://hg.mozilla.org/integration/autoland/rev/11133b5a9855
Don't muck with the RCD-RSF composition bounds clip rect during event delivery. r=mstange
https://hg.mozilla.org/integration/autoland/rev/5bbfd524941c
Disable container scrolling in APZ zoom mochitests. r=kats
You need to log in before you can comment on or make changes to this bug.