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

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: Created 5 years, 11 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 0a20ef231c5d5a32e95adcb13a10f11f955cb73b..a82b344f000f9c825e6a2e4508b7c38050ca2860 100644
--- a/LayoutTests/fast/forms/resources/picker-common.js
+++ b/LayoutTests/fast/forms/resources/picker-common.js
@@ -20,7 +20,7 @@ function sendKey(input, keyName, ctrlKey, altKey) {
}
function openPicker(input, callback) {
- window.moveTo();
+ window.moveTo(window.screenX, window.screenY);
Jens Widell 2015/01/29 13:53:44 This one is funny. This call is required to make t
input.offsetTop; // Force to lay out
if (input.type === "color") {
input.focus();
@@ -34,7 +34,7 @@ function openPicker(input, 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.
- window.moveTo();
+ window.moveTo(window.screenX, window.screenY);
callback();
}).bind(this, callback);
popupWindow.addEventListener("didOpenPicker", popupOpenCallbackWrapper, false);
« no previous file with comments | « LayoutTests/fast/dom/Window/window-resize-and-move-arguments-expected.txt ('k') | Source/core/frame/DOMWindow.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698