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

Issue 2783423002: [Merge M58] arc: Provide UMA for OptIn flow. (Closed)

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

Description

[Merge M58] arc: Provide UMA for OptIn flow. We have set of different UMAs for ARC that we can use to analyze how OptIn flow passed. However there was no metric that explicitly defines how OptIn finished. This adds top level metric that shows how OptIn has finished. NOTRY=true NOPRESUBMIT=true TBR=lhchavez@chromium.org,isherman@chromium.org BUG=706564 TEST=Various modes for managed/unmanaged users. chrome://histograms shows expected results. Review-Url: https://codereview.chromium.org/2785813002 Cr-Commit-Position: refs/heads/master@{#460892} (cherry picked from commit 8575dd76a6ceb99e0c87738d6ccc3db0c70d5176) Review-Url: https://codereview.chromium.org/2783423002 Cr-Commit-Position: refs/branch-heads/3029@{#501} Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471} Committed: https://chromium.googlesource.com/chromium/src/+/895e748b018282644cf4678908124b0267011688

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+183 lines, -15 lines) Patch
M chrome/browser/chromeos/arc/arc_optin_uma.h View 2 chunks +72 lines, -14 lines 0 comments Download
M chrome/browser/chromeos/arc/arc_optin_uma.cc View 1 chunk +5 lines, -0 lines 0 comments Download
M chrome/browser/chromeos/arc/arc_session_manager.h View 2 chunks +5 lines, -0 lines 0 comments Download
M chrome/browser/chromeos/arc/arc_session_manager.cc View 7 chunks +80 lines, -0 lines 0 comments Download
M tools/metrics/histograms/histograms.xml View 2 chunks +21 lines, -1 line 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/2783423002/1
3 years, 8 months ago (2017-03-30 22:59:39 UTC) #4
commit-bot: I haz the power
3 years, 8 months ago (2017-03-30 23:37:18 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/895e748b018282644cf467890812...

Powered by Google App Engine
This is Rietveld 408576698