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

Unified Diff: third_party/WebKit/LayoutTests/imported/wpt_automation/pointerevents/pointerevent_capture_suppressing_mouse-manual-automation.js

Issue 2380943003: Update automation scripts for wpt tests (Closed)
Patch Set: Created 4 years, 3 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/imported/wpt_automation/pointerevents/pointerevent_common_input.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/imported/wpt_automation/pointerevents/pointerevent_capture_suppressing_mouse-manual-automation.js
diff --git a/third_party/WebKit/LayoutTests/imported/wpt_automation/pointerevents/pointerevent_capture_suppressing_mouse-manual-automation.js b/third_party/WebKit/LayoutTests/imported/wpt_automation/pointerevents/pointerevent_capture_suppressing_mouse-manual-automation.js
index 2de2feb4dd8d2e0f92961c3aa8699038ef458fa5..ad1a2e9621216282bb8586db22d376dcd0eb52a8 100644
--- a/third_party/WebKit/LayoutTests/imported/wpt_automation/pointerevents/pointerevent_capture_suppressing_mouse-manual-automation.js
+++ b/third_party/WebKit/LayoutTests/imported/wpt_automation/pointerevents/pointerevent_capture_suppressing_mouse-manual-automation.js
@@ -4,11 +4,9 @@ function inject_input() {
return mouseMoveIntoTarget('#target0').then(function() {
return mouseMoveIntoTarget('#target1');
}).then(function() {
- return mouseDragInTargets(['#btnCapture', '#target1', '#target0', '#target1']);
+ return mouseDragInTargets(['#btnCapture', '#btnCapture', '#target1', '#target0', '#target1']);
mustaq 2016/09/29 17:47:18 Why two #btnCapture now?
Navid Zolghadr 2016/09/29 17:48:59 Because I needed an extra move after clicking down
mustaq 2016/09/29 17:56:43 Makes sense with your github PR, thanks. https://g
}).then(function() {
- // To handle delayed capturing.
- mouseMoveIntoTarget('#target0');
+ return mouseMoveIntoTarget('#target1');
});
-
}
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/imported/wpt_automation/pointerevents/pointerevent_common_input.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698