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

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

Issue 2442843002: Override SigninManager::SignOut if force-signin is enabled. (Closed)
Patch Set: Created 4 years, 1 month 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
« no previous file with comments | « components/signin/core/browser/signin_manager.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 3880fc1f6fb4b0efd40713527a77ab2563af189d..fc3a0c503a53c9c0c5ce14b476132140f98c477a 100644
--- a/components/signin/core/browser/signin_manager.cc
+++ b/components/signin/core/browser/signin_manager.cc
@@ -125,6 +125,7 @@ void SigninManager::CopyCredentialsFrom(const SigninManager& source) {
possibly_invalid_email_ = source.possibly_invalid_email_;
temp_refresh_token_ = source.temp_refresh_token_;
password_ = source.password_;
+ source.client_->AfterCredentialsCopied();
}
void SigninManager::ClearTransientSigninData() {
@@ -148,6 +149,14 @@ void SigninManager::HandleAuthError(const GoogleServiceAuthError& error) {
void SigninManager::SignOut(
signin_metrics::ProfileSignout signout_source_metric,
signin_metrics::SignoutDelete signout_delete_metric) {
+ client_->PreSignOut(base::Bind(&SigninManager::DoSignOut,
+ base::Unretained(this), signout_source_metric,
+ signout_delete_metric));
+}
+
+void SigninManager::DoSignOut(
+ signin_metrics::ProfileSignout signout_source_metric,
+ signin_metrics::SignoutDelete signout_delete_metric) {
DCHECK(IsInitialized());
signin_metrics::LogSignout(signout_source_metric, signout_delete_metric);
« no previous file with comments | « components/signin/core/browser/signin_manager.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698