Support GestureBegin and GestureEnd in ui::GestureProvider

content/browser/android/content_view_core_impl.cc

Index: content/browser/android/content_view_core_impl.cc
diff --git a/content/browser/android/content_view_core_impl.cc b/content/browser/android/content_view_core_impl.cc
index 8606d5cd47712beb0675f21298f964e653eb0415..81928131b6f8105ac8f071ca4c50258cda6d76c9 100644
--- a/content/browser/android/content_view_core_impl.cc
+++ b/content/browser/android/content_view_core_impl.cc
@@ -369,8 +369,10 @@ void ContentViewCoreImpl::RenderViewReady() {
 }
 
 void ContentViewCoreImpl::OnGestureEvent(const ui::GestureEventData& gesture) {
-  SendGestureEvent(
-      CreateWebGestureEventFromGestureEventData(gesture, 1.f / dpi_scale()));
+  WebGestureEvent event =
+      CreateWebGestureEventFromGestureEventData(gesture, 1.f / dpi_scale());
+  if (event.type != WebInputEvent::Undefined)

[jdduke (slow), 2014/04/03 20:17:16]

Shouldn't this be a DCHECK? We won't use the Gesture{Begin,End} on Android?

[tdresser, 2014/04/03 21:14:41]

Done.

+    SendGestureEvent(event);
 }
 
 RenderWidgetHostViewAndroid*