Green up Android tests with containerless scrolling enabled (part 1)
Categories
(Core :: Panning and Zooming, enhancement, P3)
Tracking
()
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.
Assignee | ||
Comment 1•5 years ago
|
||
Assignee | ||
Comment 2•5 years ago
|
||
Depends on D19218
Assignee | ||
Comment 3•5 years ago
|
||
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
Assignee | ||
Comment 4•5 years ago
|
||
Depends on D19221
Updated•5 years ago
|
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
Comment 6•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/fe6aac2cec49
https://hg.mozilla.org/mozilla-central/rev/4009b930fbf1
https://hg.mozilla.org/mozilla-central/rev/11133b5a9855
https://hg.mozilla.org/mozilla-central/rev/5bbfd524941c
Description
•