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

Unified Diff: chrome/browser/ui/cocoa/autofill/autofill_account_chooser.mm

Issue 23806002: [rAc] Disable sign-in link while the user's signed-in state is unknown. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 4 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
« no previous file with comments | « no previous file | chrome/browser/ui/views/autofill/autofill_dialog_views.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/cocoa/autofill/autofill_account_chooser.mm
diff --git a/chrome/browser/ui/cocoa/autofill/autofill_account_chooser.mm b/chrome/browser/ui/cocoa/autofill/autofill_account_chooser.mm
index 782beba08ab63278090618a36d9d7125f52c5664..e4e8ef4ef216643b1c04b8f9528cb2f8e00f7cb1 100644
--- a/chrome/browser/ui/cocoa/autofill/autofill_account_chooser.mm
+++ b/chrome/browser/ui/cocoa/autofill/autofill_account_chooser.mm
@@ -107,6 +107,8 @@ void AddMenuItem(NSMenu *menu, id target, SEL selector, NSString* title,
NSString* linkTitle =
base::SysUTF16ToNSString(delegate_->SignInLinkText());
[link_ setTitle:linkTitle];
+ [link_
+ setEnabled:(delegate_->SignedInState() != autofill::REQUIRES_RESPONSE)];
groby-ooo-7-16 2013/08/30 01:48:12 nit: line break after setEnabled: i.e. [link_ set
Ilya Sherman 2013/08/30 03:24:40 Done.
// populate menu
NSMenu* accountMenu = [popup_ attachedMenu];
« no previous file with comments | « no previous file | chrome/browser/ui/views/autofill/autofill_dialog_views.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698