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

Unified Diff: chrome/browser/ui/cocoa/browser_window_controller.h

Issue 117533002: [Mac] Redesign of the avatar menu button (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . 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/cocoa/browser_window_controller.h
diff --git a/chrome/browser/ui/cocoa/browser_window_controller.h b/chrome/browser/ui/cocoa/browser_window_controller.h
index 28ee5f71ed9bfaef7ae6ef3211b6c68ce4026f6f..eaa997e925928546a8e6dab7a2402f087ca89931 100644
--- a/chrome/browser/ui/cocoa/browser_window_controller.h
+++ b/chrome/browser/ui/cocoa/browser_window_controller.h
@@ -104,8 +104,12 @@ class WebContents;
NSInteger currentZoomStepDelta_;
// The view controller that manages the incognito badge or the multi-profile
- // avatar icon. The view is always in the view hierarchy, but will be hidden
- // unless it's appropriate to show it.
+ // avatar button. Depending on whether the --new-profile-management flag is
+ // used, the multi-profile button can either be the avatar's icon badge or a
+ // button with the profile's name. If the flag is used, the button is always
+ // shown, otherwise the view will always be in the view hierarchy but will
+ // be hidden unless it's appropriate to show it (i.e. if there's more than
+ // one profile).
base::scoped_nsobject<AvatarButtonController> avatarButtonController_;
// Lazily created view which draws the background for the floating set of bars
@@ -264,6 +268,9 @@ class WebContents;
// user's profile avatar.
- (BOOL)shouldShowAvatar;
+// Whether or not to show the new avatar button used by --new-profile-maagement.
+- (BOOL)shouldUseNewAvatarButton;
+
- (BOOL)isBookmarkBarVisible;
// Returns YES if the bookmark bar is currently animating.

Powered by Google App Engine
This is Rietveld 408576698