Chromium Code Reviews| Index: ios/chrome/browser/ui/commands/show_signin_command.mm |
| diff --git a/ios/chrome/browser/ui/commands/show_signin_command.mm b/ios/chrome/browser/ui/commands/show_signin_command.mm |
| index b71ec1e1b404501df21d8ba36ff6c8ec040c556f..0228d4b29c503b1172dbda8f6d8cd5012b013b01 100644 |
| --- a/ios/chrome/browser/ui/commands/show_signin_command.mm |
| +++ b/ios/chrome/browser/ui/commands/show_signin_command.mm |
| @@ -14,6 +14,7 @@ |
| @implementation ShowSigninCommand |
| @synthesize operation = _operation; |
| +@synthesize identity = _identity; |
| @synthesize accessPoint = _accessPoint; |
| @synthesize promoAction = _promoAction; |
| @synthesize callback = _callback; |
| @@ -24,12 +25,14 @@ |
| } |
| - (instancetype)initWithOperation:(AuthenticationOperation)operation |
| + identity:(ChromeIdentity*)identity |
| accessPoint:(signin_metrics::AccessPoint)accessPoint |
| promoAction:(signin_metrics::PromoAction)promoAction |
| callback: |
| (ShowSigninCommandCompletionCallback)callback { |
| if ((self = [super initWithTag:IDC_SHOW_SIGNIN_IOS])) { |
| _operation = operation; |
|
sdefresne
2017/05/17 08:49:01
Can you add a DCHECK to ensure the invariant that
jlebel
2017/05/17 12:17:37
Done.
|
| + _identity = identity; |
| _accessPoint = accessPoint; |
| _promoAction = promoAction; |
| _callback = [callback copy]; |
| @@ -41,6 +44,7 @@ |
| accessPoint:(signin_metrics::AccessPoint)accessPoint |
| promoAction:(signin_metrics::PromoAction)promoAction { |
| return [self initWithOperation:operation |
| + identity:nil |
| accessPoint:accessPoint |
| promoAction:promoAction |
| callback:nil]; |