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

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

Issue 2249773002: Scan card holder name. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Pass CreditCard, don't alter the constructor. Created 4 years, 4 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/credit_card_scanner_controller.cc
diff --git a/chrome/browser/ui/autofill/credit_card_scanner_controller.cc b/chrome/browser/ui/autofill/credit_card_scanner_controller.cc
index 9bff3b0dab7e058caac922504aa9b4bd4a45cd2c..438cf342ef69d597b9bad06157edb12f97b4f0cb 100644
--- a/chrome/browser/ui/autofill/credit_card_scanner_controller.cc
+++ b/chrome/browser/ui/autofill/credit_card_scanner_controller.cc
@@ -49,12 +49,10 @@ class Controller : public CreditCardScannerViewDelegate,
}
// CreditCardScannerViewDelegate implementation.
- void ScanCompleted(const base::string16& card_number,
- int expiration_month,
- int expiration_year) override {
+ void ScanCompleted(const CreditCard& card) override {
AutofillMetrics::LogScanCreditCardCompleted(
base::TimeTicks::Now() - show_time_, true);
- callback_.Run(card_number, expiration_month, expiration_year);
+ callback_.Run(card);
delete this;
}

Powered by Google App Engine
This is Rietveld 408576698