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 495603 75b958f179a8ba69ef1a3bc909ef741d0e8ec146
parent 495602 ca3b9c38e8e5af2a405353c2eb41f354f5e5f5ba
child 495604 237c55ddb11256f42d744146f8b7f3dd11b7491b
push id9984
push userffxbld-merge
push dateMon, 15 Oct 2018 21:07:35 +0000
treeherdermozilla-beta@183d27ea8570 [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.