| Index: chrome/browser/ui/webui/app_launcher_login_handler.cc
|
| diff --git a/chrome/browser/ui/webui/app_launcher_login_handler.cc b/chrome/browser/ui/webui/app_launcher_login_handler.cc
|
| index c5cdb440d2d17b81fe5ff23470fd6358e0a92e53..460ec652ad89fc2640f103a26c6c94513c4270f7 100644
|
| --- a/chrome/browser/ui/webui/app_launcher_login_handler.cc
|
| +++ b/chrome/browser/ui/webui/app_launcher_login_handler.cc
|
| @@ -126,11 +126,11 @@ void AppLauncherLoginHandler::HandleShowSyncLoginUI(
|
| return;
|
|
|
| // The user isn't signed in, show the sign in promo.
|
| - signin_metrics::Source source =
|
| - web_contents->GetURL().spec() == chrome::kChromeUIAppsURL ?
|
| - signin_metrics::SOURCE_APPS_PAGE_LINK :
|
| - signin_metrics::SOURCE_NTP_LINK;
|
| - chrome::ShowBrowserSignin(browser, source);
|
| + signin_metrics::AccessPoint access_point =
|
| + web_contents->GetURL().spec() == chrome::kChromeUIAppsURL
|
| + ? signin_metrics::AccessPoint::ACCESS_POINT_APPS_PAGE_LINK
|
| + : signin_metrics::AccessPoint::ACCESS_POINT_NTP_LINK;
|
| + chrome::ShowBrowserSignin(browser, access_point);
|
| RecordInHistogram(NTP_SIGN_IN_PROMO_CLICKED);
|
| }
|
|
|
| @@ -157,10 +157,15 @@ void AppLauncherLoginHandler::HandleLoginMessageSeen(
|
|
|
| void AppLauncherLoginHandler::HandleShowAdvancedLoginUI(
|
| const base::ListValue* args) {
|
| - Browser* browser =
|
| - chrome::FindBrowserWithWebContents(web_ui()->GetWebContents());
|
| - if (browser)
|
| - chrome::ShowBrowserSignin(browser, signin_metrics::SOURCE_NTP_LINK);
|
| + content::WebContents* web_contents = web_ui()->GetWebContents();
|
| + Browser* browser = chrome::FindBrowserWithWebContents(web_contents);
|
| + if (!browser)
|
| + return;
|
| + signin_metrics::AccessPoint access_point =
|
| + web_contents->GetURL().spec() == chrome::kChromeUIAppsURL
|
| + ? signin_metrics::AccessPoint::ACCESS_POINT_APPS_PAGE_LINK
|
| + : signin_metrics::AccessPoint::ACCESS_POINT_NTP_LINK;
|
| + chrome::ShowBrowserSignin(browser, access_point);
|
| }
|
|
|
| void AppLauncherLoginHandler::UpdateLogin() {
|
|
|