From 163943418863cd52a6fb39d428086ea5a672e6a1 Mon Sep 17 00:00:00 2001 From: Jamie Nicol Date: Fri, 7 Feb 2020 20:45:39 +0000 Subject: [PATCH] Bug 1613144 - Add reftest. r=botond Add a reftest that would fail in webrender before the corresponding fix landed. Ensures that there is an async zoom and that both the layout and visual viewports have async scroll offsets. To pass, we must apply each of the layout and visual offsets in their correct coordinate spaces. Differential Revision: https://phabricator.services.mozilla.com/D61788 --HG-- extra : moz-landing-system : lando --- .../async-scroll-and-zoom-ref.html | 24 ++++++++++++ .../async-scroll-and-zoom.html | 39 +++++++++++++++++++ layout/reftests/async-scrolling/reftest.list | 2 + 3 files changed, 65 insertions(+) create mode 100644 layout/reftests/async-scrolling/async-scroll-and-zoom-ref.html create mode 100644 layout/reftests/async-scrolling/async-scroll-and-zoom.html diff --git a/layout/reftests/async-scrolling/async-scroll-and-zoom-ref.html b/layout/reftests/async-scrolling/async-scroll-and-zoom-ref.html new file mode 100644 index 000000000000..ecc40d509065 --- /dev/null +++ b/layout/reftests/async-scrolling/async-scroll-and-zoom-ref.html @@ -0,0 +1,24 @@ + + + + + + +
+ + diff --git a/layout/reftests/async-scrolling/async-scroll-and-zoom.html b/layout/reftests/async-scrolling/async-scroll-and-zoom.html new file mode 100644 index 000000000000..a7795a54d67c --- /dev/null +++ b/layout/reftests/async-scrolling/async-scroll-and-zoom.html @@ -0,0 +1,39 @@ + + + + + + + +
+ + diff --git a/layout/reftests/async-scrolling/reftest.list b/layout/reftests/async-scrolling/reftest.list index 5497e3ba81c6..073490eb2495 100644 --- a/layout/reftests/async-scrolling/reftest.list +++ b/layout/reftests/async-scrolling/reftest.list @@ -94,6 +94,8 @@ skip-if(!Android) pref(apz.allow_zooming,true) == position-fixed-async-zoom-4.ht skip-if(!Android) pref(apz.allow_zooming,true) == position-sticky-async-zoom-1.html position-sticky-async-zoom-1-ref.html skip-if(!Android) pref(apz.allow_zooming,true) == position-sticky-async-zoom-2.html position-sticky-async-zoom-2-ref.html +fuzzy-if(!webrender,111-112,600-600) pref(apz.allow_zooming,true) == async-scroll-and-zoom.html async-scroll-and-zoom-ref.html + # for this test, apz.allow_zooming is needed to ensure we take the containerless scrolling codepath that creates # an async zoom container (since we are testing a regression in that codepath) skip-if(!Android) pref(apz.allow_zooming,true) test-pref(apz.fixed-margin-override.enabled,true) test-pref(apz.fixed-margin-override.bottom,50) fails-if(geckoview&&webrender) == dynamic-toolbar-fixed-bottom-1.html dynamic-toolbar-fixed-bottom-1-ref.html