diff options
author | Eugene Susla <eugenesusla@google.com> | 2019-08-20 16:49:11 -0700 |
---|---|---|
committer | Eugene Susla <eugenesusla@google.com> | 2019-08-30 22:42:25 +0000 |
commit | 2053fa84e15492670692cb13bbe05748e84b2ebe (patch) | |
tree | 2229032ad57f59c1d315a688b5453489d1ffb197 | |
parent | 1719c99a5f31621c270ad10068e90eec48c61812 (diff) | |
download | android_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.java | 4 | ||||
-rw-r--r-- | src/com/android/packageinstaller/role/service/RoleControllerServiceImpl.java | 5 |
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; } |