Android double/triple click

content/browser/renderer_host/render_widget_host_view_android.cc

 // Copyright (c) 2012 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 "content/browser/renderer_host/render_widget_host_view_android.h"
 
 #include <android/bitmap.h>
 
 #include <utility>
 
@@ skipping 84 matching lines @@
 #include "ui/gfx/android/view_configuration.h"
 #include "ui/gfx/geometry/dip_util.h"
 #include "ui/gfx/geometry/size_conversions.h"
 #include "ui/touch_selection/touch_selection_controller.h"
 
 namespace content {
 
 namespace {
 
 static const char kAsyncReadBackString[] = "Compositing.CopyFromSurfaceTime";
+static const base::TimeDelta kClickCountInterval =
+    base::TimeDelta::FromSecondsD(0.5);
+static const float kClickCountRadiusDIP = 5;
 
 class PendingReadbackLock;
 
 PendingReadbackLock* g_pending_readback_lock = nullptr;
 
 class PendingReadbackLock : public base::RefCounted<PendingReadbackLock> {
  public:
   PendingReadbackLock() {
     DCHECK_EQ(g_pending_readback_lock, nullptr);
     g_pending_readback_lock = this;
@@ skipping 1679 matching lines @@
   latency_info.AddLatencyNumber(ui::INPUT_EVENT_LATENCY_UI_COMPONENT, 0, 0);
   target_host->ForwardKeyboardEventWithLatencyInfo(event, latency_info);
 }
 
 void RenderWidgetHostViewAndroid::SendMouseEvent(
     const ui::MotionEventAndroid& motion_event,
     int action_button) {
   blink::WebInputEvent::Type webMouseEventType =
       ui::ToWebMouseEventType(motion_event.GetAction());
 
+  const base::TimeTicks current_time = base::TimeTicks::Now();
+  if (webMouseEventType == blink::WebInputEvent::kMouseDown) {
+    if (action_button == ui::MotionEventAndroid::BUTTON_PRIMARY) {
+      gfx::Point mousedown_point(motion_event.GetX(0), motion_event.GetY(0));
+      const base::TimeDelta time_delay =
+          current_time - prev_mousedown_timestamp_;
+      const float distance = (mousedown_point - prev_mousedown_point_).Length();

[aelias_OOO_until_Jul13, 2017/06/21 01:13:12]

Although this doesn't happen on mousemove I still don't really like having a
unnecessary sqrt call in an input critical path because I've seen that bubble up
in profiles before -- I would prefer "LengthSquared() <
kClickCountRadiusDIPSquared)".  I might go back and refactor vector2d.h later to
have a call LengthLessThan() instead of LengthSquared() to make the code nicer,
but for this patch let's go with that.

[amaralp, 2017/06/21 19:34:21]

Done.

+      if (click_count_ > 2 || time_delay > kClickCountInterval ||
+          distance > kClickCountRadiusDIP) {
+        click_count_ = 0;
+      }
+      prev_mousedown_timestamp_ = current_time;
+      prev_mousedown_point_ = mousedown_point;
+    } else {
+      // Reset state if left button isn't pressed.

[aelias_OOO_until_Jul13, 2017/06/21 01:13:12]

"if middle or right button was pressed"

[amaralp, 2017/06/21 19:34:21]

Done.

+      click_count_ = 0;
+      prev_mousedown_timestamp_ = base::TimeTicks();
+      prev_mousedown_point_ = gfx::Point();

[aelias_OOO_until_Jul13, 2017/06/21 01:13:12]

gfx::Point(0, 0) can be a legal offset or close to it, so it's not a principled
invalid value.  I'd suggest deleting this line as it doesn't do anything, to
avoid giving the impression that there might be a bug with double-clicks near
the top-left of the screen.

[amaralp, 2017/06/21 19:34:22]

Done.

+    }
+    click_count_++;
+  }
+
   blink::WebMouseEvent mouse_event = WebMouseEventBuilder::Build(
       webMouseEventType,
       ui::EventTimeStampToSeconds(motion_event.GetEventTime()),
       motion_event.GetX(0), motion_event.GetY(0), motion_event.GetFlags(),
-      motion_event.GetButtonState() ? 1 : 0 /* click count */,
+      motion_event.GetButtonState() ? click_count_ : 0 /* click count */,
       motion_event.GetPointerId(0), motion_event.GetPressure(0),
       motion_event.GetOrientation(0), motion_event.GetTiltX(0),
       motion_event.GetTiltY(0), action_button, motion_event.GetToolType(0));
 
   if (!host_ || !host_->delegate())
     return;
 
   if (SiteIsolationPolicy::AreCrossProcessFramesPossible() &&
       host_->delegate()->GetInputEventRouter()) {
     host_->delegate()->GetInputEventRouter()->RouteMouseEvent(
@@ skipping 457 matching lines @@
 
   ui::WindowAndroidCompositor* compositor = window_android->GetCompositor();
   if (!compositor)
     return;
 
   overscroll_controller_ = base::MakeUnique<OverscrollControllerAndroid>(
       overscroll_refresh_handler, compositor, view_.GetDipScale());
 }
 
 }  // namespace content