Disable tap suppression logic for WebView

ui/events/gesture_detection/gesture_configuration.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_CONFIGURATION_H_
 #define UI_EVENTS_GESTURE_DETECTION_GESTURE_CONFIGURATION_H_
 
 #include "base/basictypes.h"
 #include "ui/events/gesture_detection/gesture_detection_export.h"
 #include "ui/events/gesture_detection/velocity_tracker.h"
@@ skipping 10 matching lines @@
   // Ordered alphabetically ignoring underscores.
   float default_radius() const { return default_radius_; }
   void set_default_radius(float radius) {
     default_radius_ = radius;
     min_scaling_touch_major_ = default_radius_ * 2;
     min_gesture_bounds_length_ = default_radius_;
   }
   bool double_tap_enabled() const { return double_tap_enabled_; }
   void set_double_tap_enabled(bool enabled) { double_tap_enabled_ = enabled; }
   int double_tap_timeout_in_ms() const { return double_tap_timeout_in_ms_; }
+  bool fling_touchpad_tap_suppression_enabled() const {
+    return fling_touchpad_tap_suppression_enabled_;
+  }
+  void set_fling_touchpad_tap_suppression_enabled(bool enabled) {
+    fling_touchpad_tap_suppression_enabled_ = enabled;
+  }
+  bool fling_touchscreen_tap_suppression_enabled() const {
+    return fling_touchscreen_tap_suppression_enabled_;
+  }
+  void set_fling_touchscreen_tap_suppression_enabled(bool enabled) {
+    fling_touchscreen_tap_suppression_enabled_ = enabled;
+  }
   int fling_max_cancel_to_down_time_in_ms() const {
     return fling_max_cancel_to_down_time_in_ms_;
   }
   void set_fling_max_cancel_to_down_time_in_ms(int val) {
     fling_max_cancel_to_down_time_in_ms_ = val;
   }
   int fling_max_tap_gap_time_in_ms() const {
     return fling_max_tap_gap_time_in_ms_;
   }
   void set_fling_max_tap_gap_time_in_ms(int val) {
@@ skipping 148 matching lines @@
   // NOTE: Adding new configuration parameters requires initializing
   // corresponding entries in aura_test_base.cc's SetUp().
 
   // The default touch radius length used when the only information given
   // by the device is the touch center.
   float default_radius_;
 
   bool double_tap_enabled_;
   int double_tap_timeout_in_ms_;
 
+  // Whether to suppress touchscreen/touchpad taps that occur during a fling (
+  // in particular, when such taps cancel the active fling).
+  bool fling_touchpad_tap_suppression_enabled_;
+  bool fling_touchscreen_tap_suppression_enabled_;

[tdresser, 2015/08/04 12:20:50]

I have a mild preference for the name:
touch.*_fling_tap_suppression_enabled_.

Feel free to leave it this way if you prefer this.

[jdduke (slow), 2015/08/04 17:19:25]

Yeah, I actually had that originally, and went back and forth. The reason I
somewhat prefer this version is because the source (touchpad/touchscreen) refers
to the tap source, rather than the fling source. I considered dropping the
"fling_" prefix altogether, but there's some precedent in the other variables.
Let me know what you'd like either way.

[tdresser, 2015/08/04 21:00:15]

LGTM.

+
   // Maximum time between a GestureFlingCancel and a mousedown such that the
   // mousedown is considered associated with the cancel event.
   int fling_max_cancel_to_down_time_in_ms_;
 
   // Maxium time between a mousedown/mouseup pair that is considered to be a
   // suppressable tap.
   int fling_max_tap_gap_time_in_ms_;
   bool gesture_begin_end_types_enabled_;
   int long_press_time_in_ms_;
   float max_distance_between_taps_for_double_tap_;
@@ skipping 27 matching lines @@
   int tab_scrub_activation_delay_in_ms_;
   bool two_finger_tap_enabled_;
   VelocityTracker::Strategy velocity_tracker_strategy_;
 
   DISALLOW_COPY_AND_ASSIGN(GestureConfiguration);
 };
 
 }  // namespace ui
 
 #endif  // UI_EVENTS_GESTURE_DETECTION_GESTURE_CONFIGURATION_H_