New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 666072 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Double/Triple click with mouse not working on Android

Project Member Reported by amaralp@chromium.org, Nov 16 2016

Issue description

Version: M56
OS: Android

What steps will reproduce the problem?
(1) Double/Triple click on a text (doesn't matter if it is editable or not).

What is the expected result?
Text should be selected

What happens instead?
Nothing

Related to  crbug.com/666060 .

 
Please let us know if there are any updates on this issue.
Thank you!
Please let us know if there are any updates on this issue.
Thank you!

Comment 3 by aelias@chromium.org, Jun 20 2017

We're actively working on this in https://codereview.chromium.org/2943383002/.
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 22 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/30fe79726764ec6ce93001069bee890ceb529a20

commit 30fe79726764ec6ce93001069bee890ceb529a20
Author: amaralp <amaralp@chromium.org>
Date: Thu Jun 22 02:17:57 2017

Android double/triple click

Unlike most OSes Android's mouse click events don't come with the click count.
This CL keeps track of the click count for Android mouse clicks so that double
and triple clicks can work. From experimentation on Linux and online research
I found that the click count depends both on the time between the clicks and
the distance between them. Windows has a max time delay of 0.5 seconds which is
what I've used in this CL. I couldn't find a max distance value but from my
experimentation 5 DIPs seems reasonable.

BUG= 666072 

Review-Url: https://codereview.chromium.org/2943383002
Cr-Commit-Position: refs/heads/master@{#481405}

[modify] https://crrev.com/30fe79726764ec6ce93001069bee890ceb529a20/content/browser/renderer_host/render_widget_host_view_android.cc
[modify] https://crrev.com/30fe79726764ec6ce93001069bee890ceb529a20/content/browser/renderer_host/render_widget_host_view_android.h

Status: Fixed (was: Assigned)

Comment 6 by aelias@chromium.org, Jun 23 2017

The fix will roll out with Chrome 61 (scheduled to go to stable channel roughly in late August).
By any chance, is there a possibility to have this change pulled into Chrome 60?
@amaralp, please let us know if there are any possibilities to have this change pulled into Chrome 60.
Thank you!

Comment 9 by aelias@chromium.org, Jun 28 2017

It doesn't seem to meet our merge guidelines, since it affects a small number of users, there is alternate way of achieving it (drag to select), and the bug  has already been in stable channel for 2 versions already.

Sign in to add a comment