Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright 2017 The Chromium Authors. All rights reserved. | 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 | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #import "ios/chrome/browser/ui/authentication/signin_promo_view_mediator.h" | 5 #import "ios/chrome/browser/ui/authentication/signin_promo_view_mediator.h" |
| 6 | 6 |
| 7 #include "base/memory/ptr_util.h" | 7 #include "base/memory/ptr_util.h" |
| 8 #include "base/metrics/user_metrics.h" | 8 #include "base/metrics/user_metrics.h" |
| 9 #include "base/metrics/user_metrics_action.h" | 9 #include "base/metrics/user_metrics_action.h" |
| 10 #include "base/strings/sys_string_conversions.h" | 10 #include "base/strings/sys_string_conversions.h" |
| (...skipping 188 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 199 promoAction:signin_metrics::PromoAction::PROMO_ACTION_NEW_ACCOUNT]; | 199 promoAction:signin_metrics::PromoAction::PROMO_ACTION_NEW_ACCOUNT]; |
| 200 [signinPromoView chromeExecuteCommand:command]; | 200 [signinPromoView chromeExecuteCommand:command]; |
| 201 } | 201 } |
| 202 | 202 |
| 203 - (void)signinPromoViewSigninWithDefaultAccount: | 203 - (void)signinPromoViewSigninWithDefaultAccount: |
| 204 (SigninPromoView*)signinPromoView { | 204 (SigninPromoView*)signinPromoView { |
| 205 DCHECK(_defaultIdentity); | 205 DCHECK(_defaultIdentity); |
| 206 RecordSigninUserActionForAccessPoint(_accessPoint); | 206 RecordSigninUserActionForAccessPoint(_accessPoint); |
| 207 RecordSigninDefaultUserActionForAccessPoint(_accessPoint); | 207 RecordSigninDefaultUserActionForAccessPoint(_accessPoint); |
| 208 ShowSigninCommand* command = [[ShowSigninCommand alloc] | 208 ShowSigninCommand* command = [[ShowSigninCommand alloc] |
| 209 initWithOperation:AUTHENTICATION_OPERATION_SIGNIN_PROMO_CONTINUE_AS | 209 initWithOperation:AUTHENTICATION_OPERATION_SIGNIN_PROMO_CONTINUE_AS |
|
msarda
2017/05/16 13:33:53
Use AUTHENTICATION_OPERATION_SIGNIN.
jlebel
2017/05/16 17:55:08
Done.
| |
| 210 identity:_defaultIdentity | |
| 210 accessPoint:_accessPoint | 211 accessPoint:_accessPoint |
| 211 promoAction:signin_metrics::PromoAction::PROMO_ACTION_WITH_DEFAULT]; | 212 promoAction:signin_metrics::PromoAction::PROMO_ACTION_WITH_DEFAULT |
| 213 callback:nil]; | |
|
msarda
2017/05/16 13:33:53
Remove "callback" (it is nil by default).
jlebel
2017/05/16 17:55:08
-[ShowSigninCommand initWithOperation:identity:acc
| |
| 212 [signinPromoView chromeExecuteCommand:command]; | 214 [signinPromoView chromeExecuteCommand:command]; |
| 213 } | 215 } |
| 214 | 216 |
| 215 - (void)signinPromoViewSigninWithOtherAccount: | 217 - (void)signinPromoViewSigninWithOtherAccount: |
| 216 (SigninPromoView*)signinPromoView { | 218 (SigninPromoView*)signinPromoView { |
| 217 DCHECK(_defaultIdentity); | 219 DCHECK(_defaultIdentity); |
| 218 RecordSigninNotDefaultUserActionForAccessPoint(_accessPoint); | 220 RecordSigninNotDefaultUserActionForAccessPoint(_accessPoint); |
| 219 RecordSigninUserActionForAccessPoint(_accessPoint); | 221 RecordSigninUserActionForAccessPoint(_accessPoint); |
| 220 ShowSigninCommand* command = [[ShowSigninCommand alloc] | 222 ShowSigninCommand* command = [[ShowSigninCommand alloc] |
| 221 initWithOperation:AUTHENTICATION_OPERATION_SIGNIN | 223 initWithOperation:AUTHENTICATION_OPERATION_SIGNIN |
| 222 accessPoint:_accessPoint | 224 accessPoint:_accessPoint |
| 223 promoAction:signin_metrics::PromoAction::PROMO_ACTION_NOT_DEFAULT]; | 225 promoAction:signin_metrics::PromoAction::PROMO_ACTION_NOT_DEFAULT]; |
| 224 [signinPromoView chromeExecuteCommand:command]; | 226 [signinPromoView chromeExecuteCommand:command]; |
| 225 } | 227 } |
| 226 | 228 |
| 227 @end | 229 @end |
| OLD | NEW |