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

Unified Diff: chrome/browser/ui/autofill/autofill_dialog_controller_impl.h

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: Rebase Created 7 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
« no previous file with comments | « no previous file | chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/autofill/autofill_dialog_controller_impl.h
diff --git a/chrome/browser/ui/autofill/autofill_dialog_controller_impl.h b/chrome/browser/ui/autofill/autofill_dialog_controller_impl.h
index 43e8eba330beba77ac5751f807554627f7bc80b6..a8bdf29cda628cc3d43ce3d5fd5ee07fea94c514 100644
--- a/chrome/browser/ui/autofill/autofill_dialog_controller_impl.h
+++ b/chrome/browser/ui/autofill/autofill_dialog_controller_impl.h
@@ -109,7 +109,7 @@ class AutofillDialogControllerImpl : public AutofillDialogViewDelegate,
virtual string16 SaveLocallyText() const OVERRIDE;
virtual string16 SaveLocallyTooltip() const OVERRIDE;
virtual string16 LegalDocumentsText() OVERRIDE;
- virtual DialogSignedInState SignedInState() const OVERRIDE;
+ virtual bool ShouldDisableSignInLink() const OVERRIDE;
virtual bool ShouldShowSpinner() const OVERRIDE;
virtual bool ShouldOfferToSaveInChrome() const OVERRIDE;
virtual ui::MenuModel* MenuModelForAccountChooser() OVERRIDE;
@@ -283,6 +283,14 @@ class AutofillDialogControllerImpl : public AutofillDialogViewDelegate,
scoped_ptr<AutofillProfile> billing_profile);
private:
+ enum DialogSignedInState {
+ REQUIRES_RESPONSE,
+ REQUIRES_SIGN_IN,
+ REQUIRES_PASSIVE_SIGN_IN,
+ SIGNED_IN,
+ SIGN_IN_DISABLED,
+ };
+
// Whether or not the current request wants credit info back.
bool RequestingCreditCardInfo() const;
@@ -298,6 +306,9 @@ class AutofillDialogControllerImpl : public AutofillDialogViewDelegate,
// Stop showing sign in flow.
void HideSignIn();
+ // Whether the user is known to be signed in.
+ DialogSignedInState SignedInState() const;
+
// Handles the SignedInState() on Wallet or sign-in state update.
// Triggers the user name fetch and passive sign-in.
void SignedInStateUpdated();
« no previous file with comments | « no previous file | chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698