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

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: 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..1c76d2e19f9977f607192d3a6d6ef61b3fb7462e 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. Can be called directly from this function.
msarda 2017/03/28 11:09:02 Optional nit: s/Can be called directly from this f
bsazonov 2017/03/28 12:59:58 Done.
*/
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