Touchmoves should not be suppressed if the touchstart is consumed

third_party/WebKit/Source/core/input/TouchEventManager.cpp

Index: third_party/WebKit/Source/core/input/TouchEventManager.cpp
diff --git a/third_party/WebKit/Source/core/input/TouchEventManager.cpp b/third_party/WebKit/Source/core/input/TouchEventManager.cpp
index 149cff36bc908842d65b6dc63fe73d9b538f3a66..1e77f99e80a2656764b7b5086c1cabd4352d89de 100644
--- a/third_party/WebKit/Source/core/input/TouchEventManager.cpp
+++ b/third_party/WebKit/Source/core/input/TouchEventManager.cpp
@@ -292,6 +292,11 @@ WebInputEventResult TouchEventManager::dispatchTouchEvents(
     }
   }
 
+  if (m_suppressingTouchmovesWithinSlop &&
+      eventResult != WebInputEventResult::NotHandled) {

[mustaq, 2017/02/16 17:06:26]

This "!=NotHandled " applies only to touchstarts because in suppressed-state,
touchmoves can't be prevent-defaulted. Please add a comment here that this
applies only to touchstarts.

Also update the CL description accordingly.

[lanwei, 2017/02/16 19:48:59]

Done.

[mustaq, 2017/02/16 21:09:53]

Let me clarify a bit: this condition is met only for touchstarts, so we don't
need a new condition. Let's just add an ASSERT(event.type() ==
WebInputEvent::TouchStart) w/o any comments.

+    m_suppressingTouchmovesWithinSlop = false;
+  }
+
   return eventResult;
 }