Chromium Code Reviews| Index: chrome/browser/profiles/profile_metrics.h |
| diff --git a/chrome/browser/profiles/profile_metrics.h b/chrome/browser/profiles/profile_metrics.h |
| index 174c203c239affe270a5e0092e43b6a61ca85ab5..87228a4fe649592a9a6f6494d45d97dd87f85238 100644 |
| --- a/chrome/browser/profiles/profile_metrics.h |
| +++ b/chrome/browser/profiles/profile_metrics.h |
| @@ -17,6 +17,10 @@ namespace base { |
| class FilePath; |
| } |
| +namespace signin { |
| +enum GAIA_SERVICE_TYPE; |
| +} |
| + |
| class ProfileMetrics { |
| public: |
| // Enum for counting the ways users were added. |
| @@ -96,6 +100,21 @@ class ProfileMetrics { |
| NUM_PROFILE_ENROLLMENT_METRICS, |
| }; |
| + // Enum for tracking user interactions with the user menu and user manager. |
| + // Interactions initiated from the content area are logged into a different |
| + // histogram from those that were initiated from the avatar button. |
| + // An example of the interaction beginning in the content area is |
| + // clicking "Manage Accounts" within account selection on a Google property. |
| + enum ProfileDesktopMenu { |
| + // User opened the user menu, and clicked lock. |
| + PROFILE_DESKTOP_MENU_LOCK = 0, |
| + // User opened the user menu, and started removing an account. |
|
guohui
2014/05/22 20:20:49
i think its more interesting to log the actual acc
Mike Lerman
2014/05/23 17:15:36
Done.
|
| + PROFILE_DESKTOP_MENU_REMOVE_ACCT, |
| + // User opened the user menu, and started adding an account. |
| + PROFILE_DESKTOP_MENU_ADD_ACCT, |
| + NUM_PROFILE_DESKTOP_MENU_METRICS, |
| + }; |
|
guohui
2014/05/22 20:20:49
its also interesting to log clicking "not you" and
Mike Lerman
2014/05/23 17:15:36
I'm planning to do so in a new CL. Should I add a
|
| + |
| static void UpdateReportedProfilesStatistics(ProfileManager* manager); |
| static void LogNumberOfProfiles(ProfileManager* manager); |
| @@ -108,6 +127,8 @@ class ProfileMetrics { |
| static void LogProfileSyncInfo(ProfileSync metric); |
| static void LogProfileAuthResult(ProfileAuth metric); |
| static void LogProfileUpgradeEnrollment(ProfileUpgradeEnrollment metric); |
| + static void LogProfileDesktopMenu(ProfileDesktopMenu metric, |
| + signin::GAIA_SERVICE_TYPE gaia_service); |
| // These functions should only be called on the UI thread because they hook |
| // into g_browser_process through a helper function. |