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

Unified Diff: sync/android/java/src/org/chromium/sync/signin/AccountManagerDelegate.java

Issue 1504283002: [Sync] Switch to using GoogleAuthUtil.clearToken(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove USE_CREDENTIALS check again :) Created 5 years 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 | sync/android/java/src/org/chromium/sync/signin/AccountManagerHelper.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sync/android/java/src/org/chromium/sync/signin/AccountManagerDelegate.java
diff --git a/sync/android/java/src/org/chromium/sync/signin/AccountManagerDelegate.java b/sync/android/java/src/org/chromium/sync/signin/AccountManagerDelegate.java
index eafc4ae8202ef7e62d700a60987366426a14cccd..39c6865a53842e63b6ab15f05dbbd59566569319 100644
--- a/sync/android/java/src/org/chromium/sync/signin/AccountManagerDelegate.java
+++ b/sync/android/java/src/org/chromium/sync/signin/AccountManagerDelegate.java
@@ -26,20 +26,22 @@ public interface AccountManagerDelegate {
void getAccountsByType(String type, Callback<Account[]> callback);
/**
+ * Get an auth token. This should only be called on a background thread.
+ *
* @param account The {@link Account} for which the auth token is requested.
* @param authTokenScope The scope of the authToken being requested.
* @return The auth token fetched from the authenticator.
- * The authenticator can throw an {@link AuthException} to indicate a failure in fetching the
- * auth token perhaps due to a transient error or when user intervention is required (like
- * confirming the credentials) which is expressed as an {@link Intent} to the handler.
- * This should only be called on a background thread.
+ * @throws AuthException Indicates a failure in fetching the auth token perhaps due to a
+ * transient error or when user intervention is required (like confirming the credentials)
+ * which is expressed as an {@link Intent} to the handler.
*/
String getAuthToken(Account account, String authTokenScope) throws AuthException;
/**
- * Invalidate the {@code authToken} associated with account type {@code accountType}.
+ * @param authToken The auth token to invalidate.
+ * @throws AuthException Indicates a failure clearing the auth token; can be transient.
*/
- void invalidateAuthToken(String accountType, String authToken);
+ void invalidateAuthToken(String authToken) throws AuthException;
/**
* Get all the available authenticator types.
« no previous file with comments | « no previous file | sync/android/java/src/org/chromium/sync/signin/AccountManagerHelper.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698