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

Unified Diff: components/signin/core/browser/signin_manager.cc

Issue 466043002: Track when user signs in the profile. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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 side-by-side diff with in-line comments
Download patch
Index: components/signin/core/browser/signin_manager.cc
diff --git a/components/signin/core/browser/signin_manager.cc b/components/signin/core/browser/signin_manager.cc
index 9c68eb98915ca9b5c5c0d78a990e7c1bd2ac50b0..737a45328b63cba704bbfacfdfd70e36ac047c6d 100644
--- a/components/signin/core/browser/signin_manager.cc
+++ b/components/signin/core/browser/signin_manager.cc
@@ -363,6 +363,9 @@ void SigninManager::OnSignedIn(const std::string& username) {
client_->GoogleSigninSucceeded(GetAuthenticatedUsername(), password_);
+ signin_metrics::LogSigninProfile(client_->IsFirstRun(),
+ client_->GetInstallDate());
+
password_.clear(); // Don't need it anymore.
DisableOneClickSignIn(client_->GetPrefs()); // Don't ever offer again.
}

Powered by Google App Engine
This is Rietveld 408576698