Chromium Code Reviews| Index: content/browser/renderer_host/input/tap_suppression_controller.h |
| diff --git a/content/browser/renderer_host/input/tap_suppression_controller.h b/content/browser/renderer_host/input/tap_suppression_controller.h |
| index ef662f32f7b99bcc03be76b623a98264aef38d07..7f2a4702a1ab1cfc2ce243aee3b0103032737a4e 100644 |
| --- a/content/browser/renderer_host/input/tap_suppression_controller.h |
| +++ b/content/browser/renderer_host/input/tap_suppression_controller.h |
| @@ -19,7 +19,22 @@ class TapSuppressionControllerClient; |
| // GestureFlingCancel are suppressed. |
| class CONTENT_EXPORT TapSuppressionController { |
| public: |
| - explicit TapSuppressionController(TapSuppressionControllerClient* client); |
| + struct CONTENT_EXPORT Config { |
| + Config(); |
| + |
| + // Defaults to false, in which case no suppression is performed. |
| + bool enabled; |
| + |
| + // The maximum time allowed between a GestureFlingCancel and its |
| + // corresponding tap down. |
| + base::TimeDelta max_cancel_to_down_time; |
| + |
| + // The maximum time allowed between a single tap's down and up events. |
| + base::TimeDelta max_tap_gap_time; |
| + }; |
| + |
| + TapSuppressionController(TapSuppressionControllerClient* client, |
| + const Config& config); |
| virtual ~TapSuppressionController(); |
| // Should be called whenever a GestureFlingCancel event is received. |
| @@ -49,6 +64,7 @@ class CONTENT_EXPORT TapSuppressionController { |
| friend class MockTapSuppressionController; |
| enum State { |
| + DISABLED, |
| NOTHING, |
| GFC_IN_PROGRESS, |
| TAP_DOWN_STASHED, |
| @@ -60,6 +76,10 @@ class CONTENT_EXPORT TapSuppressionController { |
| base::OneShotTimer<TapSuppressionController> tap_down_timer_; |
| State state_; |
| + bool enabled_; |
|
tdresser
2014/05/01 13:51:56
enabled_ is unused.
jdduke (slow)
2014/05/01 16:44:25
Done.
|
| + base::TimeDelta max_cancel_to_down_time_; |
| + base::TimeDelta max_tap_gap_time_; |
| + |
| // TODO(rjkroege): During debugging, the event times did not prove reliable. |
| // Replace the use of base::TimeTicks with an accurate event time when they |
| // become available post http://crbug.com/119556. |