Getting RenderWidgetHostViewWin ready for a GestureRecognizer.

content/browser/renderer_host/render_widget_host_view_win.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_WIN_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_WIN_H_
 #pragma once
 
 #include <atlbase.h>
 #include <atlapp.h>
 #include <atlcrack.h>
 #include <atlmisc.h>
 #include <peninputpanel.h>
 #include <vector>
 
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/scoped_vector.h"
 #include "base/memory/weak_ptr.h"
 #include "base/time.h"
 #include "base/win/scoped_comptr.h"
 #include "content/browser/accessibility/browser_accessibility_manager.h"
 #include "content/browser/renderer_host/render_widget_host_view_base.h"
 #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"
 #include "ui/gfx/surface/accelerated_surface_win.h"
 #include "ui/gfx/sys_color_change_listener.h"
 #include "webkit/glue/webcursor.h"
 
 class BackingStore;
 class SkRegion;
 
@@ skipping 50 matching lines @@
 //     references to it must become NULL."
 //
 // RenderWidgetHostView class hierarchy described in render_widget_host_view.h.
 class RenderWidgetHostViewWin
     : public CWindowImpl<RenderWidgetHostViewWin,
                          CWindow,
                          RenderWidgetHostHWNDTraits>,
       public content::RenderWidgetHostViewBase,
       public content::NotificationObserver,
       public BrowserAccessibilityDelegate,
-      public gfx::SysColorChangeListener {
+      public gfx::SysColorChangeListener,
+      public ui::GestureEventHelper {
  public:
   virtual ~RenderWidgetHostViewWin();
 
   CONTENT_EXPORT void CreateWnd(HWND parent);
 
   void ScheduleComposite();
 
   DECLARE_WND_CLASS_EX(kRenderWidgetHostHWNDClass, CS_DBLCLKS, 0);
 
   BEGIN_MSG_MAP(RenderWidgetHostHWND)
@@ skipping 133 matching lines @@
   virtual void AccessibilityScrollToMakeVisible(
       int acc_obj_id, gfx::Rect subfocus) OVERRIDE;
   virtual void AccessibilityScrollToPoint(
       int acc_obj_id, gfx::Point point) OVERRIDE;
   virtual void AccessibilitySetTextSelection(
       int acc_obj_id, int start_offset, int end_offset) OVERRIDE;
 
   // 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;
 
   // Should construct only via RenderWidgetHostView::CreateViewForWidget.
   //
   // The view will associate itself with the given widget.
   explicit RenderWidgetHostViewWin(content::RenderWidgetHost* widget);
 
   // Windows Message Handlers
   LRESULT OnCreate(CREATESTRUCT* create_struct);
@@ skipping 202 matching lines @@
 
     WebKit::WebTouchEvent touch_event_;
     const CWindowImpl* const window_;
   };
 
   // The touch-state. Its touch-points are updated as necessary. A new
   // touch-point is added from an TOUCHEVENTF_DOWN message, and a touch-point
   // 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().
   bool about_to_validate_and_paint_;
 
   // true if the View should be closed when its HWND is deactivated (used to
   // support SELECT popups which are closed when they are deactivated).
   bool close_on_deactivate_;
 
   // Whether Destroy() has been called.  Used to detect a crasher
@@ skipping 91 matching lines @@
 
   // Are touch events currently enabled?
   bool touch_events_enabled_;
 
   gfx::ScopedSysColorChangeListener sys_color_change_listener_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostViewWin);
 };
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_WIN_H_