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

Unified Diff: ios/chrome/browser/ui/payments/payment_request_coordinator.mm

Issue 2959133002: [Payment Request] unit tests for the FullCardRequester (Closed)
Patch Set: Created 3 years, 6 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: ios/chrome/browser/ui/payments/payment_request_coordinator.mm
diff --git a/ios/chrome/browser/ui/payments/payment_request_coordinator.mm b/ios/chrome/browser/ui/payments/payment_request_coordinator.mm
index 87624db2d4d92a353f130e84917ad0539e9cd82a..70ecdcddc94f29db354b6d68f6a805d81d5d405f 100644
--- a/ios/chrome/browser/ui/payments/payment_request_coordinator.mm
+++ b/ios/chrome/browser/ui/payments/payment_request_coordinator.mm
@@ -13,7 +13,6 @@
#include "ios/chrome/browser/application_context.h"
#include "ios/chrome/browser/payments/payment_request.h"
#include "ios/chrome/browser/payments/payment_request_util.h"
-#include "ios/chrome/browser/ui/payments/full_card_requester.h"
#include "ios/chrome/browser/ui/payments/payment_request_mediator.h"
#include "ui/base/l10n/l10n_util.h"
@@ -124,6 +123,8 @@ const NSTimeInterval kUpdatePaymentSummaryItemIntervalSeconds = 10.0;
_fullCardRequester->GetFullCard(card, _autofillManager);
}
+#pragma mark - FullCardRequesterConsumer
jif 2017/06/28 16:30:15 Thanks for adding those, I find them very useful.
+
- (void)fullCardRequestDidSucceedWithCard:(const autofill::CreditCard&)card
verificationCode:
(const base::string16&)verificationCode {
@@ -137,6 +138,8 @@ const NSTimeInterval kUpdatePaymentSummaryItemIntervalSeconds = 10.0;
verificationCode:verificationCode];
}
+#pragma mark - Public methods
+
- (void)updatePaymentDetails:(web::PaymentDetails)paymentDetails {
[_updatePaymentSummaryItemTimer invalidate];

Powered by Google App Engine
This is Rietveld 408576698