[Android] Perform eager gesture recognition on MotionEvents

content/browser/android/content_view_core_impl.h

Index: content/browser/android/content_view_core_impl.h
diff --git a/content/browser/android/content_view_core_impl.h b/content/browser/android/content_view_core_impl.h
index d6b7a1a5d2f13c3cfec0ba2b3d083fe0363a5809..5fb71dade206785bfe7bb1428e9bf4c529b7dc9f 100644
--- a/content/browser/android/content_view_core_impl.h
+++ b/content/browser/android/content_view_core_impl.h
@@ -14,6 +14,7 @@
 #include "base/i18n/rtl.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/process/process.h"
+#include "content/browser/renderer_host/input/gesture_event_queue.h"
 #include "content/browser/renderer_host/render_widget_host_view_android.h"
 #include "content/browser/web_contents/web_contents_impl.h"
 #include "content/public/browser/android/content_view_core.h"
@@ -37,7 +38,8 @@ struct MenuItem;
 // TODO(jrg): this is a shell.  Upstream the rest.
 class ContentViewCoreImpl : public ContentViewCore,
                             public NotificationObserver,
-                            public WebContentsObserver {
+                            public WebContentsObserver,
+                            public GestureEventQueueClient {
  public:
   static ContentViewCoreImpl* FromWebContents(WebContents* web_contents);
   ContentViewCoreImpl(JNIEnv* env,
@@ -95,11 +97,12 @@ class ContentViewCoreImpl : public ContentViewCore,
       JNIEnv* env, jobject obj) const;
   jboolean IsIncognito(JNIEnv* env, jobject obj);
   void SendOrientationChangeEvent(JNIEnv* env, jobject obj, jint orientation);
-  jboolean SendTouchEvent(JNIEnv* env,
-                          jobject obj,
-                          jlong time_ms,
-                          jint type,
-                          jobjectArray pts);
+  void OnTouchEventHandlingBegin(JNIEnv* env,
+                                 jobject obj,
+                                 jlong time_ms,
+                                 jint type,
+                                 jobjectArray pts);
+  void OnTouchEventHandlingEnd(JNIEnv* env, jobject obj);
   jboolean SendMouseMoveEvent(JNIEnv* env,
                               jobject obj,
                               jlong time_ms,
@@ -270,9 +273,8 @@ class ContentViewCoreImpl : public ContentViewCore,
 
   bool HasFocus();
   void ConfirmTouchEvent(InputEventAckState ack_result);
-  void UnhandledFlingStartEvent();
+  void UnhandledFlingStartEvent(float vx, float vy);
   void OnScrollUpdateGestureConsumed();
-  void HasTouchEventHandlers(bool need_touch_events);
   void OnSelectionChanged(const std::string& text);
   void OnSelectionBoundsChanged(
       const ViewHostMsg_SelectionBounds_Params& params);
@@ -332,6 +334,10 @@ class ContentViewCoreImpl : public ContentViewCore,
   // WebContentsObserver implementation.
   virtual void RenderViewReady() OVERRIDE;
 
+  // GestureEventQueueClient implementation.
+  virtual void ForwardGestureEvent(
+      const blink::WebGestureEvent& event) OVERRIDE;
+
   // --------------------------------------------------------------------------
   // Other private methods and data
   // --------------------------------------------------------------------------
@@ -392,6 +398,11 @@ class ContentViewCoreImpl : public ContentViewCore,
 
   bool geolocation_needs_pause_;
 
+  scoped_ptr<GestureEventQueue> gesture_event_queue_;

[tdresser, 2014/01/16 14:31:45]

Why not just allocate this on the stack?

[jdduke (slow), 2014/01/16 17:17:53]

I had this behind a flag before, so there's no longer any need... though it will
never be on the stack proper, as CVCI is always heap allocated.  I'll change.

+  bool handling_touch_event_;
+  blink::WebTouchEvent pending_touch_event_;
+  GestureEventPacket pending_gesture_packet_;
+
   DISALLOW_COPY_AND_ASSIGN(ContentViewCoreImpl);
 };