Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(86)

Unified Diff: LayoutTests/fast/forms/resources/picker-common.js

Issue 886463010: Make arguments to window.{move,resize}{To,By} non-optional (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: rebased Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: LayoutTests/fast/forms/resources/picker-common.js
diff --git a/LayoutTests/fast/forms/resources/picker-common.js b/LayoutTests/fast/forms/resources/picker-common.js
index f69889604146faf42588d269ed93266eefcf5763..e0ebc67910593157f503cfc77cb0181e7ac60fed 100644
--- a/LayoutTests/fast/forms/resources/picker-common.js
+++ b/LayoutTests/fast/forms/resources/picker-common.js
@@ -28,7 +28,7 @@ function rootWindow() {
}
function openPicker(element, callback, errorCallback) {
- rootWindow().moveTo();
+ rootWindow().moveTo(window.screenX, window.screenY);
element.offsetTop; // Force to lay out
if (element.tagName === "SELECT") {
sendKey(element, "Down", false, true);
@@ -48,7 +48,7 @@ function openPicker(element, callback, errorCallback) {
}
function clickToOpenPicker(x, y, callback, errorCallback) {
- rootWindow().moveTo();
+ rootWindow().moveTo(window.screenX, window.screenY);
eventSender.mouseMoveTo(x, y);
eventSender.mouseDown();
eventSender.mouseUp();
@@ -65,7 +65,7 @@ function setPopupOpenCallback(callback) {
// We need to move the window to the top left of available space
// because the window will move back to (0, 0) when the
// ShellViewMsg_SetTestConfiguration IPC arrives.
- rootWindow().moveTo();
+ rootWindow().moveTo(window.screenX, window.screenY);
callback();
}).bind(this, callback);
popupWindow.addEventListener("didOpenPicker", popupOpenCallbackWrapper, false);

Powered by Google App Engine
This is Rietveld 408576698