diff options
| author | alekseys@chromium.org <alekseys@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-01 00:38:07 +0000 |
|---|---|---|
| committer | alekseys@chromium.org <alekseys@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-01 00:38:07 +0000 |
| commit | 846db3ba245807d02e4421777cc18b7e2dddbb6f (patch) | |
| tree | 81bd33e8084dc371ce6caea6660038e71b8369ad | |
| parent | 22c1fb44419e6498bc8e0c2637695a57dcd3f824 (diff) | |
| download | chromium_src-846db3ba245807d02e4421777cc18b7e2dddbb6f.zip chromium_src-846db3ba245807d02e4421777cc18b7e2dddbb6f.tar.gz chromium_src-846db3ba245807d02e4421777cc18b7e2dddbb6f.tar.bz2 | |
Add metrics to printing destination account selection.
BUG=139170
Review URL: https://codereview.chromium.org/262523002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@267395 0039d316-1c4b-4281-b951-d872f2087c98
4 files changed, 13 insertions, 1 deletions
diff --git a/chrome/browser/resources/print_preview/metrics.js b/chrome/browser/resources/print_preview/metrics.js index ef81ecb..081cebe 100644 --- a/chrome/browser/resources/print_preview/metrics.js +++ b/chrome/browser/resources/print_preview/metrics.js @@ -50,7 +50,11 @@ cr.define('print_preview', function() { // Used when a user sees a register promo for a cloud print printer. REGISTER_PROMO_SHOWN: 7, // Used when a user selects a register promo for a cloud print printer. - REGISTER_PROMO_SELECTED: 8 + REGISTER_PROMO_SELECTED: 8, + // User changed active account. + ACCOUNT_CHANGED: 9, + // User tried to log into another account. + ADD_ACCOUNT_SELECTED: 10 }; /** diff --git a/chrome/browser/resources/print_preview/search/destination_search.js b/chrome/browser/resources/print_preview/search/destination_search.js index 24deb6d..5fbe06c 100644 --- a/chrome/browser/resources/print_preview/search/destination_search.js +++ b/chrome/browser/resources/print_preview/search/destination_search.js @@ -583,6 +583,8 @@ cr.define('print_preview', function() { if (account) { this.userInfo_.activeUser = account; this.destinationStore_.reloadUserCookieBasedDestinations(); + this.metrics_.incrementDestinationSearchBucket( + print_preview.Metrics.DestinationSearchBucket.ACCOUNT_CHANGED); } else { cr.dispatchSimpleEvent(this, DestinationSearch.EventType.ADD_ACCOUNT); // Set selection back to the active user. @@ -592,6 +594,8 @@ cr.define('print_preview', function() { break; } } + this.metrics_.incrementDestinationSearchBucket( + print_preview.Metrics.DestinationSearchBucket.ADD_ACCOUNT_SELECTED); } }, diff --git a/chrome/browser/ui/webui/print_preview/print_preview_handler.cc b/chrome/browser/ui/webui/print_preview/print_preview_handler.cc index 1250aecf..6534f82 100644 --- a/chrome/browser/ui/webui/print_preview/print_preview_handler.cc +++ b/chrome/browser/ui/webui/print_preview/print_preview_handler.cc @@ -130,6 +130,8 @@ enum PrintDestinationBuckets { CLOUD_DUPLICATE_SELECTED, REGISTER_PROMO_SHOWN, REGISTER_PROMO_SELECTED, + ACCOUNT_CHANGED, + ADD_ACCOUNT_SELECTED, PRINT_DESTINATION_BUCKET_BOUNDARY }; diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml index 1cd9eeb..ffc9b84 100644 --- a/tools/metrics/histograms/histograms.xml +++ b/tools/metrics/histograms/histograms.xml @@ -39432,6 +39432,8 @@ Therefore, the affected-histogram name has to have at least one dot in it. <int value="6" label="CLOUD_DUPLICATE_SELECTED"/> <int value="7" label="REGISTER_PROMO_SHOWN"/> <int value="8" label="REGISTER_PROMO_SELECTED"/> + <int value="9" label="ACCOUNT_CHANGED"/> + <int value="10" label="ADD_ACCOUNT_SELECTED"/> </enum> <enum name="PrintPreviewUserActionType" type="int"> |