Load and send Wallet Risk params after user has agreed

chrome/browser/ui/autofill/autofill_dialog_controller_impl.h

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 60bec1db7112520d1ad9ff553226d5e8faf173f8..2eb84b63d6c1b0fb909a8acde29b656c4dc3bbee 100644
--- a/chrome/browser/ui/autofill/autofill_dialog_controller_impl.h
+++ b/chrome/browser/ui/autofill/autofill_dialog_controller_impl.h
@@ -265,6 +265,14 @@ class AutofillDialogControllerImpl : public AutofillDialogController,
   // Exposed and virtual for testing.
   virtual bool IsFirstRun() const;
 
+  // Asks risk module to asynchronously load fingerprint data. Data will be
+  // returned via |OnDidLoadRiskFingerprintData()|. Exposed for testing.
+  virtual void LoadRiskFingerprintData();
+
+  // Called when loading of risk fingerprint data is done.

[Ilya Sherman, 2013/05/22 23:53:59]

nit: This comment seems redundant with the one on line 269.  I'd keep these two
methods grouped as they were before.

[Dan Beam, 2013/05/23 00:30:40]

Done.

+  virtual void OnDidLoadRiskFingerprintData(
+      scoped_ptr<risk::Fingerprint> fingerprint);
+
   // Opens the given URL in a new foreground tab.
   virtual void OpenTabWithUrl(const GURL& url);
 
@@ -379,11 +387,6 @@ class AutofillDialogControllerImpl : public AutofillDialogController,
   // Hides |popup_controller_|'s popup view, if it exists.
   void HidePopup();
 
-  // Asks risk module to asynchronously load fingerprint data. Data will be
-  // returned via OnDidLoadRiskFingerprintData.
-  void LoadRiskFingerprintData();
-  void OnDidLoadRiskFingerprintData(scoped_ptr<risk::Fingerprint> fingerprint);
-
   // Whether the user has chosen to enter all new data in |section|. This
   // happens via choosing "Add a new X..." from a section's suggestion menu.
   bool IsManuallyEditingSection(DialogSection section) const;
@@ -417,6 +420,9 @@ class AutofillDialogControllerImpl : public AutofillDialogController,
   // or Online Wallet (|is_submitting_|) and update the view.
   void SetIsSubmitting(bool submitting);
 
+  // Whether the user has accepted all the current legal documents' terms.
+  bool LegalDocumentsAreCurrent() const;

[Ilya Sherman, 2013/05/22 23:53:59]

nit: "AreLegalDocumentsCurrent()"

[Dan Beam, 2013/05/23 00:30:40]

Done.

+
   // Start the submit proccess to interact with Online Wallet (might do various
   // things like accept documents, save details, update details, respond to
   // required actions, etc.).
@@ -437,6 +443,10 @@ class AutofillDialogControllerImpl : public AutofillDialogController,
   // This information is decoded to reveal a fronting (proxy) card.
   void GetFullWallet();
 
+  // Calls |GetFullWallet()| if the required members (|risk_data_|,
+  // |active_instrument_id_|, and |active_address_id_|) are populated.
+  void GetFullWalletIfReady();
+
   // Updates the state of the controller and |view_| based on any required
   // actions returned by Save or Update calls to Wallet.
   void HandleSaveOrUpdateRequiredActions(
@@ -518,6 +528,10 @@ class AutofillDialogControllerImpl : public AutofillDialogController,
   scoped_ptr<wallet::WalletItems> wallet_items_;
   scoped_ptr<wallet::FullWallet> full_wallet_;
 
+  // Local machine signals to pass along on each request to trigger (or
+  // discourage) risk challenges; sent if the user is up to date on legal docs.
+  std::string risk_data_;
+
   // The text to display when the user is accepting new terms of service, etc.
   string16 legal_documents_text_;
   // The ranges within |legal_documents_text_| to linkify.
@@ -574,6 +588,9 @@ class AutofillDialogControllerImpl : public AutofillDialogController,
   // Whether this is the first time this profile has seen the Autofill dialog.
   bool is_first_run_;
 
+  // Whether a user accepted legal documents while this dialog is running.
+  bool has_accepted_legal_documents_;
+
   // True after the user first accepts the dialog and presses "Submit". May
   // continue to be true while processing required actions.
   bool is_submitting_;