Chromium Code Reviews| 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 5fc7301acb220e99466b0a7e81e5616eb344cfc4..6c16fe2c52357f95cba1f268fe6e0e5a2daef2db 100644 |
| --- a/chrome/browser/ui/views/profiles/profile_chooser_view.cc |
| +++ b/chrome/browser/ui/views/profiles/profile_chooser_view.cc |
| @@ -11,6 +11,7 @@ |
| #include "chrome/browser/profiles/profile_avatar_icon_util.h" |
| #include "chrome/browser/profiles/profile_info_cache.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" |
| @@ -529,9 +530,14 @@ 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::ENROLL_CLOSE_WELCOME_CARD); |
| ShowView(BUBBLE_VIEW_MODE_PROFILE_CHOOSER, avatar_menu_.get()); |
| } else if (sender == tutorial_enable_new_profile_management_button_) { |
| profiles::EnableNewProfileManagementPreview(); |
| + ProfileMetrics:: |
| + LogProfileMirrorEnrollment(ProfileMetrics::ENROLL_ACCEPT_MIRROR); |
| } else if (sender == remove_account_and_relaunch_button_) { |
| RemoveAccount(); |
| } else if (sender == account_removal_cancel_button_) { |
| @@ -597,6 +603,8 @@ void ProfileChooserView::LinkClicked(views::Link* sender, int event_flags) { |
| } else { |
| DCHECK(sender == tutorial_learn_more_link_); |
| // TODO(guohui): update |learn_more_url| once it is decided. |
| + ProfileMetrics:: |
| + LogProfileMirrorEnrollment(ProfileMetrics::ENROLL_LAUNCH_LEARN_MORE); |
| const GURL lear_more_url("https://support.google.com/chrome/?hl=en#to"); |
| chrome::NavigateParams params( |
| browser_->profile(), |
| @@ -672,6 +680,12 @@ views::View* ProfileChooserView::CreateProfileChooserView( |
| } |
| if (tutorial_view) { |
| + // Be sure not to track the tutorial display on View refresh, and only count |
| + // the preview-promo view, not the welcome-enabled view. |
| + if (!switches::IsNewProfileManagement()) { |
| + ProfileMetrics:: |
| + LogProfileMirrorEnrollment(ProfileMetrics::ENROLL_SHOW_PREVIEW_PROMO); |
| + } |
|
guohui
2014/04/28 19:26:18
hmm, but you do track the tutorial display on view
Mike Lerman
2014/04/28 20:43:27
Done.
|
| layout->StartRow(1, 0); |
| layout->AddView(tutorial_view); |
| } |
| @@ -762,7 +776,7 @@ views::View* ProfileChooserView::CreateTutorialView( |
| title_label->SetHorizontalAlignment(gfx::ALIGN_LEFT); |
| title_label->SetAutoColorReadabilityEnabled(false); |
| title_label->SetEnabledColor(SK_ColorWHITE); |
| - title_label ->SetFontList(ui::ResourceBundle::GetSharedInstance().GetFontList( |
| + title_label->SetFontList(ui::ResourceBundle::GetSharedInstance().GetFontList( |
| ui::ResourceBundle::MediumFont)); |
| layout->StartRow(1, 0); |
| layout->AddView(title_label); |