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

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: Cleaning up 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 8f3896917bf10ee866f383523c2622d5ab36a918..6f80648ab99681ac19ad5cd6bea0867f8d58d645 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&)>();
}
}
@@ -1714,8 +1716,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