Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(416)

Unified Diff: content/browser/renderer_host/input/input_router_config_helper.cc

Issue 613373004: Adopt ui::GestureConfiguration on Android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Copyright Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/input/input_router_config_helper.cc
diff --git a/content/browser/renderer_host/input/input_router_config_helper.cc b/content/browser/renderer_host/input/input_router_config_helper.cc
index 1956fe65107ba5ff43a8323a03391566bd632753..d78b6cb2a90732dabbc36036c0756c8e9b851438 100644
--- a/content/browser/renderer_host/input/input_router_config_helper.cc
+++ b/content/browser/renderer_host/input/input_router_config_helper.cc
@@ -9,7 +9,7 @@
#include "ui/events/gesture_detection/gesture_detector.h"
#if defined(USE_AURA)
-#include "ui/events/gestures/gesture_configuration.h"
+#include "ui/events/gesture_detection/gesture_configuration.h"
#elif defined(OS_ANDROID)
#include "ui/gfx/android/view_configuration.h"
#include "ui/gfx/screen.h"
@@ -19,30 +19,32 @@ namespace content {
namespace {
#if defined(USE_AURA)
-
+// TODO(jdduke): Consolidate router configuration paths using
+// ui::GestureConfiguration.
GestureEventQueue::Config GetGestureEventQueueConfig() {
GestureEventQueue::Config config;
-
+ ui::GestureConfiguration* gesture_config =
+ ui::GestureConfiguration::GetInstance();
config.debounce_interval = base::TimeDelta::FromMilliseconds(
- ui::GestureConfiguration::scroll_debounce_interval_in_ms());
+ gesture_config->scroll_debounce_interval_in_ms());
config.touchscreen_tap_suppression_config.enabled = true;
config.touchscreen_tap_suppression_config.max_cancel_to_down_time =
base::TimeDelta::FromMilliseconds(
- ui::GestureConfiguration::fling_max_cancel_to_down_time_in_ms());
+ gesture_config->fling_max_cancel_to_down_time_in_ms());
config.touchscreen_tap_suppression_config.max_tap_gap_time =
base::TimeDelta::FromMilliseconds(
- ui::GestureConfiguration::semi_long_press_time_in_ms());
+ gesture_config->semi_long_press_time_in_ms());
config.touchpad_tap_suppression_config.enabled = true;
config.touchpad_tap_suppression_config.max_cancel_to_down_time =
base::TimeDelta::FromMilliseconds(
- ui::GestureConfiguration::fling_max_cancel_to_down_time_in_ms());
+ gesture_config->fling_max_cancel_to_down_time_in_ms());
config.touchpad_tap_suppression_config.max_tap_gap_time =
base::TimeDelta::FromMilliseconds(
- ui::GestureConfiguration::fling_max_tap_gap_time_in_ms());
+ gesture_config->fling_max_tap_gap_time_in_ms());
return config;
}
@@ -51,7 +53,8 @@ TouchEventQueue::Config GetTouchEventQueueConfig() {
TouchEventQueue::Config config;
config.touchmove_slop_suppression_length_dips =
- ui::GestureConfiguration::max_touch_move_in_pixels_for_click();
+ ui::GestureConfiguration::GetInstance()
+ ->max_touch_move_in_pixels_for_click();
return config;
}

Powered by Google App Engine
This is Rietveld 408576698