Chromium Code Reviews| OLD | NEW |
|---|---|
| (Empty) | |
| 1 // Copyright 2017 The Chromium Authors. All rights reserved. | |
| 2 // Use of this source code is governed by a BSD-style license that can be | |
| 3 // found in the LICENSE file. | |
| 4 | |
| 5 #ifndef IOS_CHROME_BROWSER_PAYMENTS_PAYMENT_REQUEST_ERROR_COORDINATOR_H_ | |
| 6 #define IOS_CHROME_BROWSER_PAYMENTS_PAYMENT_REQUEST_ERROR_COORDINATOR_H_ | |
| 7 | |
| 8 #import <UIKit/UIKit.h> | |
| 9 | |
| 10 #import "ios/chrome/browser/chrome_coordinator.h" | |
| 11 #import "ios/chrome/browser/payments/payment_request_error_view_controller.h" | |
| 12 | |
| 13 @class PaymentRequestErrorCoordinator; | |
| 14 | |
| 15 // Delegate protocol for PaymentRequestErrorCoordinator. | |
| 16 @protocol PaymentRequestErrorCoordinatorDelegate<NSObject> | |
| 17 | |
| 18 // Notifies the delegate that the user has dismissed the error. | |
| 19 - (void)paymentRequestErrorCoordinatorDidDismiss: | |
| 20 (PaymentRequestErrorCoordinator*)coordinator; | |
| 21 | |
| 22 @end | |
| 23 | |
| 24 // Coordinator responsible for creating and presenting the payment request error | |
| 25 // view controller. | |
| 26 @interface PaymentRequestErrorCoordinator | |
| 27 : ChromeCoordinator<PaymentRequestErrorViewControllerDelegate> | |
| 28 | |
| 29 // The callback to be called once the error is dismissed, if any. | |
| 30 @property(nonatomic, copy) void (^callback)(); | |
|
lpromero
2017/02/21 16:51:07
This is leaked as you don't use ARC.
lpromero
2017/02/21 16:51:07
Nit: Use ProceduralBlock.
Moe
2017/02/21 22:56:14
Done. Started using base::mac::ReleaseProperties(s
Moe
2017/02/21 22:56:15
Done.
| |
| 31 | |
| 32 // The delegate to be notified when the user dismisses the error. | |
| 33 @property(nonatomic, weak) id<PaymentRequestErrorCoordinatorDelegate> delegate; | |
|
lpromero
2017/02/21 16:51:07
s/weak/assign under non-ARC.
Moe
2017/02/21 22:56:14
Done.
| |
| 34 | |
| 35 @end | |
| 36 | |
| 37 #endif // IOS_CHROME_BROWSER_PAYMENTS_PAYMENT_REQUEST_ERROR_COORDINATOR_H_ | |
| OLD | NEW |