Getting RenderWidgetHostViewWin ready for a GestureRecognizer.

content/browser/renderer_host/render_widget_host_view_win.h

Index: content/browser/renderer_host/render_widget_host_view_win.h
diff --git a/content/browser/renderer_host/render_widget_host_view_win.h b/content/browser/renderer_host/render_widget_host_view_win.h
index e1178947cbc53d71ca5eca1e5e9484fc07b00ea2..0c6963a08ae35bb9e2762e1eeb2771081724d4fa 100644
--- a/content/browser/renderer_host/render_widget_host_view_win.h
+++ b/content/browser/renderer_host/render_widget_host_view_win.h
@@ -24,6 +24,8 @@
 #include "content/common/content_export.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
+#include "ui/base/gestures/gesture_recognizer.h"
+#include "ui/base/gestures/gesture_types.h"
 #include "ui/base/win/ime_input.h"
 #include "ui/gfx/native_widget_types.h"
 #include "ui/gfx/point.h"
@@ -94,7 +96,8 @@ class RenderWidgetHostViewWin
       public content::RenderWidgetHostViewBase,
       public content::NotificationObserver,
       public BrowserAccessibilityDelegate,
-      public gfx::SysColorChangeListener {
+      public gfx::SysColorChangeListener,
+      public ui::GestureEventHelper {
  public:
   virtual ~RenderWidgetHostViewWin();
 
@@ -248,6 +251,21 @@ class RenderWidgetHostViewWin
   // Implementation of SysColorChangeListener:
   virtual void OnSysColorChange() OVERRIDE;
 
+  // Overridden from ui::GestureEventHelper.
+  virtual ui::GestureEvent* CreateGestureEvent(ui::EventType type,
+      const gfx::Point& location,
+      int flags,
+      const base::Time time,
+      float param_first,
+      float param_second,
+      unsigned int touch_id_bitfield) OVERRIDE;
+  virtual ui::TouchEvent* CreateTouchEvent(ui::EventType type,
+      const gfx::Point& location,
+      int touch_id,
+      base::TimeDelta time_stamp) OVERRIDE;
+  virtual bool DispatchLongPressGestureEvent(ui::GestureEvent* event) OVERRIDE;
+  virtual bool DispatchCancelTouchEvent(ui::TouchEvent* event) OVERRIDE;
+
  protected:
   friend class content::RenderWidgetHostView;
 
@@ -470,6 +488,8 @@ class RenderWidgetHostViewWin
   // is removed from the list on an TOUCHEVENTF_UP message.
   WebTouchState touch_state_;
 
+  ui::GestureRecognizer * gesture_recognizer_;

[sky, 2012/05/03 23:22:39]

This leaks now. I'm saying declare it by value:
ui::GestureRecognizer gesture_recognizer_;

That said, you're not even using gesture_recognizer_ in this patch. Hold off on
adding it until you actually use it.

[tfarina, 2012/05/03 23:25:26]

If you use scoped_ptr here, you can forward declare GestireRecognizer. You don't
need to include gesture_recognizer.h. And gesture_types.h is only needed to
inherit from ui::GestureEventHelper.

+
   // True if we're in the midst of a paint operation and should respond to
   // DidPaintRect() notifications by merely invalidating.  See comments on
   // render_widget_host_view.h:DidPaintRect().