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

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

Issue 2542453003: Suppress LongPress/Tap, and TwoFingerTap when TapDown cancels a fling. (Closed)
Patch Set: Created 4 years 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 53aaee3f8800c06d437dbbe1be75a8147c05596e..e48618180add7ddc7333f4ea31fe2b54acf70aca 100644
--- a/content/browser/renderer_host/input/input_router_config_helper.cc
+++ b/content/browser/renderer_host/input/input_router_config_helper.cc
@@ -49,9 +49,12 @@ GestureEventQueue::Config GetGestureEventQueueConfig() {
config.touchscreen_tap_suppression_config.max_cancel_to_down_time =
base::TimeDelta::FromMilliseconds(
gesture_config->fling_max_cancel_to_down_time_in_ms());
+ // Tap suppression controller forwards the stashed tapDown when the
+ // tapDownTimer expires. The timer delay should be large enough for a long
+ // press to get suppressed before forwarding the stashed tapDown.
tdresser 2016/12/07 19:53:21 Explain why we need this delay - to protect agains
sahel 2016/12/09 16:49:11 Done.
config.touchscreen_tap_suppression_config.max_tap_gap_time =
base::TimeDelta::FromMilliseconds(
- gesture_config->long_press_time_in_ms());
+ gesture_config->long_press_time_in_ms() + 50);
tdresser 2016/12/07 19:53:21 We should make sure to handle the case where a lon
sahel 2016/12/09 16:49:11 Done.
config.touchpad_tap_suppression_config.enabled =
gesture_config->fling_touchpad_tap_suppression_enabled();

Powered by Google App Engine
This is Rietveld 408576698