Chromium Code Reviews| Index: chrome/browser/profiles/avatar_menu.cc |
| diff --git a/chrome/browser/profiles/avatar_menu.cc b/chrome/browser/profiles/avatar_menu.cc |
| index f3db24a0f926732401414ec927d612e16df7b7c6..8ba3d15a51e3bc4be5bc5aa669e1689f157c29bd 100644 |
| --- a/chrome/browser/profiles/avatar_menu.cc |
| +++ b/chrome/browser/profiles/avatar_menu.cc |
| @@ -115,7 +115,8 @@ bool AvatarMenu::CompareItems(const Item* item1, const Item* item2) { |
| base::i18n::ToLower(item2->name)) < 0; |
| } |
| -void AvatarMenu::SwitchToProfile(size_t index, bool always_create) { |
| +void AvatarMenu::SwitchToProfile( |
| + size_t index, bool always_create, ProfileMetrics::ProfileOpen metric) { |
|
Alexei Svitkine (slow)
2014/01/28 16:25:31
If you wrap params, it should be 1 per line.
bcwhite
2014/01/28 17:03:03
Done.
|
| DCHECK(profiles::IsMultipleProfilesEnabled() || |
| index == GetActiveProfileIndex()); |
| const Item& item = GetItemAt(index); |
| @@ -136,8 +137,8 @@ void AvatarMenu::SwitchToProfile(size_t index, bool always_create) { |
| desktop_type = browser_->host_desktop_type(); |
| profiles::SwitchToProfile(path, desktop_type, always_create, |
| - profiles::ProfileSwitchingDoneCallback()); |
| - ProfileMetrics::LogProfileSwitchUser(ProfileMetrics::SWITCH_PROFILE_ICON); |
| + profiles::ProfileSwitchingDoneCallback(), |
| + metric); |
| } |
| void AvatarMenu::AddNewProfile(ProfileMetrics::ProfileAdd type) { |