Interactive autofill: Handle Online Wallet being unavailable.

chrome/browser/ui/autofill/autofill_dialog_controller.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_DIALOG_CONTROLLER_H_
 #define CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_DIALOG_CONTROLLER_H_
 
 #include <vector>
 
 #include "base/string16.h"
@@ skipping 20 matching lines @@
 
 namespace autofill {
 
 // This class defines the interface to the controller that the dialog view sees.
 class AutofillDialogController {
  public:
 
   // Strings -------------------------------------------------------------------
 
   virtual string16 DialogTitle() const = 0;
+  virtual string16 PayWithoutWallet() const = 0;
   virtual string16 EditSuggestionText() const = 0;
   virtual string16 UseBillingForShippingText() const = 0;
   virtual string16 WalletOptionText() const = 0;
   virtual string16 CancelButtonText() const = 0;
   virtual string16 ConfirmButtonText() const = 0;
   virtual string16 SignInText() const = 0;
+  virtual string16 SignedInUser() const = 0;

[Evan Stade, 2013/02/06 21:57:49]

notice how these all end in a word that makes it obvious it's a string function?

[Dan Beam, 2013/02/07 01:22:40]

I do now! :)

   virtual string16 CancelSignInText() const = 0;
   virtual string16 SaveLocallyText() const = 0;
   virtual string16 ProgressBarText() const = 0;
 
   // State ---------------------------------------------------------------------
 
   // Whether the user is known to be signed in.
   virtual DialogSignedInState SignedInState() const = 0;
 
+  // Whether wallet is available to get information from.

[Evan Stade, 2013/02/06 21:57:49]

this comment doesn't add much value... you should be more specific about what
"available" means. When might it return false?

[Dan Beam, 2013/02/07 01:22:40]

I've added a better comment and renamed to CanPayWithWallet() to more closely
match what the view really wants to know.

+  virtual bool IsWalletAvailable() const = 0;
+
   // Detail inputs -------------------------------------------------------------
 
   // Returns the set of inputs the page has requested which fall under
   // |section|.
   virtual const DetailInputs& RequestedFieldsForSection(DialogSection section)
       const = 0;
 
   // Returns the combobox model for inputs of type |type|, or NULL if the input
   // should be a text field.
   virtual ui::ComboboxModel* ComboboxModelForAutofillType(
@@ skipping 48 matching lines @@
   // The web contents that prompted the dialog.
   virtual content::WebContents* web_contents() = 0;
 
  protected:
   virtual ~AutofillDialogController();
 };
 
 }  // namespace autofill
 
 #endif  // CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_DIALOG_CONTROLLER_H_