Interactive autofill: Handle Online Wallet being unavailable.

chrome/browser/ui/autofill/autofill_dialog_controller_impl.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_IMPL_H_
 #define CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_DIALOG_CONTROLLER_IMPL_H_
 
 #include <vector>
 
 #include "base/callback.h"
@@ skipping 42 matching lines @@
       const GURL& source_url,
       const content::SSLStatus& ssl_status,
       const base::Callback<void(const FormStructure*)>& callback);
   virtual ~AutofillDialogControllerImpl();
 
   void Show();
   void Hide();
 
   // AutofillDialogController implementation.
   virtual string16 DialogTitle() const OVERRIDE;
+  virtual string16 PayWithoutWallet() const OVERRIDE;
   virtual string16 EditSuggestionText() const OVERRIDE;
   virtual string16 UseBillingForShippingText() const OVERRIDE;
   virtual string16 WalletOptionText() const OVERRIDE;
   virtual string16 CancelButtonText() const OVERRIDE;
   virtual string16 ConfirmButtonText() const OVERRIDE;
   virtual string16 SignInText() const OVERRIDE;
+  virtual string16 SignedInUser() const OVERRIDE;
   virtual string16 SaveLocallyText() const OVERRIDE;
   virtual string16 CancelSignInText() const OVERRIDE;
   virtual string16 ProgressBarText() const OVERRIDE;
   virtual DialogSignedInState SignedInState() const OVERRIDE;
+  virtual bool IsWalletAvailable() const OVERRIDE;
   virtual const DetailInputs& RequestedFieldsForSection(DialogSection section)
       const OVERRIDE;
   virtual ui::ComboboxModel* ComboboxModelForAutofillType(
       AutofillFieldType type) OVERRIDE;
   virtual ui::MenuModel* MenuModelForSection(DialogSection section) OVERRIDE;
   virtual string16 LabelForSection(DialogSection section) const OVERRIDE;
   virtual string16 SuggestionTextForSection(DialogSection section) OVERRIDE;
   virtual gfx::Image SuggestionIconForSection(DialogSection section) OVERRIDE;
   virtual void EditClickedForSection(DialogSection section) OVERRIDE;
   virtual bool InputIsValid(const DetailInput* input, const string16& value)
@@ skipping 54 matching lines @@
 
   // Updates the progress bar based on the Autocheckout progress. |value| should
   // be in [0.0, 1.0].
   void UpdateProgressBar(double value);
 
  private:
   // Determines whether |input| and |field| match.
   typedef base::Callback<bool(const DetailInput& input,
                               const AutofillField& field)> InputFieldComparator;
 
+  // Refresh wallet items when there's not a request in progress.

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

"Refresh wallet items immediately if there's no request in progress, otherwise
wait until the current request completes."

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

Done.

+  void ScheduleRefreshWalletItems();
+
+  // Called when a request to Online Wallet completes. |success| is true when
+  // there was no network error, the response wasn't malformed, and no Wallet
+  // error occurred.

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

good example of a helpful comment.

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

If only my brain worked the same way at all times of the day! :P

+  void WalletRequestCompleted(bool success);
+
   // Whether or not the current request wants credit info back.
   bool RequestingCreditCardInfo() const;
 
   // Whether the information input in this dialog will be securely transmitted
   // to the requesting site.
   bool TransmissionWillBeSecure() const;
 
   // Convenience method to tell whether we need to address |action|.
   bool HasRequiredAction(wallet::RequiredAction action) const;
 
@@ skipping 64 matching lines @@
 
   // The SSL info from the invoking site.
   content::SSLStatus ssl_status_;
 
   // The callback via which we return the collected data.
   base::Callback<void(const FormStructure*)> callback_;
 
   // A client to talk to the Online Wallet API.
   wallet::WalletClient wallet_client_;
 
+  // Whether there's currently a request for WalletItems in progress.
+  bool refresh_wallet_items_in_progress_;

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

the seemingly interchangeable use of "refresh" and "request" is confusing, can
we just stick with "request"

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

Oh, sorry, these were named request_* before, I thought I updated all of the
comments -- Done.

+
+  // Whether another request for WalletItems should be started when the current
+  // one is done.
+  bool refresh_wallet_items_queued_;
+
+  // Whether there has been a wallet error while this dialog has been open.
+  bool had_wallet_error_;
+
   // The most recently received WalletItems retrieved via |wallet_client_|.
   scoped_ptr<wallet::WalletItems> wallet_items_;
 
   // The fields for billing and shipping which the page has actually requested.
   DetailInputs requested_email_fields_;
   DetailInputs requested_cc_fields_;
   DetailInputs requested_billing_fields_;
   DetailInputs requested_shipping_fields_;
 
   // Models for the credit card expiration inputs.
@@ skipping 25 matching lines @@
 
   // A NotificationRegistrar for tracking the completion of sign-in.
   content::NotificationRegistrar registrar_;
 
   DISALLOW_COPY_AND_ASSIGN(AutofillDialogControllerImpl);
 };
 
 }  // namespace autofill
 
 #endif  // CHROME_BROWSER_UI_AUTOFILL_AUTOFILL_DIALOG_CONTROLLER_IMPL_H_