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

Side by Side Diff: ios/chrome/browser/ui/authentication/signin_promo_view_mediator.mm

Issue 2880333003: Adding ChromeIdentity in ShowSigninCommand. (Closed)
Patch Set: Created 3 years, 7 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 unified diff | Download patch
OLDNEW
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
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
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698