Consolidate all touch/gesture related constants in content

content/browser/renderer_host/input/tap_suppression_controller.h

 // Copyright 2013 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_INPUT_TAP_SUPPRESSION_CONTROLLER_H_
 #define CONTENT_BROWSER_RENDERER_HOST_INPUT_TAP_SUPPRESSION_CONTROLLER_H_
 
 #include "base/time/time.h"
 #include "base/timer/timer.h"
 #include "content/common/content_export.h"
 
 namespace content {
 
 class TapSuppressionControllerClient;
 
 // The core controller for suppression of taps (touchpad or touchscreen)
 // immediately following a GestureFlingCancel event (caused by the same tap).
 // Only taps of sufficient speed and within a specified time window after a
 // 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.
   void GestureFlingCancel();
 
   // Should be called whenever an ACK for a GestureFlingCancel event is
   // received. |processed| is true when the GestureFlingCancel actually stopped
   // a fling and therefore should suppress the forwarding of the following tap.
   void GestureFlingCancelAck(bool processed);
 
   // Should be called whenever a tap down (touchpad or touchscreen) is received.
   // Returns true if the tap down should be deferred. The caller is responsible
   // for keeping the event for later release, if needed.
   bool ShouldDeferTapDown();
 
   // Should be called whenever a tap ending event is received. Returns true if
   // the tap event should be suppressed.
   bool ShouldSuppressTapEnd();
 
  protected:
   virtual base::TimeTicks Now();
   virtual void StartTapDownTimer(const base::TimeDelta& delay);
   virtual void StopTapDownTimer();
   void TapDownTimerExpired();
 
  private:
   friend class MockTapSuppressionController;
 
   enum State {
+    DISABLED,
     NOTHING,
     GFC_IN_PROGRESS,
     TAP_DOWN_STASHED,
     LAST_CANCEL_STOPPED_FLING,
   };
 
 
   TapSuppressionControllerClient* client_;
   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.
   base::TimeTicks fling_cancel_time_;
 
   DISALLOW_COPY_AND_ASSIGN(TapSuppressionController);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_INPUT_TAP_SUPPRESSION_CONTROLLER_H_