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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/child_accounts/ChildAccountService.java

Issue 893003004: Temporary logging to help ANR dissection (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 11 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/child_accounts/ChildAccountService.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/child_accounts/ChildAccountService.java b/chrome/android/java/src/org/chromium/chrome/browser/child_accounts/ChildAccountService.java
index 15066cc7da8e9c86c9a72aaad4928558d760aca9..012513b662f534b09c9194ea6404e281264c2ff9 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/child_accounts/ChildAccountService.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/child_accounts/ChildAccountService.java
@@ -128,6 +128,7 @@ public class ChildAccountService {
new AccountManagerCallback<Boolean>() {
@Override
public void run(AccountManagerFuture<Boolean> future) {
+ Log.i(TAG, "completed AM request");
assert future == mAccountManagerFuture;
assert future.isDone();
@@ -143,7 +144,10 @@ public class ChildAccountService {
timer.schedule(new TimerTask() {
@Override
public void run() {
- if (!mAccountManagerFuture.isDone()) mAccountManagerFuture.cancel(true);
+ if (!mAccountManagerFuture.isDone()) {
+ Log.i(TAG, "cancelling AM request");
+ mAccountManagerFuture.cancel(true);
+ }
}}, CHILD_ACCOUNT_TIMEOUT_MS);
}
@@ -155,7 +159,10 @@ public class ChildAccountService {
private boolean getFutureResult() {
try {
- return mAccountManagerFuture.getResult();
+ Log.i(TAG, "before mAccountManagerFuture.getResult()");
+ boolean result = mAccountManagerFuture.getResult();
+ Log.i(TAG, "after mAccountManagerFuture.getResult()");
+ return result;
} catch (OperationCanceledException e) {
Log.e(TAG, "Timed out fetching child account flag: ", e);
} catch (AuthenticatorException e) {

Powered by Google App Engine
This is Rietveld 408576698