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

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

Issue 213823004: Calls FireRefreshTokenRevoked if an account is removed. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: comments + tests Created 6 years, 8 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: chrome/browser/signin/android_profile_oauth2_token_service.h
diff --git a/chrome/browser/signin/android_profile_oauth2_token_service.h b/chrome/browser/signin/android_profile_oauth2_token_service.h
index 7d6a66029992f14806215f627225c64796c112c0..c869cb3e43229aee26d69904f5ab876484c3daa4 100644
--- a/chrome/browser/signin/android_profile_oauth2_token_service.h
+++ b/chrome/browser/signin/android_profile_oauth2_token_service.h
@@ -42,15 +42,20 @@ class AndroidProfileOAuth2TokenService : public ProfileOAuth2TokenService {
// Lists account IDs of all accounts with a refresh token.
virtual std::vector<std::string> GetAccounts() OVERRIDE;
+ // Lists account at the OS level.
+ std::vector<std::string> GetSystemAccounts();
+
void ValidateAccounts(JNIEnv* env,
jobject obj,
- jobjectArray accounts,
jstring current_account);
// Takes a the signed in sync account as well as all the other
// android account ids and check the token status of each.
- void ValidateAccounts(const std::string& signed_in_account,
- const std::vector<std::string>& account_ids);
+ void ValidateAccounts(const std::string& signed_in_account);
+
+ bool ValidateAccounts(const std::string& signed_in_account,
nyquist 2014/04/24 22:27:05 Why is this function public again? It seems to onl
+ const std::vector<std::string>& prev_account_ids,
+ const std::vector<std::string>& curr_account_ids);
// Triggers a notification to all observers of the OAuth2TokenService that a
// refresh token is now available. This may cause observers to retry
@@ -67,6 +72,10 @@ class AndroidProfileOAuth2TokenService : public ProfileOAuth2TokenService {
// refresh tokens have now been loaded.
virtual void FireRefreshTokensLoadedFromJava(JNIEnv* env, jobject obj);
+ // Overridden from OAuth2TokenService to complete signout of all
+ // OA2TService aware accounts.
+ virtual void RevokeAllCredentials() OVERRIDE;
+
protected:
friend class ProfileOAuth2TokenServiceFactory;
AndroidProfileOAuth2TokenService();

Powered by Google App Engine
This is Rietveld 408576698