Make autofill stop depending on InfoBarService

chrome/browser/autofill/autofill_cc_infobar_delegate.h

Index: chrome/browser/autofill/autofill_cc_infobar_delegate.h
diff --git a/chrome/browser/autofill/autofill_cc_infobar_delegate.h b/chrome/browser/autofill/autofill_cc_infobar_delegate.h
index 8e1e3f5daea3d8b6e18d433c0fb690968d44afd8..67a582d4ff3ad29a16a66d81ffc5b9a080a610dd 100644
--- a/chrome/browser/autofill/autofill_cc_infobar_delegate.h
+++ b/chrome/browser/autofill/autofill_cc_infobar_delegate.h
@@ -6,6 +6,7 @@
 #define CHROME_BROWSER_AUTOFILL_AUTOFILL_CC_INFOBAR_DELEGATE_H_
 
 #include "base/basictypes.h"
+#include "base/callback.h"
 #include "base/gtest_prod_util.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/string16.h"
@@ -24,27 +25,19 @@ struct LoadCommittedDetails;
 // card information gathered from a form submission.
 class AutofillCCInfoBarDelegate : public ConfirmInfoBarDelegate {
  public:
-  // Creates an autofill credit card delegate and adds it to |infobar_service|.
+  // Creates an AutofillCCInfoBarDelegate and adds it to |infobar_service|.
   static void Create(InfoBarService* infobar_service,
-                     const CreditCard* credit_card,
-                     PersonalDataManager* personal_data,
-                     const AutofillMetrics* metric_logger);
+                     const AutofillMetrics* metric_logger,
+                     const base::Closure& import_callback);

[Ilya Sherman, 2013/03/06 09:13:24]

nit: import_callback -> save_card_callback or something like that

[kaiwang, 2013/03/06 18:25:04]

Done.

 
-#if defined(UNIT_TEST)
-  static scoped_ptr<ConfirmInfoBarDelegate> Create(
-      const CreditCard* credit_card,
-      PersonalDataManager* personal_data,
-      const AutofillMetrics* metric_logger) {
-    return scoped_ptr<ConfirmInfoBarDelegate>(new AutofillCCInfoBarDelegate(
-        NULL, credit_card, personal_data, metric_logger));
-  }
-#endif
+  static scoped_ptr<ConfirmInfoBarDelegate> CreateForTesting(
+      const AutofillMetrics* metric_logger,
+      const base::Closure& import_callback);
 
  private:
   AutofillCCInfoBarDelegate(InfoBarService* infobar_service,
-                            const CreditCard* credit_card,
-                            PersonalDataManager* personal_data,
-                            const AutofillMetrics* metric_logger);
+                            const AutofillMetrics* metric_logger,
+                            const base::Closure& import_callback);
   virtual ~AutofillCCInfoBarDelegate();
 
   void LogUserAction(AutofillMetrics::InfoBarMetric user_action);
@@ -62,17 +55,13 @@ class AutofillCCInfoBarDelegate : public ConfirmInfoBarDelegate {
   virtual string16 GetLinkText() const OVERRIDE;
   virtual bool LinkClicked(WindowOpenDisposition disposition) OVERRIDE;
 
-  // The credit card that should be saved if the user accepts the infobar.
-  scoped_ptr<const CreditCard> credit_card_;
-
-  // The personal data manager to which the credit card should be saved.
-  // Weak reference.
-  PersonalDataManager* personal_data_;
-
   // For logging UMA metrics.
   // Weak reference. Owned by the AutofillManager that initiated this infobar.
   const AutofillMetrics* metric_logger_;
 
+  // The callback to do credit card importing.

[Ilya Sherman, 2013/03/06 09:13:24]

nit: "The callback to save the credit card if the user accepts the infobar." 
(In general, prefer "save" to "import" for this.)

[kaiwang, 2013/03/06 18:25:04]

Done.

+  base::Closure import_callback_;
+
   // Did the user ever explicitly accept or dismiss this infobar?
   bool had_user_interaction_;