Add User Actions and adding more details to UMA metrics for overscroll navigation

content/browser/web_contents/aura/gesture_nav_simple.cc

Index: content/browser/web_contents/aura/gesture_nav_simple.cc
diff --git a/content/browser/web_contents/aura/gesture_nav_simple.cc b/content/browser/web_contents/aura/gesture_nav_simple.cc
index 17b94bcc97b0006a63472be3bd2fc9836c54af89..ad5cdda448dc728fab74fc12cda1c5d98ed321cf 100644
--- a/content/browser/web_contents/aura/gesture_nav_simple.cc
+++ b/content/browser/web_contents/aura/gesture_nav_simple.cc
@@ -374,7 +374,8 @@ void GestureNavSimple::OnOverscrollComplete(OverscrollMode overscroll_mode) {
 }
 
 void GestureNavSimple::OnOverscrollModeChange(OverscrollMode old_mode,
-                                              OverscrollMode new_mode) {
+                                              OverscrollMode new_mode,
+                                              OverscrollSource source) {
   NavigationControllerImpl& controller = web_contents_->GetController();
   if (!ShouldNavigateForward(controller, new_mode) &&
       !ShouldNavigateBack(controller, new_mode)) {
@@ -384,10 +385,17 @@ void GestureNavSimple::OnOverscrollModeChange(OverscrollMode old_mode,
 
   aura::Window* window = web_contents_->GetNativeView();
   const gfx::Rect& window_bounds = window->bounds();
+  DCHECK_NE(source, OverscrollSource::NONE);
+  float start_threshold =
+      source == OverscrollSource::TOUCHPAD
+          ? GetOverscrollConfig(
+                OVERSCROLL_CONFIG_HORIZ_THRESHOLD_START_TOUCHPAD)
+          : GetOverscrollConfig(
+                OVERSCROLL_CONFIG_HORIZ_THRESHOLD_START_TOUCHSCREEN);

[mohsen, 2017/02/23 19:19:10]

nit: I would put conditional ternary operator inside GetOverscrollConfig() to
avoid typing GetOverscrollConfig() twice!

   completion_threshold_ =
       window_bounds.width() *
           GetOverscrollConfig(OVERSCROLL_CONFIG_HORIZ_THRESHOLD_COMPLETE) -
-      GetOverscrollConfig(OVERSCROLL_CONFIG_HORIZ_THRESHOLD_START_TOUCHSCREEN);
+      start_threshold;
 
   affordance_.reset(new Affordance(new_mode, window_bounds));