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

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: Handle a null in unit tests 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..cf24423d72ecac27519d753681dd257114456d8e 100644
--- a/chrome/browser/ui/cocoa/profiles/profile_chooser_controller.mm
+++ b/chrome/browser/ui/cocoa/profiles/profile_chooser_controller.mm
@@ -1179,7 +1179,8 @@ class ActiveProfileObserverBridge : public AvatarMenuObserver,
}
}
currentProfileView = [self createCurrentProfileView:item];
- displayLock = switches::IsNewProfileManagement() && item.signed_in;
+ displayLock = switches::IsNewProfileManagement() && item.signed_in &&
+ profiles::IsLockAvailable(browser_->profile());
msw 2014/09/19 21:47:40 This seems to hide the lock button, but your CL de
Mike Lerman 2014/09/21 00:19:36 CL updated - thanks.
} else {
[otherProfiles addObject:[self createOtherProfileView:i]];
}

Powered by Google App Engine
This is Rietveld 408576698