[Autofill] Componentize AutofillCCInfoBarDelegate.

components/autofill/core/browser/autofill_client.h

 // Copyright 2014 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 COMPONENTS_AUTOFILL_CORE_BROWSER_AUTOFILL_CLIENT_H_
 #define COMPONENTS_AUTOFILL_CORE_BROWSER_AUTOFILL_CLIENT_H_
 
 #include <vector>
 
 #include "base/callback_forward.h"
 #include "base/i18n/rtl.h"
 #include "base/memory/weak_ptr.h"
 #include "base/strings/string16.h"
 
 namespace content {
 class RenderFrameHost;
 }
 
 namespace gfx {
 class Rect;
 class RectF;
 }
 
 class GURL;
 class PrefService;
 
 namespace autofill {
 
+class AutofillDriver;
 class AutofillPopupDelegate;
 class AutofillWebDataService;
 class CreditCard;
 class FormStructure;
 class PersonalDataManager;
 struct FormData;
 struct Suggestion;
 
 // A client interface that needs to be supplied to the Autofill component by the
 // embedder.
@@ skipping 35 matching lines @@
   virtual void HideRequestAutocompleteDialog() = 0;
 
   // Causes the Autofill settings UI to be shown.
   virtual void ShowAutofillSettings() = 0;
 
   // A user has attempted to use a masked card. Prompt them for further
   // information to proceed.
   virtual void ShowUnmaskPrompt() = 0;
 
   // Run |save_card_callback| if the credit card should be imported as personal
-  // data. |metric_logger| can be used to log user actions.
+  // data. |autofill_driver| abstracts call to open url. |metric_logger| can be

[Peter Kasting, 2014/12/29 22:24:30]

The added sentence here is meaningless: what call and what URL?  Instead, this
function's comment should say something about opening an infobar to prompt the
user, and it probably doesn't even need to mention the driver.

[Pritam Nikam, 2014/12/30 13:22:14]

Done.

+  // used to log user actions.
   virtual void ConfirmSaveCreditCard(
+      AutofillDriver* autofill_driver,
       const base::Closure& save_card_callback) = 0;
 
   // Returns true if both the platform and the device support scanning credit
   // cards. Should be called before ScanCreditCard().
   virtual bool HasCreditCardScanFeature() = 0;
 
   // Shows the user interface for scanning a credit card. Invokes the |callback|
   // when a credit card is scanned successfully. Should be called only if
   // HasCreditCardScanFeature() returns true.
   virtual void ScanCreditCard(const CreditCardScanCallback& callback) = 0;
@@ skipping 35 matching lines @@
       const base::string16& autofilled_value,
       const base::string16& profile_full_name) = 0;
 
   // Informs the client that a user gesture has been observed.
   virtual void OnFirstUserGestureObserved() = 0;
 };
 
 }  // namespace autofill
 
 #endif  // COMPONENTS_AUTOFILL_CORE_BROWSER_AUTOFILL_CLIENT_H_