Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(758)

Unified Diff: chrome/browser/ui/autofill/autofill_dialog_controller_impl.h

Issue 12457033: Implements SendAutocheckoutStatus API calls for stats tracking. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixing unit tests Created 7 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 b19910f90c8d8e885239cb42d5aa1d61d99b2029..71c9c34a4a5f4f2e0940a0013afd7c9c42ed95a6 100644
--- a/chrome/browser/ui/autofill/autofill_dialog_controller_impl.h
+++ b/chrome/browser/ui/autofill/autofill_dialog_controller_impl.h
@@ -75,7 +75,8 @@ class AutofillDialogControllerImpl : public AutofillDialogController,
const GURL& source_url,
const AutofillMetrics& metric_logger,
const DialogType dialog_type,
- const base::Callback<void(const FormStructure*)>& callback);
+ const base::Callback<void(const FormStructure*,
+ const std::string&)>& callback);
virtual ~AutofillDialogControllerImpl();
static void RegisterUserPrefs(PrefRegistrySyncable* registry);
@@ -182,7 +183,6 @@ class AutofillDialogControllerImpl : public AutofillDialogController,
const std::string& instrument_id,
const std::string& address_id,
const std::vector<wallet::RequiredAction>& required_actions) OVERRIDE;
- virtual void OnDidSendAutocheckoutStatus() OVERRIDE;
virtual void OnDidUpdateAddress(
const std::string& address_id,
const std::vector<wallet::RequiredAction>& required_actions) OVERRIDE;
@@ -361,8 +361,9 @@ class AutofillDialogControllerImpl : public AutofillDialogController,
// 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_;
+ // The callback via which we return the collected data and, if Online Wallet
+ // was used, the Google transaction id.
+ base::Callback<void(const FormStructure*, const std::string&)> callback_;
// The AccountChooserModel acts as the MenuModel for the account chooser,
// and also tracks which data source the dialog is using.

Powered by Google App Engine
This is Rietveld 408576698