Bug 1452820 - Use correct window property for testing end of scroll range. r=rhunt
authorBotond Ballo <botond@mozilla.com>
Fri, 05 Oct 2018 23:58:04 +0000
changeset 439884 75b958f179a8ba69ef1a3bc909ef741d0e8ec146
parent 439883 ca3b9c38e8e5af2a405353c2eb41f354f5e5f5ba
child 439885 237c55ddb11256f42d744146f8b7f3dd11b7491b
push id70499
push userbballo@mozilla.com
push dateSat, 06 Oct 2018 00:17:22 +0000
treeherderautoland@75b958f179a8 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersrhunt
bugs1452820
milestone64.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1452820 - Use correct window property for testing end of scroll range. r=rhunt Differential Revision: https://phabricator.services.mozilla.com/D7950
gfx/layers/apz/test/mochitest/helper_key_scroll.html
--- a/gfx/layers/apz/test/mochitest/helper_key_scroll.html
+++ b/gfx/layers/apz/test/mochitest/helper_key_scroll.html
@@ -31,17 +31,17 @@ https://bugzilla.mozilla.org/show_bug.cg
       // Send a key to initiate a page scroll to take us to the bottom of the
       // page. This scroll is done synchronously because APZ doesn't have
       // current focus state at page load.
       window.addEventListener("scroll", waitForScrollBottom);
       window.synthesizeKey("KEY_End");
     };
 
     function waitForScrollBottom() {
-      if (window.scrollY < window.scrollYMax) {
+      if (window.scrollY < window.scrollMaxY) {
         return;
       }
       SimpleTest.info("Reached final scroll position of sync KEY_End scroll");
       window.removeEventListener("scroll", waitForScrollBottom);
 
       // Spin the refresh driver a few times, so that the AsyncScroll instance
       // that was running the main-thread scroll animation finishes up and
       // triggers any repaints that it needs to.