Chromium Code Reviews| Index: chrome/browser/ui/webui/signin/inline_login_handler.cc |
| diff --git a/chrome/browser/ui/webui/signin/inline_login_handler.cc b/chrome/browser/ui/webui/signin/inline_login_handler.cc |
| index f02145620f8654db225432864a203b6b2bfb16e4..5efdaf352e388752d9eddb4caba0be9b6492a108 100644 |
| --- a/chrome/browser/ui/webui/signin/inline_login_handler.cc |
| +++ b/chrome/browser/ui/webui/signin/inline_login_handler.cc |
| @@ -69,17 +69,17 @@ void InlineLoginHandler::ContinueHandleInitializeMessage() { |
| params.SetInteger("authMode", InlineLoginHandler::kDesktopAuthMode); |
| const GURL& current_url = web_ui()->GetWebContents()->GetURL(); |
| - signin_metrics::Source source = signin::GetSourceForPromoURL(current_url); |
| + signin_metrics::AccessPoint access_point = |
| + signin::GetAccessPointForPromoURL(current_url); |
| + signin_metrics::LogSigninAccessPoint(access_point, false); |
|
Roger Tawa OOO till Jul 10th
2015/12/02 20:33:48
Instead of a bool second arg, would be better to h
gogerald1
2015/12/03 17:49:03
Done.
|
| - params.SetString( |
| - "continueUrl", |
| - signin::GetLandingURL(signin::kSignInPromoQueryKeySource, |
| - static_cast<int>(source)).spec()); |
| + params.SetString("continueUrl", signin::GetLandingURL(access_point).spec()); |
| Profile* profile = Profile::FromWebUI(web_ui()); |
| + signin_metrics::Reason reason = |
| + signin::GetSigninReasonForPromoURL(current_url); |
| std::string default_email; |
| - if (source != signin_metrics::SOURCE_AVATAR_BUBBLE_ADD_ACCOUNT && |
| - source != signin_metrics::SOURCE_REAUTH) { |
| + if (reason == signin_metrics::REASON_SIGNIN_PRIMARY_ACCOUNT) { |
| default_email = |
| profile->GetPrefs()->GetString(prefs::kGoogleServicesLastUsername); |
| } else { |