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

Unified Diff: third_party/WebKit/LayoutTests/fast/events/wheel/wheelevent-ctrl.html

Issue 2943183002: Revert of Re-target wheel events only when a new scroll sequence has started. (Closed)
Patch Set: Created 3 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: third_party/WebKit/LayoutTests/fast/events/wheel/wheelevent-ctrl.html
diff --git a/third_party/WebKit/LayoutTests/fast/events/wheel/wheelevent-ctrl.html b/third_party/WebKit/LayoutTests/fast/events/wheel/wheelevent-ctrl.html
index 581c776bca03e0821cd58b872fb657e5083de2fd..94f1428ac42c8d695a2c2d3d648c17c52334c029 100644
--- a/third_party/WebKit/LayoutTests/fast/events/wheel/wheelevent-ctrl.html
+++ b/third_party/WebKit/LayoutTests/fast/events/wheel/wheelevent-ctrl.html
@@ -23,7 +23,7 @@
debug('With ctrl modifier set and canScroll set to be false');
wheelEventCount = 0;
eventSender.mouseMoveTo(testDiv.offsetLeft + 5, testDiv.offsetTop + 5);
- eventSender.mouseScrollBy(0, scrollAmount, false, true, "ctrlKey", false , "phaseBegan");
+ eventSender.mouseScrollBy(0, scrollAmount, false, true, "ctrlKey", false);
shouldBe("wheelEventCount", "1");
shouldEvaluateTo("deltaY", expectedDeltaY);
shouldBeTrue("ctrlKey");
@@ -32,7 +32,7 @@
debug('Without ctrl and canScroll set to be default true');
wheelEventCount = 0;
eventSender.mouseMoveTo(testDiv.offsetLeft + 5, testDiv.offsetTop + 5);
- eventSender.mouseScrollBy(0, scrollAmount, false, true, 0, true, "phaseChanged");
+ eventSender.mouseScrollBy(0, scrollAmount, false, true);
shouldBe("wheelEventCount", "1");
shouldEvaluateTo("deltaY", expectedDeltaY);
shouldBeFalse("ctrlKey");
@@ -45,7 +45,7 @@
debug('With ctrl modifier set and canScroll set to be false');
wheelEventCount = 0;
eventSender.mouseMoveTo(testDiv.offsetLeft + 5, testDiv.offsetTop + 5);
- eventSender.mouseScrollBy(0, scrollAmount, false, true, "ctrlKey", false, "phaseChanged");
+ eventSender.mouseScrollBy(0, scrollAmount, false, true, "ctrlKey", false);
shouldBe("wheelEventCount", "1");
shouldEvaluateTo("deltaY", expectedDeltaY);
shouldBeTrue("ctrlKey");
@@ -54,7 +54,7 @@
debug('With ctrl modifier set and canScroll set to be true');
wheelEventCount = 0;
eventSender.mouseMoveTo(testDiv.offsetLeft + 5, testDiv.offsetTop + 5);
- eventSender.mouseScrollBy(0, scrollAmount, false, true, "ctrlKey", true, "phaseChanged");
+ eventSender.mouseScrollBy(0, scrollAmount, false, true, "ctrlKey", true);
shouldBe("wheelEventCount", "1");
shouldEvaluateTo("deltaY", expectedDeltaY);
shouldBeTrue("ctrlKey");
@@ -63,7 +63,7 @@
debug('Now without ctrl and canScroll set to be default true');
wheelEventCount = 0;
eventSender.mouseMoveTo(testDiv.offsetLeft + 5, testDiv.offsetTop + 5);
- eventSender.mouseScrollBy(0, scrollAmount, false, true, 0, true, "phaseChanged");
+ eventSender.mouseScrollBy(0, scrollAmount, false, true);
shouldBe("wheelEventCount", "1");
shouldEvaluateTo("deltaY", expectedDeltaY);
shouldBeFalse("ctrlKey");

Powered by Google App Engine
This is Rietveld 408576698