Support GestureBegin and GestureEnd in ui::GestureProvider

ui/events/gesture_detection/gesture_provider.h

 // Copyright 2014 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 UI_EVENTS_GESTURE_DETECTION_GESTURE_PROVIDER_H_
 #define UI_EVENTS_GESTURE_DETECTION_GESTURE_PROVIDER_H_
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
 #include "ui/events/gesture_detection/gesture_detection_export.h"
@@ skipping 15 matching lines @@
 // event dispatch.
 class GESTURE_DETECTION_EXPORT GestureProvider {
  public:
   struct GESTURE_DETECTION_EXPORT Config {
     Config();
     ~Config();
     GestureDetector::Config gesture_detector_config;
     ScaleGestureDetector::Config scale_gesture_detector_config;
     SnapScrollController::Config snap_scroll_controller_config;
     bool disable_click_delay;
+    bool gesture_begin_end_types_enabled;

[jdduke (slow), 2014/04/03 22:03:28]

Maybe a short comment explaining what this does?  I should have added one for
|disable_click_delay| as well.

[tdresser, 2014/04/04 16:39:31]

Done. Please double check, particularly the comment I added for
disable_click_delay.

   };
 
   GestureProvider(const Config& config, GestureProviderClient* client);
   ~GestureProvider();
 
   // Handle the incoming MotionEvent, returning false if the event could not
   // be handled.
   bool OnTouchEvent(const MotionEvent& event);
 
   // Resets all gesture detectors; called on DidStartLoading().
   void ResetGestureDetectors();
 
   // Update whether multi-touch gestures are supported.
   void SetMultiTouchSupportEnabled(bool enabled);
 
   // Update whether double-tap gestures are supported. This allows
   // double-tap gesture suppression independent of whether or not the page's
   // viewport and scale would normally prevent double-tap.
   // Note: This should not be called while a double-tap gesture is in progress.
   void SetDoubleTapSupportForPlatformEnabled(bool enabled);
 
   // Update whether double-tap gesture detection should be suppressed due to
   // the viewport or scale of the current page. Suppressing double-tap gesture
   // detection allows for rapid and responsive single-tap gestures.
   void SetDoubleTapSupportForPageEnabled(bool enabled);
 
+  // Update whether GestureBegin and GestureEnd events are supported.
+  void SetBeginEndTypesEnabled(bool enabled);
+
   // Whether a scroll gesture is in-progress.
   bool IsScrollInProgress() const;
 
   // Whether a pinch gesture is in-progress (i.e. a pinch update has been
   // forwarded and detection is still active).
   bool IsPinchInProgress() const;
 
   // Whether a double tap-gesture is in-progress.
   bool IsDoubleTapInProgress() const;
 
@@ skipping 14 matching lines @@
   void Fling(base::TimeTicks time,
              float x,
              float y,
              float velocity_x,
              float velocity_y);
   void Send(const GestureEventData& gesture);
   void SendTapCancelIfNecessary(const MotionEvent& event);
   bool SendLongTapIfNecessary(const MotionEvent& event);
   void EndTouchScrollIfNecessary(base::TimeTicks time,
                                  bool send_scroll_end_event);
+  void OnTouchEventHandlingBegin(const MotionEvent& event);
+  void OnTouchEventHandlingEnd(const MotionEvent& event,
+                               bool handled,
+                               bool was_touch_scrolling);
 
   GestureProviderClient* const client_;
 
   class GestureListenerImpl;
   friend class GestureListenerImpl;
   scoped_ptr<GestureListenerImpl> gesture_listener_;
 
   class ScaleGestureListenerImpl;
   friend class ScaleGestureListenerImpl;
   scoped_ptr<ScaleGestureListenerImpl> scale_gesture_listener_;
@@ skipping 11 matching lines @@
 
   // Whether the respective {SCROLL,PINCH}_BEGIN gestures have been terminated
   // with a {SCROLL,PINCH}_END.
   bool touch_scroll_in_progress_;
   bool pinch_in_progress_;
 
   // Keeps track of the current GESTURE_LONG_PRESS event. If a context menu is
   // opened after a GESTURE_LONG_PRESS, this is used to insert a
   // GESTURE_TAP_CANCEL for removing any ::active styling.
   base::TimeTicks current_longpress_time_;
+
+  bool gesture_begin_end_types_enabled_;
 };
 
 }  //  namespace ui
 
 #endif  // UI_EVENTS_GESTURE_DETECTION_GESTURE_PROVIDER_H_