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

Unified Diff: chrome/browser/ui/cocoa/profiles/profile_chooser_controller.mm

Issue 566933005: Do not display lock for hosted domains (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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/profiles/profile_chooser_controller.mm
diff --git a/chrome/browser/ui/cocoa/profiles/profile_chooser_controller.mm b/chrome/browser/ui/cocoa/profiles/profile_chooser_controller.mm
index b90c224829c609200234f69ba377c858882eb9f7..75f92689bfd7db92941054224768984bff9916be 100644
--- a/chrome/browser/ui/cocoa/profiles/profile_chooser_controller.mm
+++ b/chrome/browser/ui/cocoa/profiles/profile_chooser_controller.mm
@@ -1722,6 +1722,11 @@ class ActiveProfileObserverBridge : public AvatarMenuObserver,
IDS_PROFILES_PROFILE_SIGNOUT_BUTTON)
imageResourceId:IDR_ICON_PROFILES_MENU_LOCK
action:@selector(lockProfile:)];
+
+ if (!browser_->profile()->GetPrefs()->GetBoolean(
+ prefs::kProfileIsLockable)) {
noms (inactive) 2014/09/12 19:33:26 I don't think we should display the button at all
Mike Lerman 2014/09/15 14:08:56 Chatted on one bug or another (I can find it if yo
+ [lockButton setEnabled:NO];
+ }
[container addSubview:lockButton];
viewRect.origin.y = NSMaxY([lockButton frame]);

Powered by Google App Engine
This is Rietveld 408576698