From b826edb2af1b8f9b7c67b63cb4902cca9acb23d8 Mon Sep 17 00:00:00 2001
From: Daniel Nishi <dhnishi@google.com>
Date: Tue, 21 Feb 2017 12:57:57 -0800
Subject: [PATCH] Add logging to help debug loader issue.

Sometimes, it looks like the loader is not returning the result
to populate the storage settings view. The current bug reports
have no smoking guns. By adding this debug logging to the build,
the bug reports should hopefully help lead us to where the bug
is originating.

Bug: 35389719
Test: None
Change-Id: Ifa3f69f1fb4669aa72e6ef9c7ccc9d74b9e0fa71
---
 .../settings/deviceinfo/storage/StorageAsyncLoader.java     | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/com/android/settings/deviceinfo/storage/StorageAsyncLoader.java b/src/com/android/settings/deviceinfo/storage/StorageAsyncLoader.java
index 4e39bb303e6..a60831ba9db 100644
--- a/src/com/android/settings/deviceinfo/storage/StorageAsyncLoader.java
+++ b/src/com/android/settings/deviceinfo/storage/StorageAsyncLoader.java
@@ -26,6 +26,7 @@ import android.os.UserHandle;
 import android.util.ArrayMap;
 import android.util.ArraySet;
 import android.util.SparseArray;
+import android.util.Log;
 
 import com.android.settings.applications.PackageManagerWrapper;
 import com.android.settings.applications.UserManagerWrapper;
@@ -42,6 +43,8 @@ import java.util.Map;
 public class StorageAsyncLoader
         extends AsyncLoader<SparseArray<StorageAsyncLoader.AppsStorageResult>> {
     private UserManagerWrapper mUserManager;
+    private static final String TAG = "StorageAsyncLoader";
+
     private String mUuid;
     private StorageStatsSource mStatsManager;
     private PackageManagerWrapper mPackageManager;
@@ -71,6 +74,7 @@ public class StorageAsyncLoader
     }
 
     private AppsStorageResult getStorageResultForUser(int userId) {
+        Log.d(TAG, "Loading apps");
         List<ApplicationInfo> applicationInfos =
                 mPackageManager.getInstalledApplicationsAsUser(0, userId);
         ArraySet<Integer> seenUid = new ArraySet<>(); // some apps share a uid
@@ -98,7 +102,9 @@ public class StorageAsyncLoader
             }
         }
 
+        Log.d(TAG, "Loading external stats");
         result.externalStats = mStatsManager.getExternalStorageStats(mUuid, UserHandle.of(userId));
+        Log.d(TAG, "Obtaining result completed");
         return result;
     }