Android: zooming behaves strangely on about:support
Categories
(Core :: Graphics: WebRender, enhancement, P3)
Tracking
()
People
(Reporter: kats, Assigned: kats)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
about:support loads in the UI process I think. But I assumed zoomable content is always in the content process. https://searchfox.org/mozilla-central/rev/2edebf41a2b2e624859bf15ed5a98bdd002f73b4/gfx/layers/wr/WebRenderCommandBuilder.cpp#1437 This results in some strange behaviour when zooming/scrolling on about:support, and probably on other about: pages.
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Comment 1•6 years ago
|
||
The way we control APZ zooming in WebRender is by inserting an animation
property placeholder on a WR stacking context, and then having APZ
update the animation transform value with the proper matrix at composite
time.
Previously, the stacking context being used was the rootmost
stacking context in the content process. However this doesn't work for
zoomable content in the UI process (e.g. about:support), and after
recent changes, also wraps display items that should not be affected by
zoom (e.g. scrollbars or the background color item).
This patch moves the animation property placeholder so that it
corresponds to the newly added nsDisplayAsyncZoom display item, which
corrects both of the above problems and is conceptually in line with the
desired behaviour.
Pushed by kgupta@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/fa641ad0f4b8 Use nsDisplayAsyncZoom items to insert zooming animations in WR. r=jrmuizel,botond
Comment 3•6 years ago
|
||
bugherder |
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Description
•