New issue
Advanced search Search tips

Issue 668248 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

We need UMA counters for touch vs mouse activity on Android

Project Member Reported by mustaq@chromium.org, Nov 23 2016

Issue description

Currently we have no data on what fraction of Chrome Android users use mouse. UseCounters for MotionEvents with tool-type mouse vs touch should be enough.
 
Labels: ReleaseBlock-Beta
We want the counts in M56-beta, at the same time Android mouse event change (https://docs.google.com/document/d/1mpBR7J7kgTXvp0QACVjhxtwNJ7bgGoTMmxfxN2dupGg/edit?usp=sharing) goes live, to be able to assess the risk before something comes up.

Labels: Merge-Request-56
Status: Fixed (was: Assigned)

Comment 4 by dimu@chromium.org, Dec 5 2016

Labels: -Merge-Request-56 Merge-Approved-56 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M56 (branch: 2924)
Project Member

Comment 5 by bugdroid1@chromium.org, Dec 5 2016

Labels: -merge-approved-56 merge-merged-2924
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ee99ab8496faa85a4986ab7f8dc7ec1044eff379

commit ee99ab8496faa85a4986ab7f8dc7ec1044eff379
Author: Mustaq Ahmed <mustaq@google.com>
Date: Mon Dec 05 15:10:07 2016

Added UMA counters for Android tool-types.

BUG= 668248 

Review-Url: https://codereview.chromium.org/2536413003
Cr-Commit-Position: refs/heads/master@{#436069}
(cherry picked from commit 097c9551cffc40afcb8a4cbd78d06f6523f412d6)

Review URL: https://codereview.chromium.org/2551023003 .

Cr-Commit-Position: refs/branch-heads/2924@{#329}
Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059}

[modify] https://crrev.com/ee99ab8496faa85a4986ab7f8dc7ec1044eff379/content/browser/android/content_view_core_impl.cc
[modify] https://crrev.com/ee99ab8496faa85a4986ab7f8dc7ec1044eff379/tools/metrics/histograms/histograms.xml
[modify] https://crrev.com/ee99ab8496faa85a4986ab7f8dc7ec1044eff379/ui/events/gesture_detection/motion_event.h

Sign in to add a comment