Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(90)

Issue 2922383002: [Merge M60] arc: Provide metric to monitor Play Store window is shown time. (Closed)

Created:
3 years, 6 months ago by khmel
Modified:
3 years, 6 months ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/branch-heads/3112
Project:
chromium
Visibility:
Public.

Description

[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} Committed: https://chromium.googlesource.com/chromium/src/+/5e6cd66d550e0942591b62710c8b429d65d65dbc

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+74 lines, -10 lines) Patch
M chrome/browser/chromeos/arc/arc_optin_uma.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/chromeos/arc/arc_optin_uma.cc View 3 chunks +20 lines, -9 lines 0 comments Download
M chrome/browser/ui/ash/launcher/arc_app_window_launcher_controller.h View 4 chunks +14 lines, -1 line 0 comments Download
M chrome/browser/ui/ash/launcher/arc_app_window_launcher_controller.cc View 5 chunks +24 lines, -0 lines 0 comments Download
M tools/metrics/histograms/histograms.xml View 2 chunks +15 lines, -0 lines 0 comments Download

Messages

Total messages: 7 (5 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2922383002/1
3 years, 6 months ago (2017-06-06 19:35:55 UTC) #4
commit-bot: I haz the power
3 years, 6 months ago (2017-06-06 19:36:31 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/5e6cd66d550e0942591b62710c8b...

Powered by Google App Engine
This is Rietveld 408576698