summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEugene Susla <eugenesusla@google.com>2019-08-20 16:49:11 -0700
committerEugene Susla <eugenesusla@google.com>2019-08-30 22:42:25 +0000
commit2053fa84e15492670692cb13bbe05748e84b2ebe (patch)
tree2229032ad57f59c1d315a688b5453489d1ffb197
parent1719c99a5f31621c270ad10068e90eec48c61812 (diff)
downloadandroid_packages_apps_PackageInstaller-2053fa84e15492670692cb13bbe05748e84b2ebe.tar.gz
android_packages_apps_PackageInstaller-2053fa84e15492670692cb13bbe05748e84b2ebe.tar.bz2
android_packages_apps_PackageInstaller-2053fa84e15492670692cb13bbe05748e84b2ebe.zip
updateUserSensitive asynchronously
This is known to take 500ms and affects only UI, so can be done async Test: Ensure nothing looks badly broken; presubmit Fixes: 139485700 Change-Id: I3837404ea7d9b1598efa0d35cc999af0e86cc7f2
-rw-r--r--src/com/android/packageinstaller/permission/service/PermissionControllerServiceImpl.java4
-rw-r--r--src/com/android/packageinstaller/role/service/RoleControllerServiceImpl.java5
2 files changed, 4 insertions, 5 deletions
diff --git a/src/com/android/packageinstaller/permission/service/PermissionControllerServiceImpl.java b/src/com/android/packageinstaller/permission/service/PermissionControllerServiceImpl.java
index d846ce09..d0ee1809 100644
--- a/src/com/android/packageinstaller/permission/service/PermissionControllerServiceImpl.java
+++ b/src/com/android/packageinstaller/permission/service/PermissionControllerServiceImpl.java
@@ -34,6 +34,7 @@ import android.content.Context;
import android.content.pm.PackageInfo;
import android.content.pm.PackageManager;
import android.os.AsyncTask;
+import android.os.Process;
import android.os.UserHandle;
import android.permission.PermissionControllerService;
import android.permission.PermissionManager;
@@ -569,6 +570,9 @@ public final class PermissionControllerServiceImpl extends PermissionControllerS
AsyncTask.execute(() -> {
onGrantOrUpgradeDefaultRuntimePermissions();
callback.run();
+
+ // Async part
+ Utils.updateUserSensitive(getApplication(), Process.myUserHandle());
});
}
diff --git a/src/com/android/packageinstaller/role/service/RoleControllerServiceImpl.java b/src/com/android/packageinstaller/role/service/RoleControllerServiceImpl.java
index 1ba0e0ff..fde630b5 100644
--- a/src/com/android/packageinstaller/role/service/RoleControllerServiceImpl.java
+++ b/src/com/android/packageinstaller/role/service/RoleControllerServiceImpl.java
@@ -29,7 +29,6 @@ import androidx.annotation.NonNull;
import androidx.annotation.WorkerThread;
import com.android.packageinstaller.permission.utils.CollectionUtils;
-import com.android.packageinstaller.permission.utils.Utils;
import com.android.packageinstaller.role.model.Role;
import com.android.packageinstaller.role.model.Roles;
import com.android.packageinstaller.role.utils.PackageUtils;
@@ -169,10 +168,6 @@ public class RoleControllerServiceImpl extends RoleControllerService {
}
}
- // Load data on this thread instead of background.
- // TODO: Move out of this thread
- Utils.updateUserSensitive(getApplication(), Process.myUserHandle());
-
return true;
}