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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/signin/SigninManager.java

Issue 2772203004: Add progress and timeout dialogs for getting account management policy (Closed)
Patch Set: Updated strings and addressed comments Created 3 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
Index: chrome/android/java/src/org/chromium/chrome/browser/signin/SigninManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/signin/SigninManager.java b/chrome/android/java/src/org/chromium/chrome/browser/signin/SigninManager.java
index 623205c443fcbc56dc55212c8457a193d9f31f90..dc4aef64020d8e51cce40ccffc7b6a199a930954 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/signin/SigninManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/signin/SigninManager.java
@@ -644,21 +644,13 @@ public class SigninManager implements AccountTrackerService.OnSystemAccountsSeed
/**
* Performs an asynchronous check to see if the user is a managed user.
* @param callback A callback to be called with true if the user is a managed user and false
- * otherwise.
+ * otherwise. May be called synchronously from this function.
*/
public static void isUserManaged(String email, final Callback<Boolean> callback) {
if (nativeShouldLoadPolicyForUser(email)) {
nativeIsUserManaged(email, callback);
} else {
- // Although we know the result immediately, the caller may not be able to handle the
- // callback being executed during this method call. So we post the callback on the
- // looper.
- ThreadUtils.postOnUiThread(new Runnable() {
- @Override
- public void run() {
- callback.onResult(false);
- }
- });
+ callback.onResult(false);
}
}

Powered by Google App Engine
This is Rietveld 408576698