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

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

Issue 2826713002: [Payment Request] Refactors the remaining selector view controllers (Closed)
Patch Set: Addressed comment 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/payment_request_selector_view_controller.mm
diff --git a/ios/chrome/browser/payments/payment_request_selector_view_controller.mm b/ios/chrome/browser/payments/payment_request_selector_view_controller.mm
index d0e3b18a88ec0904c3d27032ec1958748c2f3725..7017fdce75169db578b3251fa84acb7f0a694a7b 100644
--- a/ios/chrome/browser/payments/payment_request_selector_view_controller.mm
+++ b/ios/chrome/browser/payments/payment_request_selector_view_controller.mm
@@ -151,6 +151,14 @@ typedef NS_ENUM(NSInteger, ItemType) {
}
break;
}
+ case ItemTypeSelectableItem: {
+ if ([cell isKindOfClass:[PaymentsTextCell class]]) {
+ PaymentsTextCell* textCell =
+ base::mac::ObjCCastStrict<PaymentsTextCell>(cell);
+ textCell.detailTextLabel.textColor = [[MDCPalette greyPalette] tint500];
+ }
+ break;
+ }
default:
break;
}
@@ -194,7 +202,12 @@ typedef NS_ENUM(NSInteger, ItemType) {
break;
}
case ItemTypeAddItem: {
- [self.delegate paymentRequestSelectorViewControllerDidSelectAddItem:self];
+ if ([self.delegate
+ respondsToSelector:@selector
+ (paymentRequestSelectorViewControllerDidSelectAddItem:)]) {
+ [self.delegate
+ paymentRequestSelectorViewControllerDidSelectAddItem:self];
+ }
break;
}
default:

Powered by Google App Engine
This is Rietveld 408576698