[Payment Request] Selector view controller

ios/chrome/browser/payments/cells/payments_text_item.h

 // 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.
 
 #ifndef IOS_CHROME_BROWSER_PAYMENTS_CELLS_PAYMENTS_TEXT_ITEM_H_
 #define IOS_CHROME_BROWSER_PAYMENTS_CELLS_PAYMENTS_TEXT_ITEM_H_
 
 #import <UIKit/UIKit.h>
 
 #import "ios/chrome/browser/ui/collection_view/cells/collection_view_item.h"
 #import "ios/third_party/material_components_ios/src/components/CollectionCells/src/MaterialCollectionCells.h"
 
 // PaymentsTextItem is the model class corresponding to PaymentsTextCell.
 @interface PaymentsTextItem : CollectionViewItem
 
 // The message to display.
-@property(nonatomic, copy) NSString* text;
+@property(nonatomic, nullable, copy) NSString* text;
 
 // The image to display.
-@property(nonatomic, strong) UIImage* image;
+@property(nonatomic, nullable, strong) UIImage* image;
+
+// The font of the text. Default is the regular Roboto font of size 14.
+@property(nonatomic, null_resettable, copy) UIFont* textFont;

[lpromero, 2017/04/11 15:59:06]

This goes on the "styling" idea territory. Is this just for the support of
header vs normal cell?
If so, I would create two items and two cells. Or just have an enum with the
style. That way the item keeps being descriptive rather than imperative. The
cell can then choose the font and color based on the style enum.

[Moe, 2017/04/14 06:05:49]

Removed the style properties from the item and moved the defaults back to the
cell, having the CVC override them when necessary. similar to:
https://codereview.chromium.org/2817953002/

+
+// The color of the text. Default is the 900 tint color of the grey palette.
+@property(nonatomic, null_resettable, copy) UIColor* textColor;
 
 @end
 
 // PaymentsTextCell implements a MDCCollectionViewCell subclass containing
 // a text label and an optional image. The label is laid out to fill the full
 // width of the cell and is wrapped as needed to fit in the cell. The image is
 // laid out on the leading edge of the cell. The text label is laid out on the
 // the trailing edge of the image, if one exists, or the leading edge of the
 // cell otherwise, up to the trailing edge of the cell. This is suitable for
 // displaying text that needs to wrap which may or may not feature an image.
 @interface PaymentsTextCell : MDCCollectionViewCell
 
 // UILabel corresponding to |text| from the item.
-@property(nonatomic, readonly, strong) UILabel* textLabel;
+@property(nonatomic, readonly, nullable, strong) UILabel* textLabel;
 
 // UIImageView corresponding to |image| from the item.
-@property(nonatomic, readonly, strong) UIImageView* imageView;
+@property(nonatomic, readonly, nullable, strong) UIImageView* imageView;
 
 @end
 
 #endif  // IOS_CHROME_BROWSER_PAYMENTS_CELLS_PAYMENTS_TEXT_ITEM_H_