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

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 189 matching lines...) Expand 10 before | Expand all | Expand 10 after
200 promoAction:signin_metrics::PromoAction::PROMO_ACTION_NEW_ACCOUNT]; 200 promoAction:signin_metrics::PromoAction::PROMO_ACTION_NEW_ACCOUNT];
201 [signinPromoView chromeExecuteCommand:command]; 201 [signinPromoView chromeExecuteCommand:command];
202 } 202 }
203 203
204 - (void)signinPromoViewDidTapSigninWithDefaultAccount: 204 - (void)signinPromoViewDidTapSigninWithDefaultAccount:
205 (SigninPromoView*)signinPromoView { 205 (SigninPromoView*)signinPromoView {
206 DCHECK(_defaultIdentity); 206 DCHECK(_defaultIdentity);
207 RecordSigninUserActionForAccessPoint(_accessPoint); 207 RecordSigninUserActionForAccessPoint(_accessPoint);
208 RecordSigninDefaultUserActionForAccessPoint(_accessPoint); 208 RecordSigninDefaultUserActionForAccessPoint(_accessPoint);
209 ShowSigninCommand* command = [[ShowSigninCommand alloc] 209 ShowSigninCommand* command = [[ShowSigninCommand alloc]
210 initWithOperation:AUTHENTICATION_OPERATION_SIGNIN_PROMO_CONTINUE_AS 210 initWithOperation:AUTHENTICATION_OPERATION_SIGNIN
211 identity:_defaultIdentity
211 accessPoint:_accessPoint 212 accessPoint:_accessPoint
212 promoAction:signin_metrics::PromoAction::PROMO_ACTION_WITH_DEFAULT]; 213 promoAction:signin_metrics::PromoAction::PROMO_ACTION_WITH_DEFAULT
214 callback:nil];
213 [signinPromoView chromeExecuteCommand:command]; 215 [signinPromoView chromeExecuteCommand:command];
214 } 216 }
215 217
216 - (void)signinPromoViewDidTapSigninWithOtherAccount: 218 - (void)signinPromoViewDidTapSigninWithOtherAccount:
217 (SigninPromoView*)signinPromoView { 219 (SigninPromoView*)signinPromoView {
218 DCHECK(_defaultIdentity); 220 DCHECK(_defaultIdentity);
219 RecordSigninNotDefaultUserActionForAccessPoint(_accessPoint); 221 RecordSigninNotDefaultUserActionForAccessPoint(_accessPoint);
220 RecordSigninUserActionForAccessPoint(_accessPoint); 222 RecordSigninUserActionForAccessPoint(_accessPoint);
221 ShowSigninCommand* command = [[ShowSigninCommand alloc] 223 ShowSigninCommand* command = [[ShowSigninCommand alloc]
222 initWithOperation:AUTHENTICATION_OPERATION_SIGNIN 224 initWithOperation:AUTHENTICATION_OPERATION_SIGNIN
223 accessPoint:_accessPoint 225 accessPoint:_accessPoint
224 promoAction:signin_metrics::PromoAction::PROMO_ACTION_NOT_DEFAULT]; 226 promoAction:signin_metrics::PromoAction::PROMO_ACTION_NOT_DEFAULT];
225 [signinPromoView chromeExecuteCommand:command]; 227 [signinPromoView chromeExecuteCommand:command];
226 } 228 }
227 229
228 @end 230 @end
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698