Gesture recognition constants should all be stored in the GestureConfiguration object.

ui/aura/gestures/gesture_sequence.cc

 // 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.
 
 #include "ui/aura/gestures/gesture_sequence.h"
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/time.h"
 #include "ui/aura/event.h"
 #include "ui/aura/root_window.h"
 #include "ui/aura/gestures/gesture_configuration.h"
 #include "ui/base/events.h"
 
-namespace {
-
-// TODO(girard): Make these configurable in sync with this CL
-//               http://crbug.com/100773
-const float kMinimumPinchUpdateDistance = 5;  // in pixels
-const float kMinimumDistanceForPinchScroll = 20;
-const float kLongPressTimeInMilliseconds = 500;
-
-}  // namespace
-
 namespace aura {
 
 namespace {
 
 // ui::EventType is mapped to TouchState so it can fit into 3 bits of
 // Signature.
 enum TouchState {
   TS_RELEASED,
   TS_PRESSED,
   TS_MOVED,
@@ skipping 450 matching lines @@
   Reset();
   return false;
 }
 
 bool GestureSequence::TouchDown(const TouchEvent& event,
     const GesturePoint& point, Gestures* gestures) {
   DCHECK(state_ == GS_NO_GESTURE);
   AppendTapDownGestureEvent(point, gestures);
   long_press_timer_->Start(
       FROM_HERE,
-      base::TimeDelta::FromMilliseconds(kLongPressTimeInMilliseconds),
+      base::TimeDelta::FromMilliseconds(
+          GestureConfiguration::long_press_time_in_ms()),
       this,
       &GestureSequence::AppendLongPressGestureEvent);
   return true;
 }
 
 void GestureSequence::AppendLongPressGestureEvent() {
   const GesturePoint* point = GetPointByPointId(0);
   GestureEvent gesture(
       ui::ET_GESTURE_LONG_PRESS,
       point->first_touch_position().x(),
@@ skipping 41 matching lines @@
 
 bool GestureSequence::PinchUpdate(const TouchEvent& event,
     const GesturePoint& point, Gestures* gestures) {
   DCHECK(state_ == GS_PINCH);
 
   const GesturePoint* point1 = GetPointByPointId(0);
   const GesturePoint* point2 = GetPointByPointId(1);
 
   float distance = point1->Distance(*point2);
   if (abs(distance - pinch_distance_current_) <
-      GestureConfiguration::minimum_pinch_update_distance_in_pixels()) {
+      GestureConfiguration::min_pinch_update_distance_in_pixels()) {
     // The fingers didn't move towards each other, or away from each other,
     // enough to constitute a pinch. But perhaps they moved enough in the same
     // direction to do a two-finger scroll.
     if (!point1->DidScroll(event,
-        GestureConfiguration::minimum_distance_for_pinch_scroll_in_pixels()) ||
+        GestureConfiguration::min_distance_for_pinch_scroll_in_pixels()) ||
         !point2->DidScroll(event,
-        GestureConfiguration::minimum_distance_for_pinch_scroll_in_pixels()))
+        GestureConfiguration::min_distance_for_pinch_scroll_in_pixels()))
       return false;
 
     gfx::Point center = point1->last_touch_position().Middle(
                         point2->last_touch_position());
     AppendScrollGestureUpdate(point, center, gestures);
   } else {
     AppendPinchGestureUpdate(*point1, *point2,
         distance / pinch_distance_current_, gestures);
     pinch_distance_current_ = distance;
   }
@@ skipping 10 matching lines @@
   float distance = point1->Distance(*point2);
   AppendPinchGestureEnd(*point1, *point2,
       distance / pinch_distance_start_, gestures);
 
   pinch_distance_start_ = 0;
   pinch_distance_current_ = 0;
   return true;
 }
 
 }  // namespace aura