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

Unified Diff: chrome/browser/profiles/profile_metrics.h

Issue 288493008: UMA Metrics for the user menu, including vasquette-detection (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Split user-menu data based on GAIA service type Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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.

Powered by Google App Engine
This is Rietveld 408576698