[Android] Use DIP coordinates with MotionEventAndroid

ui/events/gesture_detection/snap_scroll_controller.cc

 // Copyright 2014 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/events/gesture_detection/snap_scroll_controller.h"
 
 #include <cmath>
 
 #include "ui/events/gesture_detection/motion_event.h"
+#include "ui/gfx/display.h"
 
 namespace ui {
 namespace {
 const int kSnapBound = 16;
+const float kMinSnapChannelDistance = kSnapBound;
+const float kMaxSnapChannelDistance = kMinSnapChannelDistance * 3.f;
+const float kSnapChannelDipsPerScreenDip = kMinSnapChannelDistance / 480.f;

[tdresser, 2014/04/09 14:00:01]

Looks good, thanks.

+
+float CalculateChannelDistance(const gfx::Display& display) {
+  if (display.bounds().IsEmpty())
+    return kMinSnapChannelDistance;
+
+  float screen_size =
+      std::abs(hypot(static_cast<float>(display.bounds().width()),
+                     static_cast<float>(display.bounds().height())));
+
+  float snap_channel_distance = screen_size * kSnapChannelDipsPerScreenDip;
+  return std::max(kMinSnapChannelDistance,
+                  std::min(kMaxSnapChannelDistance, snap_channel_distance));
+}
+
 }  // namespace
 
-SnapScrollController::Config::Config()
-    : screen_width_pixels(1), screen_height_pixels(1), device_scale_factor(1) {}
 
-SnapScrollController::Config::~Config() {}
-
-SnapScrollController::SnapScrollController(const Config& config)
-    : channel_distance_(CalculateChannelDistance(config)),
+SnapScrollController::SnapScrollController(const gfx::Display& display)
+    : channel_distance_(CalculateChannelDistance(display)),
       snap_scroll_mode_(SNAP_NONE),
       first_touch_x_(-1),
       first_touch_y_(-1),
       distance_x_(0),
       distance_y_(0) {}
 
 SnapScrollController::~SnapScrollController() {}
 
 void SnapScrollController::UpdateSnapScrollMode(float distance_x,
                                                 float distance_y) {
@@ skipping 49 matching lines @@
       first_touch_x_ = -1;
       first_touch_y_ = -1;
       distance_x_ = 0;
       distance_y_ = 0;
       break;
     default:
       break;
   }
 }
 
-// static
-float SnapScrollController::CalculateChannelDistance(const Config& config) {
-  float channel_distance = 16.f;
-
-  const float screen_size = std::abs(
-      hypot((float)config.screen_width_pixels / config.device_scale_factor,
-            (float)config.screen_height_pixels / config.device_scale_factor));
-  if (screen_size < 480.f) {
-    channel_distance = 16.f;
-  } else if (screen_size < 800.f) {
-    channel_distance = 22.f;
-  } else if (screen_size < 1120.f) {
-    channel_distance = 28.f;
-  } else {
-    channel_distance = 34.f;
-  }
-  channel_distance = channel_distance * config.device_scale_factor;
-  if (channel_distance < 16.f)
-    channel_distance = 16.f;
-
-  return channel_distance;
-}
-
 }  // namespace ui