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

Unified Diff: chrome/browser/ui/views/avatar_menu_bubble_view.cc

Issue 132453004: Add Profile metric counting to new User Manager. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: added new metric information to xml file Created 6 years, 11 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/ui/views/avatar_menu_bubble_view.cc
diff --git a/chrome/browser/ui/views/avatar_menu_bubble_view.cc b/chrome/browser/ui/views/avatar_menu_bubble_view.cc
index bb6f7e887a0cd58f7ecbe6fed746133d4b07de16..4d6743c5bc22d8073a2079d485f43c79861c3e97 100644
--- a/chrome/browser/ui/views/avatar_menu_bubble_view.cc
+++ b/chrome/browser/ui/views/avatar_menu_bubble_view.cc
@@ -668,7 +668,9 @@ void AvatarMenuBubbleView::ButtonPressed(views::Button* sender,
// Clicking on the active profile shouldn't do anything.
if (!item_view->item().active) {
avatar_menu_->SwitchToProfile(
- i, ui::DispositionFromEventFlags(event.flags()) == NEW_WINDOW);
+ i,
+ ui::DispositionFromEventFlags(event.flags()) == NEW_WINDOW,
msw 2014/01/29 17:51:58 optional nit: you could wrap this onto the line ab
bcwhite 2014/01/29 19:50:52 Done.
+ ProfileMetrics::SWITCH_PROFILE_ICON);
}
break;
}

Powered by Google App Engine
This is Rietveld 408576698