Commit bbea865b authored by gogerald's avatar gogerald Committed by Commit bot

Don't record re-auths, unlocks, or secondary account additions in Signin.SigninStartedAccessPoint

BUG=632052

Review-Url: https://codereview.chromium.org/2384233002
Cr-Commit-Position: refs/heads/master@{#422969}
parent 8ccfe9a0
...@@ -195,15 +195,20 @@ void InlineLoginHandler::ContinueHandleInitializeMessage() { ...@@ -195,15 +195,20 @@ void InlineLoginHandler::ContinueHandleInitializeMessage() {
const GURL& current_url = web_ui()->GetWebContents()->GetURL(); const GURL& current_url = web_ui()->GetWebContents()->GetURL();
signin_metrics::AccessPoint access_point = signin_metrics::AccessPoint access_point =
signin::GetAccessPointForPromoURL(current_url); signin::GetAccessPointForPromoURL(current_url);
signin_metrics::LogSigninAccessPointStarted(access_point); signin_metrics::Reason reason =
signin::GetSigninReasonForPromoURL(current_url);
if (reason != signin_metrics::Reason::REASON_REAUTHENTICATION ||
reason != signin_metrics::Reason::REASON_UNLOCK ||
reason != signin_metrics::Reason::REASON_ADD_SECONDARY_ACCOUNT) {
signin_metrics::LogSigninAccessPointStarted(access_point);
}
RecordSigninUserActionForAccessPoint(access_point); RecordSigninUserActionForAccessPoint(access_point);
content::RecordAction(base::UserMetricsAction("Signin_SigninPage_Loading")); content::RecordAction(base::UserMetricsAction("Signin_SigninPage_Loading"));
params.SetString("continueUrl", signin::GetLandingURL(access_point).spec()); params.SetString("continueUrl", signin::GetLandingURL(access_point).spec());
Profile* profile = Profile::FromWebUI(web_ui()); Profile* profile = Profile::FromWebUI(web_ui());
signin_metrics::Reason reason =
signin::GetSigninReasonForPromoURL(current_url);
std::string default_email; std::string default_email;
if (reason == signin_metrics::Reason::REASON_SIGNIN_PRIMARY_ACCOUNT) { if (reason == signin_metrics::Reason::REASON_SIGNIN_PRIMARY_ACCOUNT) {
default_email = default_email =
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment