UMA for New Profile Management. Track User-Menu Tutorial actions with Histograms.

chrome/browser/ui/views/profiles/profile_chooser_view.cc

Index: chrome/browser/ui/views/profiles/profile_chooser_view.cc
diff --git a/chrome/browser/ui/views/profiles/profile_chooser_view.cc b/chrome/browser/ui/views/profiles/profile_chooser_view.cc
index 7d6d5490fcbebbd443223e616fed0c4de9c899d8..b23fffbbc217f4abe2e12d2ed6f3086ac77b0173 100644
--- a/chrome/browser/ui/views/profiles/profile_chooser_view.cc
+++ b/chrome/browser/ui/views/profiles/profile_chooser_view.cc
@@ -12,6 +12,7 @@
 #include "chrome/browser/pref_service_flags_storage.h"
 #include "chrome/browser/profiles/profile_avatar_icon_util.h"
 #include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/browser/profiles/profile_metrics.h"
 #include "chrome/browser/profiles/profile_window.h"
 #include "chrome/browser/profiles/profiles_state.h"
 #include "chrome/browser/signin/profile_oauth2_token_service_factory.h"
@@ -527,6 +528,8 @@ void ProfileChooserView::ButtonPressed(views::Button* sender,
     // is indeed shown for the maximum number of times.
     browser_->profile()->GetPrefs()->SetInteger(
         prefs::kProfileAvatarTutorialShown, kProfileAvatarTutorialShowMax + 1);
+
+    ProfileMetrics::LogProfileMirrorEnrollment(ProfileMetrics::PROMO_DISMISS);

[guohui, 2014/04/16 15:47:45]

this dismisses the confirmation tutorial, not the mirror promo tutorial.
Currently mirror promo is not dismiss-able.

[Mike Lerman, 2014/04/24 15:14:16]

Done.

     ShowView(BUBBLE_VIEW_MODE_PROFILE_CHOOSER, avatar_menu_.get());
   } else if (sender == tutorial_enable_new_profile_management_button_) {
     EnableNewProfileManagementPreview();
@@ -584,6 +587,7 @@ void ProfileChooserView::LinkClicked(views::Link* sender, int event_flags) {
     ShowView(BUBBLE_VIEW_MODE_ACCOUNT_MANAGEMENT, avatar_menu_.get());
   } else if (sender == tutorial_learn_more_link_) {
     // TODO(guohui): update |learn_more_url| once it is decided.
+    ProfileMetrics::LogProfileMirrorEnrollment(ProfileMetrics::PROMO_ACCEPT);

[guohui, 2014/04/16 15:47:45]

the link opens the learn more url, it dose not mean user accepts the mirror
promo. Instead you should put the code in EnableNewProfileManagementPreview.

[Mike Lerman, 2014/04/24 15:14:16]

Renamed. Done.

     const GURL lear_more_url("https://support.google.com/chrome/?hl=en#to");
     chrome::NavigateParams params(
         browser_->profile(),
@@ -663,6 +667,9 @@ views::View* ProfileChooserView::CreateProfileChooserView(
   }
 
   if (tutorial_view) {
+    // Be sure not to track the tutorial display on View refresh.
+    if (tutorial_shown == FALSE)
+      ProfileMetrics::LogProfileMirrorEnrollment(ProfileMetrics::PROMO_DISPLAY);

[guohui, 2014/04/16 15:47:45]

we have two different tutorials at the moment, one that prompts the user to try
out mirror (aka mirror promo), and one that notifies the user that mirror has
been enabled successfully, i don't think you want to mix both. You should only
count the mirror promo tutorial created from
CreateNewProfileManagementPreviewView.

[Mike Lerman, 2014/04/24 15:14:16]

Done.

     layout->StartRow(1, 0);
     layout->AddView(tutorial_view);
   }