[Payment Request] Selector view edit mode

ios/chrome/browser/ui/payments/shipping_option_selection_mediator.mm

Index: ios/chrome/browser/ui/payments/shipping_option_selection_mediator.mm
diff --git a/ios/chrome/browser/ui/payments/shipping_option_selection_mediator.mm b/ios/chrome/browser/ui/payments/shipping_option_selection_mediator.mm
index b7d341f7bfc675d46570bb1bcc10aee205540996..a8d129e1735ed8a91f94c845aac1c7c69cf30eef 100644
--- a/ios/chrome/browser/ui/payments/shipping_option_selection_mediator.mm
+++ b/ios/chrome/browser/ui/payments/shipping_option_selection_mediator.mm
@@ -27,7 +27,10 @@
 @property(nonatomic, assign) PaymentRequest* paymentRequest;
 
 // The selectable items to display in the collection.
-@property(nonatomic, strong) NSArray<PaymentsTextItem*>* items;
+@property(nonatomic, strong) NSMutableArray<PaymentsTextItem*>* items;
+
+// Creates the selectable items to display in the collection.
+- (void)createItems;

[lpromero, 2017/06/14 09:10:13]

Optional nit: "loadItems" might convey a better semantic.

[Moe, 2017/06/14 14:35:05]

Done.

 
 @end
 
@@ -44,13 +47,17 @@
   if (self) {
     _paymentRequest = paymentRequest;
     _selectedItemIndex = NSUIntegerMax;
-    _items = [self createItems];
+    [self createItems];
   }
   return self;
 }
 
 #pragma mark - PaymentRequestSelectorViewControllerDataSource
 
+- (BOOL)allowsEditMode {
+  return NO;
+}
+
 - (CollectionViewItem*)headerItem {
   if (!self.headerText.length)
     return nil;
@@ -72,11 +79,10 @@
 
 #pragma mark - Helper methods
 
-- (NSArray<PaymentsTextItem*>*)createItems {
+- (void)createItems {
   const std::vector<web::PaymentShippingOption*>& shippingOptions =
       _paymentRequest->shipping_options();
-  NSMutableArray<PaymentsTextItem*>* items =
-      [NSMutableArray arrayWithCapacity:shippingOptions.size()];
+  _items = [NSMutableArray arrayWithCapacity:shippingOptions.size()];
   for (size_t index = 0; index < shippingOptions.size(); ++index) {
     web::PaymentShippingOption* shippingOption = shippingOptions[index];
     DCHECK(shippingOption);
@@ -89,9 +95,8 @@
     if (_paymentRequest->selected_shipping_option() == shippingOption)
       _selectedItemIndex = index;
 
-    [items addObject:item];
+    [_items addObject:item];
   }
-  return items;
 }
 
 @end