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

Unified Diff: ios/chrome/browser/payments/shipping_option_selection_mediator.mm

Issue 2826713002: [Payment Request] Refactors the remaining selector view controllers (Closed)
Patch Set: Created 3 years, 8 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/payments/shipping_option_selection_mediator.mm
diff --git a/ios/chrome/browser/payments/shipping_option_selection_mediator.mm b/ios/chrome/browser/payments/shipping_option_selection_mediator.mm
new file mode 100644
index 0000000000000000000000000000000000000000..a4dc325081330f26e91e4c016e66bd722481ba3b
--- /dev/null
+++ b/ios/chrome/browser/payments/shipping_option_selection_mediator.mm
@@ -0,0 +1,102 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include <vector>
+
+#import "ios/chrome/browser/payments/shipping_option_selection_mediator.h"
+
+#include "base/strings/sys_string_conversions.h"
+#include "base/strings/utf_string_conversions.h"
+#include "components/payments/core/currency_formatter.h"
+#import "ios/chrome/browser/payments/cells/payments_text_item.h"
+#include "ios/chrome/browser/payments/payment_request.h"
+#include "ios/chrome/browser/ui/uikit_ui_util.h"
+#include "ios/chrome/grit/ios_theme_resources.h"
+#include "ios/web/public/payments/payment_request.h"
+
+#if !defined(__has_feature) || !__has_feature(objc_arc)
+#error "This file requires ARC support."
+#endif
+
+@interface ShippingOptionSelectionMediator ()
+
+// The PaymentRequest object owning an instance of web::PaymentRequest as
+// provided by the page invoking the Payment Request API. This is a weak
+// pointer and should outlive this class.
+@property(nonatomic, assign) PaymentRequest* paymentRequest;
+
+// The selectable items to display in the collection.
+@property(nonatomic, strong) NSArray<PaymentsTextItem*>* items;
+
+@end
+
+@implementation ShippingOptionSelectionMediator
+
+@synthesize state = _state;
+@synthesize selectedItemIndex = _selectedItemIndex;
+@synthesize paymentRequest = _paymentRequest;
+@synthesize items = _items;
+@synthesize headerText = _headerText;
macourteau 2017/04/19 18:03:13 nit: Should this list match order of declaration i
Moe 2017/04/19 20:12:44 Yes! Done.
+
+- (instancetype)initWithPaymentRequest:(PaymentRequest*)paymentRequest {
+ self = [super init];
+ if (self) {
+ _paymentRequest = paymentRequest;
+ _selectedItemIndex = NSUIntegerMax;
+ _items = [self createItems];
+ }
+ return self;
+}
+
+#pragma mark - PaymentRequestSelectorViewControllerDataSource
+
+- (CollectionViewItem*)headerItem {
+ if (!self.headerText.length)
+ return nil;
+
+ PaymentsTextItem* headerItem = [[PaymentsTextItem alloc] init];
+ headerItem.text = self.headerText;
+ if (self.state == PaymentRequestSelectorStateError)
+ headerItem.image = NativeImage(IDR_IOS_PAYMENTS_WARNING);
+ return headerItem;
+}
+
+- (NSArray<CollectionViewItem*>*)selectableItems {
+ return self.items;
+}
+
+- (CollectionViewItem*)selectableItemAtIndex:(NSUInteger)index {
+ DCHECK(index < self.items.count);
+ return [self.items objectAtIndex:index];
+}
+
+- (CollectionViewItem*)addButtonItem {
+ return nil;
+}
+
+#pragma mark - Helper methods
+
+- (NSArray<PaymentsTextItem*>*)createItems {
+ const std::vector<web::PaymentShippingOption*>& shippingOptions =
+ _paymentRequest->shipping_options();
+ NSMutableArray<PaymentsTextItem*>* items =
+ [NSMutableArray arrayWithCapacity:shippingOptions.size()];
+ for (size_t index = 0; index < shippingOptions.size(); ++index) {
+ web::PaymentShippingOption* shippingOption = shippingOptions[index];
+ DCHECK(shippingOption);
+ PaymentsTextItem* item = [[PaymentsTextItem alloc] init];
+ item.text = base::SysUTF16ToNSString(shippingOption->label);
+ payments::CurrencyFormatter* currencyFormatter =
+ _paymentRequest->GetOrCreateCurrencyFormatter();
+ item.detailText = SysUTF16ToNSString(currencyFormatter->Format(
+ base::UTF16ToASCII(shippingOption->amount.value)));
+ if (_paymentRequest->selected_shipping_option() == shippingOption)
+ _selectedItemIndex = index;
+
+ [items addObject:item];
+ }
+ return items;
+}
+
+@end

Powered by Google App Engine
This is Rietveld 408576698