Remove deprecated gesture configuration parameters.

chrome/browser/ui/gesture_prefs_observer_factory_aura.cc

 // Copyright (c) 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.
 
 #include "chrome/browser/ui/gesture_prefs_observer_factory_aura.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/compiler_specific.h"
 #include "base/prefs/pref_change_registrar.h"
 #include "base/prefs/pref_service.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/profiles/incognito_helpers.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/pref_names.h"
 #include "components/keyed_service/content/browser_context_dependency_manager.h"
 #include "components/pref_registry/pref_registry_syncable.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/overscroll_configuration.h"
 #include "content/public/common/renderer_preferences.h"
 #include "ui/events/gestures/gesture_configuration.h"
 
 using ui::GestureConfiguration;
 
 namespace {
 
-// TODO(tdresser): Remove these deprecated prefs in M38. See crbug.com/379912.
-
-const char kFlingVelocityCap[] = "gesture.fling_velocity_cap";
-const char kLongPressTimeInSeconds[] =
-    "gesture.long_press_time_in_seconds";
-const char kMaxDistanceBetweenTapsForDoubleTap[] =
-    "gesture.max_distance_between_taps_for_double_tap";
-const char kMaxDistanceForTwoFingerTapInPixels[] =
-    "gesture.max_distance_for_two_finger_tap_in_pixels";
-const char kMaxSecondsBetweenDoubleClick[] =
-    "gesture.max_seconds_between_double_click";
-const char kMaxSwipeDeviationRatio[] =
-    "gesture.max_swipe_deviation_ratio";
-const char kMaxTouchDownDurationInSecondsForClick[] =
-    "gesture.max_touch_down_duration_in_seconds_for_click";
-const char kMaxTouchMoveInPixelsForClick[] =
-    "gesture.max_touch_move_in_pixels_for_click";
-const char kMinDistanceForPinchScrollInPixels[] =
-    "gesture.min_distance_for_pinch_scroll_in_pixels";
-const char kMinFlickSpeedSquared[] =
-    "gesture.min_flick_speed_squared";
-const char kMinPinchUpdateDistanceInPixels[] =
-    "gesture.min_pinch_update_distance_in_pixels";
-const char kMinRailBreakVelocity[] =
-    "gesture.min_rail_break_velocity";
-const char kMinScrollDeltaSquared[] =
-    "gesture.min_scroll_delta_squared";
-const char kMinSwipeSpeed[] =
-    "gesture.min_swipe_speed";
-const char kMinTouchDownDurationInSecondsForClick[] =
-    "gesture.min_touch_down_duration_in_seconds_for_click";
-const char kPointsBufferedForVelocity[] =
-    "gesture.points_buffered_for_velocity";
-const char kRailBreakProportion[] =
-    "gesture.rail_break_proportion";
-const char kRailStartProportion[] =
-    "gesture.rail_start_proportion";
-const char kScrollPredictionSeconds[] =
-    "gesture.scroll_prediction_seconds";
-const char kShowPressDelayInMS[] =
-    "gesture.show_press_delay_in_ms";
-
 struct OverscrollPref {
   const char* pref_name;
   content::OverscrollConfig config;
 };
 
 const std::vector<OverscrollPref>& GetOverscrollPrefs() {
   CR_DEFINE_STATIC_LOCAL(std::vector<OverscrollPref>, overscroll_prefs, ());
   if (overscroll_prefs.empty()) {
     using namespace content;
     const OverscrollPref kOverscrollPrefs[] = {
@@ skipping 67 matching lines @@
   prefs::kFlingCurveTouchpadBeta,
   prefs::kFlingCurveTouchpadGamma
 };
 
 const char* kFlingTouchscreenPrefs[] = {
   prefs::kFlingCurveTouchscreenAlpha,
   prefs::kFlingCurveTouchscreenBeta,
   prefs::kFlingCurveTouchscreenGamma,
 };
 
-const char* kPrefsToRemove[] = {
-  kFlingVelocityCap,
-  kLongPressTimeInSeconds,
-  kMaxDistanceBetweenTapsForDoubleTap,
-  kMaxDistanceForTwoFingerTapInPixels,
-  kMaxSecondsBetweenDoubleClick,
-  kMaxSwipeDeviationRatio,
-  kMaxTouchDownDurationInSecondsForClick,
-  kMaxTouchMoveInPixelsForClick,
-  kMinDistanceForPinchScrollInPixels,
-  kMinFlickSpeedSquared,
-  kMinPinchUpdateDistanceInPixels,
-  kMinRailBreakVelocity,
-  kMinScrollDeltaSquared,
-  kMinSwipeSpeed,
-  kMinTouchDownDurationInSecondsForClick,
-  kPointsBufferedForVelocity,
-  kRailBreakProportion,
-  kRailStartProportion,
-  kScrollPredictionSeconds,
-  kShowPressDelayInMS,
-};
-
 GesturePrefsObserver::GesturePrefsObserver(PrefService* prefs)
     : prefs_(prefs) {
-  // Clear for migration.
-  for (size_t i = 0; i < arraysize(kPrefsToRemove); ++i) {
-    if (prefs->FindPreference(kPrefsToRemove[i]))
-      prefs->ClearPref(kPrefsToRemove[i]);
-  }
-
   registrar_.Init(prefs);
   registrar_.RemoveAll();
   base::Closure callback = base::Bind(&GesturePrefsObserver::Update,
                                       base::Unretained(this));
 
   base::Closure notify_callback = base::Bind(&GesturePrefsObserver::Notify,
                                              base::Unretained(this));
 
   for (size_t i = 0; i < arraysize(kPrefsToObserve); ++i)
     registrar_.Add(kPrefsToObserve[i], callback);
@@ skipping 161 matching lines @@
   // Use same gesture preferences on incognito windows.
   return chrome::GetBrowserContextRedirectedInIncognito(context);
 }
 
 bool GesturePrefsObserverFactoryAura::ServiceIsNULLWhileTesting() const {
   // Some tests replace the PrefService of the TestingProfile after the
   // GesturePrefsObserver has been created, which makes Shutdown()
   // remove the registrar from a non-existent PrefService.
   return true;
 }