diff options
| author | msw@chromium.org <msw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-05 07:10:38 +0000 |
|---|---|---|
| committer | msw@chromium.org <msw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-05 07:10:38 +0000 |
| commit | 88f11549025ad46da215d208a016387619d7271d (patch) | |
| tree | 234a768a0e7131505baf45ff1d217381acb22678 /chrome/browser/first_run | |
| parent | d09ef258f1ffc7f67810359cca9f40badce37466 (diff) | |
| download | chromium_src-88f11549025ad46da215d208a016387619d7271d.zip chromium_src-88f11549025ad46da215d208a016387619d7271d.tar.gz chromium_src-88f11549025ad46da215d208a016387619d7271d.tar.bz2 | |
Add FirstRun.SearchEngineBubble UMA histogram counts.
Add an enum for first run bubble histogram counts.
Add first_run::LogFirstRunMetric utility function.
Log metrics for actions of interest (show and 'Change').
This change depends on the src-internal change (4219017).
BUG=121782
TEST=New histogram data available for first run bubble impressions and use.
Review URL: http://codereview.chromium.org/9969141
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130848 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/first_run')
| -rw-r--r-- | chrome/browser/first_run/first_run.cc | 6 | ||||
| -rw-r--r-- | chrome/browser/first_run/first_run.h | 9 |
2 files changed, 15 insertions, 0 deletions
diff --git a/chrome/browser/first_run/first_run.cc b/chrome/browser/first_run/first_run.cc index 2c267c0..1793846 100644 --- a/chrome/browser/first_run/first_run.cc +++ b/chrome/browser/first_run/first_run.cc @@ -7,6 +7,7 @@ #include "base/command_line.h" #include "base/compiler_specific.h" #include "base/file_util.h" +#include "base/metrics/histogram.h" #include "base/path_service.h" #include "base/utf_string_conversions.h" #include "build/build_config.h" @@ -462,6 +463,11 @@ bool SetPersonalDataManagerFirstRunPref() { return true; } +void LogFirstRunMetric(FirstRunBubbleMetric metric) { + UMA_HISTOGRAM_ENUMERATION("FirstRun.SearchEngineBubble", metric, + NUM_FIRST_RUN_BUBBLE_METRICS); +} + // static void FirstRunBubbleLauncher::ShowFirstRunBubbleSoon() { SetShowFirstRunBubblePref(true); diff --git a/chrome/browser/first_run/first_run.h b/chrome/browser/first_run/first_run.h index 29ef02b..be7e830 100644 --- a/chrome/browser/first_run/first_run.h +++ b/chrome/browser/first_run/first_run.h @@ -33,6 +33,12 @@ class ProcessSingleton; // install work for this user. After that the sentinel file is created. namespace first_run { +enum FirstRunBubbleMetric { + FIRST_RUN_BUBBLE_SHOWN = 0, // The search engine bubble was shown. + FIRST_RUN_BUBBLE_CHANGE_INVOKED, // The bubble's "Change" was invoked. + NUM_FIRST_RUN_BUBBLE_METRICS +}; + // See ProcessMasterPreferences for more info about this structure. struct MasterPrefs { MasterPrefs(); @@ -74,6 +80,9 @@ bool SetShowWelcomePagePref(); // Returns false if the pref could not be set. bool SetPersonalDataManagerFirstRunPref(); +// Log a metric for the "FirstRun.SearchEngineBubble" histogram. +void LogFirstRunMetric(FirstRunBubbleMetric metric); + // -- Platform-specific functions -- // Automatically import history and home page (and search engine, if |