Bug 1457763, regression, autoscroll popup should be opened with openPopupAtScreen, r=paolo
authorNeil Deakin <neil@mozilla.com>
Wed, 02 May 2018 10:32:54 -0400
changeset 472775 76855675e353231dd10ec0a1f8ca7c1211b281f1
parent 472774 3ab2fbb175bd7fa720d4e990dbaaa0a5fe0c32a9
child 472776 e2eae98b708c4d349a7853e84f3f41729178a38d
push id1728
push userjlund@mozilla.com
push dateMon, 18 Jun 2018 21:12:27 +0000
treeherdermozilla-release@c296fde26f5f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerspaolo
bugs1457763
milestone61.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 1457763, regression, autoscroll popup should be opened with openPopupAtScreen, r=paolo
toolkit/content/widgets/browser.xml
--- a/toolkit/content/widgets/browser.xml
+++ b/toolkit/content/widgets/browser.xml
@@ -1243,17 +1243,17 @@
             // autoscrolling on. So instead we do the same math as Gecko.
             const scaleFactor = 60 / Math.round(60 / screen.defaultCSSScaleFactor);
             let minX = left.value / scaleFactor + 0.5 * POPUP_SIZE;
             let maxX = (left.value + width.value) / scaleFactor - 0.5 * POPUP_SIZE;
             let minY = top.value / scaleFactor + 0.5 * POPUP_SIZE;
             let maxY = (top.value + height.value) / scaleFactor - 0.5 * POPUP_SIZE;
             let popupX = Math.max(minX, Math.min(maxX, screenX));
             let popupY = Math.max(minY, Math.min(maxY, screenY));
-            this._autoScrollPopup.openPopup(null, "after_start", popupX, popupY);
+            this._autoScrollPopup.openPopupAtScreen(popupX, popupY);
             this._ignoreMouseEvents = true;
             this._scrolling = true;
             this._startX = screenX;
             this._startY = screenY;
 
             window.addEventListener("mousemove", this, true);
             window.addEventListener("mousedown", this, true);
             window.addEventListener("mouseup", this, true);