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

Unified Diff: third_party/WebKit/LayoutTests/fast/events/touch/gesture/touch-gesture-scroll-input-field.html

Issue 1320543006: Remove touch scroll chaining from main thread scrolling. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@gclient
Patch Set: Fix silly mistake in mac expectations. Created 5 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
Index: third_party/WebKit/LayoutTests/fast/events/touch/gesture/touch-gesture-scroll-input-field.html
diff --git a/third_party/WebKit/LayoutTests/fast/events/touch/gesture/touch-gesture-scroll-input-field.html b/third_party/WebKit/LayoutTests/fast/events/touch/gesture/touch-gesture-scroll-input-field.html
index fb816b0bbe3e54a2314bf15ee9425bec273cf461..9a18295d35718c280cb0e6e4b922065e5bd14c73 100644
--- a/third_party/WebKit/LayoutTests/fast/events/touch/gesture/touch-gesture-scroll-input-field.html
+++ b/third_party/WebKit/LayoutTests/fast/events/touch/gesture/touch-gesture-scroll-input-field.html
@@ -54,11 +54,11 @@
shouldBe('container.scrollLeft', '0');
eventSender.gestureScrollBegin(gestureX, gestureY);
- // Prevent scroll to propagate by passing true for third parameter
- eventSender.gestureScrollUpdate(-10, 0, true);
- eventSender.gestureScrollUpdate(-10, 0, true);
- eventSender.gestureScrollUpdate(-10, 0, true);
- eventSender.gestureScrollUpdate(-10, 0, true);
+
+ eventSender.gestureScrollUpdate(-10, 0);
+ eventSender.gestureScrollUpdate(-10, 0);
+ eventSender.gestureScrollUpdate(-10, 0);
+ eventSender.gestureScrollUpdate(-10, 0);
eventSender.gestureScrollEnd(0, 0);
debug("Flinging input text should scroll text by the specified amount");
@@ -68,10 +68,10 @@
resetScroll();
eventSender.gestureScrollBegin(gestureX, gestureY);
- eventSender.gestureScrollUpdate(-fullyScrolled, 0, true);
- eventSender.gestureScrollUpdate(-100, 0, true);
- eventSender.gestureScrollUpdate(-100, 0, true);
- eventSender.gestureScrollUpdate(-300, 0, true);
+ eventSender.gestureScrollUpdate(-fullyScrolled, 0);
+ eventSender.gestureScrollUpdate(-100, 0);
+ eventSender.gestureScrollUpdate(-100, 0);
+ eventSender.gestureScrollUpdate(-300, 0);
eventSender.gestureScrollEnd(0, 0);
debug("Flinging input text past the scrollable width shouldn't scroll containing div");
@@ -80,8 +80,8 @@
shouldBe('container.scrollLeft', '0');
eventSender.gestureScrollBegin(gestureX, gestureY);
- eventSender.gestureScrollUpdate(-30, 0, true);
- eventSender.gestureScrollUpdate(-30, 0, true);
+ eventSender.gestureScrollUpdate(-30, 0);
+ eventSender.gestureScrollUpdate(-30, 0);
eventSender.gestureScrollEnd(0, 0);
debug("Flinging fully scrolled input text should fling containing div");
@@ -113,9 +113,9 @@
eventSender.gestureScrollUpdate(-50, 0);
eventSender.gestureScrollEnd(0, 0);
- debug("Gesture scrolling input text past scroll width should scroll container div");
+ debug("Gesture scrolling input text past scroll width shouldn't scroll container div");
shouldBe('box.scrollLeft', 'fullyScrolled');
- shouldBe('container.scrollLeft', '50');
+ shouldBe('container.scrollLeft', '0');
}
function testVerticalScroll()

Powered by Google App Engine
This is Rietveld 408576698