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

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

Issue 12457033: Implements SendAutocheckoutStatus API calls for stats tracking. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: DCHECK 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.cc
diff --git a/chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc b/chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc
index 2f946c21b1141b8ad994be74000934162c431396..ade16c44e1809ec03ac94c49c857345aa18bbb21 100644
--- a/chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc
+++ b/chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc
@@ -225,7 +225,8 @@ AutofillDialogControllerImpl::AutofillDialogControllerImpl(
const GURL& source_url,
const AutofillMetrics& metric_logger,
DialogType dialog_type,
- const base::Callback<void(const FormStructure*)>& callback)
+ const base::Callback<void(const FormStructure*,
+ const std::string&)>& callback)
: profile_(Profile::FromBrowserContext(contents->GetBrowserContext())),
contents_(contents),
form_structure_(form, std::string()),
@@ -299,7 +300,7 @@ void AutofillDialogControllerImpl::Show() {
&has_sections);
// Fail if the author didn't specify autocomplete types.
if (!has_types) {
- callback_.Run(NULL);
+ callback_.Run(NULL, std::string());
delete this;
return;
}
@@ -1003,8 +1004,9 @@ void AutofillDialogControllerImpl::OnCancel() {
// submitted to start the flow and then cancelled to close the dialog after
// the error.
if (!callback_.is_null()) {
- callback_.Run(NULL);
- callback_ = base::Callback<void(const FormStructure*)>();
+ callback_.Run(NULL, std::string());
+ callback_ = base::Callback<void(const FormStructure*,
+ const std::string&)>();
}
}
@@ -1194,10 +1196,6 @@ void AutofillDialogControllerImpl::OnDidSaveInstrumentAndAddress(
GetFullWallet();
}
-void AutofillDialogControllerImpl::OnDidSendAutocheckoutStatus() {
- NOTIMPLEMENTED();
-}
-
void AutofillDialogControllerImpl::OnDidUpdateAddress(
const std::string& address_id,
const std::vector<wallet::RequiredAction>& required_actions) {
@@ -1714,8 +1712,11 @@ void AutofillDialogControllerImpl::FinishSubmit() {
} else {
FillOutputForSection(SECTION_SHIPPING);
}
- callback_.Run(&form_structure_);
- callback_ = base::Callback<void(const FormStructure*)>();
+ if (wallet_items_)
+ callback_.Run(&form_structure_, wallet_items_->google_transaction_id());
+ else
+ callback_.Run(&form_structure_, std::string());
+ callback_ = base::Callback<void(const FormStructure*, const std::string&)>();
if (dialog_type_ == DIALOG_TYPE_REQUEST_AUTOCOMPLETE) {
// This may delete us.

Powered by Google App Engine
This is Rietveld 408576698