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

Issue 728880 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

arc: Provide metric to monitor Play Store window is shown time.

Project Member Reported by khmel@chromium.org, Jun 2 2017

Issue description

Need UMA to track time between initial ARC OptIn and moment
when the first Play Store window is shown.
 
(internal bug: b/62061882)

Project Member

Comment 2 by bugdroid1@chromium.org, Jun 3 2017

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

commit 3f9e6d5f978cfe76acff68fb181730b26a4612b1
Author: khmel <khmel@google.com>
Date: Sat Jun 03 02:59:03 2017

arc: Provide metric to monitor Play Store window is shown time.

This adds UMA to track time between initial ARC OptIn and moment
when the first Play Store window is shown.

BUG= 728880 
TEST=Manually for managed/unmanaged accounts. Confirmed that
     metric is updted only for initial ARC boot.

Change-Id: I2b988f039ec709a4d9c369a27ccd01777b688028
Reviewed-on: https://chromium-review.googlesource.com/522442
Commit-Queue: Yury Khmel <khmel@google.com>
Reviewed-by: Yusuke Sato <yusukes@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#476874}
[modify] https://crrev.com/3f9e6d5f978cfe76acff68fb181730b26a4612b1/chrome/browser/chromeos/arc/arc_optin_uma.cc
[modify] https://crrev.com/3f9e6d5f978cfe76acff68fb181730b26a4612b1/chrome/browser/chromeos/arc/arc_optin_uma.h
[modify] https://crrev.com/3f9e6d5f978cfe76acff68fb181730b26a4612b1/chrome/browser/ui/ash/launcher/arc_app_window_launcher_controller.cc
[modify] https://crrev.com/3f9e6d5f978cfe76acff68fb181730b26a4612b1/chrome/browser/ui/ash/launcher/arc_app_window_launcher_controller.h
[modify] https://crrev.com/3f9e6d5f978cfe76acff68fb181730b26a4612b1/tools/metrics/histograms/histograms.xml

Comment 3 by khmel@chromium.org, Jun 6 2017

Cc: josa...@chromium.org
Labels: Merge-Request-60
Project Member

Comment 4 by sheriffbot@chromium.org, Jun 6 2017

Labels: -Merge-Request-60 Hotlist-Merge-Approved Merge-Approved-60
Your change meets the bar and is auto-approved for M60. Please go ahead and merge the CL to branch 3112 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 5 by bugdroid1@chromium.org, Jun 6 2017

Labels: -merge-approved-60 merge-merged-3112
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5e6cd66d550e0942591b62710c8b429d65d65dbc

commit 5e6cd66d550e0942591b62710c8b429d65d65dbc
Author: khmel <khmel@chromium.org>
Date: Tue Jun 06 19:36:16 2017

[Merge M60] arc: Provide metric to monitor Play Store window is shown time.

This adds UMA to track time between initial ARC OptIn and moment
when the first Play Store window is shown.

BUG= 728880 
TEST=Manually for managed/unmanaged accounts. Confirmed that
     metric is updted only for initial ARC boot.
TBR=yusukes@chromium.org,oshima@chromium.org,isherman@chromium.org
NOTRY=true
NOPRESUBMIT=true

Change-Id: I2b988f039ec709a4d9c369a27ccd01777b688028
Reviewed-on: https://chromium-review.googlesource.com/522442
Commit-Queue: Yury Khmel <khmel@google.com>
Reviewed-by: Yusuke Sato <yusukes@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#476874}
Review-Url: https://codereview.chromium.org/2922383002
Cr-Commit-Position: refs/branch-heads/3112@{#199}
Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}

[modify] https://crrev.com/5e6cd66d550e0942591b62710c8b429d65d65dbc/chrome/browser/chromeos/arc/arc_optin_uma.cc
[modify] https://crrev.com/5e6cd66d550e0942591b62710c8b429d65d65dbc/chrome/browser/chromeos/arc/arc_optin_uma.h
[modify] https://crrev.com/5e6cd66d550e0942591b62710c8b429d65d65dbc/chrome/browser/ui/ash/launcher/arc_app_window_launcher_controller.cc
[modify] https://crrev.com/5e6cd66d550e0942591b62710c8b429d65d65dbc/chrome/browser/ui/ash/launcher/arc_app_window_launcher_controller.h
[modify] https://crrev.com/5e6cd66d550e0942591b62710c8b429d65d65dbc/tools/metrics/histograms/histograms.xml

Comment 6 by khmel@chromium.org, Jun 28 2017

Status: Fixed (was: Started)

Comment 7 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment