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 416656 76855675e353231dd10ec0a1f8ca7c1211b281f1
parent 416655 3ab2fbb175bd7fa720d4e990dbaaa0a5fe0c32a9
child 416657 e2eae98b708c4d349a7853e84f3f41729178a38d
push id102824
push userneil@mozilla.com
push dateWed, 02 May 2018 14:35:34 +0000
treeherdermozilla-inbound@76855675e353 [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);