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

Unified Diff: chrome/browser/signin/signin_manager.h

Issue 13249007: revoke unused OAuth2 tokens on signout (Closed) Base URL: /home/courage/git/chromium-src.git/@222774-invalid-grant-relogin
Patch Set: update baseurl, i hope Created 7 years, 9 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
« no previous file with comments | « no previous file | chrome/browser/signin/signin_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/signin/signin_manager.h
diff --git a/chrome/browser/signin/signin_manager.h b/chrome/browser/signin/signin_manager.h
index 7b55400a002184fdeb097ef83249ebbca2959a48..61ab8607941ca9caf28c38269cec65a190de9b9a 100644
--- a/chrome/browser/signin/signin_manager.h
+++ b/chrome/browser/signin/signin_manager.h
@@ -185,6 +185,7 @@ class SigninManager : public GaiaAuthConsumer,
virtual void OnClientOAuthSuccess(const ClientOAuthResult& result) OVERRIDE;
virtual void OnClientOAuthFailure(
const GoogleServiceAuthError& error) OVERRIDE;
+ virtual void OnOAuth2RevokeTokenCompleted() OVERRIDE;
virtual void OnGetUserInfoSuccess(const UserInfoMap& data) OVERRIDE;
virtual void OnGetUserInfoFailure(
const GoogleServiceAuthError& error) OVERRIDE;
@@ -282,6 +283,10 @@ class SigninManager : public GaiaAuthConsumer,
void HandleAuthError(const GoogleServiceAuthError& error,
bool clear_transient_data);
+ // Called to tell GAIA that we will no longer be using the current refresh
+ // token.
+ void RevokeOAuthLoginToken();
+
#if defined(ENABLE_CONFIGURATION_POLICY) && !defined(OS_CHROMEOS)
// Callback invoked once policy registration is complete. If registration
// fails, |client| will be null.
@@ -340,6 +345,9 @@ class SigninManager : public GaiaAuthConsumer,
// UbertokenFetcher to login to user to the web property.
scoped_ptr<UbertokenFetcher> ubertoken_fetcher_;
+ // OAuth revocation fetcher for sign outs.
+ scoped_ptr<GaiaAuthFetcher> revoke_token_fetcher_;
+
// Helper object to listen for changes to signin preferences stored in non-
// profile-specific local prefs (like kGoogleServicesUsernamePattern).
PrefChangeRegistrar local_state_pref_registrar_;
« no previous file with comments | « no previous file | chrome/browser/signin/signin_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698