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

Issue 714887 link

Starred by 1 user

Issue metadata

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

Blocking:
issue 688900



Sign in to add a comment

Add UMA stats for migration UI usage

Project Member Reported by dspaid@chromium.org, Apr 25 2017

Issue description

We would like to know how many people have skipped the migration.

 

Comment 1 by dspaid@chromium.org, Apr 25 2017

Additionally it would be good to know how many users have started the migration after attempting to run an android app.

Comment 2 by dspaid@chromium.org, Apr 25 2017

Labels: ArcExt4Migration

Comment 3 by uekawa@google.com, Apr 25 2017

Blocking: 688900
Components: Platform>ARC

Comment 4 by fukino@chromium.org, Apr 25 2017

Status: Assigned (was: Untriaged)
Project Member

Comment 5 by bugdroid1@chromium.org, Apr 29 2017

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

commit b75aa7267969141e12f5ece554be20d81d39b044
Author: fukino <fukino@chromium.org>
Date: Sat Apr 29 15:17:21 2017

cros: Add UMA metrics in encryption migration UI.

Some metrics about encryption migration are already recorded in cryptohomed.
In this CL, I'd like to add following UI-related UMA metrics on the app.

Cryptohome.MigrationUI.FirstScreen
Cryptohome.MigrationUI.UserChoice
Cryptohome.MigrationUI.ConsumedBatteryPercent

BUG= 714887 
TEST=Disable reboot after the migration by modifying the code, and check that chrome://histograms shows the added metrics.

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

[modify] https://crrev.com/b75aa7267969141e12f5ece554be20d81d39b044/chrome/browser/ui/webui/chromeos/login/encryption_migration_screen_handler.cc
[modify] https://crrev.com/b75aa7267969141e12f5ece554be20d81d39b044/chrome/browser/ui/webui/chromeos/login/encryption_migration_screen_handler.h
[modify] https://crrev.com/b75aa7267969141e12f5ece554be20d81d39b044/tools/metrics/histograms/histograms.xml

Labels: Merge-Request-59
Requesting a merge to get more data from dogfooders.
Project Member

Comment 7 by sheriffbot@chromium.org, May 8 2017

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

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

Comment 8 by bugdroid1@chromium.org, May 8 2017

Labels: -merge-approved-59 merge-merged-3071
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3db95f093b5dd7c3924a74b577a60a3b12e1f968

commit 3db95f093b5dd7c3924a74b577a60a3b12e1f968
Author: Naoki Fukino <fukino@chromium.org>
Date: Mon May 08 07:31:46 2017

cros: Add UMA metrics in encryption migration UI.

Some metrics about encryption migration are already recorded in cryptohomed.
In this CL, I'd like to add following UI-related UMA metrics on the app.

Cryptohome.MigrationUI.FirstScreen
Cryptohome.MigrationUI.UserChoice
Cryptohome.MigrationUI.ConsumedBatteryPercent

BUG= 714887 
TEST=Disable reboot after the migration by modifying the code, and check that chrome://histograms shows the added metrics.

Review-Url: https://codereview.chromium.org/2849833002
Cr-Commit-Position: refs/heads/master@{#468241}
(cherry picked from commit b75aa7267969141e12f5ece554be20d81d39b044)

Review-Url: https://codereview.chromium.org/2871493002 .
Cr-Commit-Position: refs/branch-heads/3071@{#444}
Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}

[modify] https://crrev.com/3db95f093b5dd7c3924a74b577a60a3b12e1f968/chrome/browser/ui/webui/chromeos/login/encryption_migration_screen_handler.cc
[modify] https://crrev.com/3db95f093b5dd7c3924a74b577a60a3b12e1f968/chrome/browser/ui/webui/chromeos/login/encryption_migration_screen_handler.h
[modify] https://crrev.com/3db95f093b5dd7c3924a74b577a60a3b12e1f968/tools/metrics/histograms/histograms.xml

Status: Fixed (was: Assigned)
Labels: UMAstats

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

Status: Archived (was: Fixed)

Sign in to add a comment