diff options
author | Philip P. Moltmann <moltmann@google.com> | 2019-09-19 12:38:42 -0700 |
---|---|---|
committer | Philip P. Moltmann <moltmann@google.com> | 2019-09-19 19:53:47 +0000 |
commit | 570755eacdf4ff9241f56b1b4ad9456e194cce50 (patch) | |
tree | ffe14e81558281651593c621cf1146e5a727d143 | |
parent | 36f401ded229f78db2e21006059af83f7c3cb2cd (diff) | |
download | android_packages_apps_PackageInstaller-570755eacdf4ff9241f56b1b4ad9456e194cce50.tar.gz android_packages_apps_PackageInstaller-570755eacdf4ff9241f56b1b4ad9456e194cce50.tar.bz2 android_packages_apps_PackageInstaller-570755eacdf4ff9241f56b1b4ad9456e194cce50.zip |
[DO NOT MERGE] Move updateUserSenstive trigger to onGrantDefaultRoles
We need to recalculate the user sensitivity every time a package is
installed or changes.
The logic to recalculate was moved to only during boot. Move it back to
where it was before, but leave it async.
Test: forcefully installed GooglePermissionsController to override
out-of-date mainline module.
Then verified that with code before this change user-sensitive
was not updated for newly installed apps. With this change
user-sensitive is updated
Bug: 141311767
Change-Id: I6c100764850870e2e5d1ffe5fcd20b5cf2f94ed1
-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, 5 insertions, 4 deletions
diff --git a/src/com/android/packageinstaller/permission/service/PermissionControllerServiceImpl.java b/src/com/android/packageinstaller/permission/service/PermissionControllerServiceImpl.java index d0ee1809..d846ce09 100644 --- a/src/com/android/packageinstaller/permission/service/PermissionControllerServiceImpl.java +++ b/src/com/android/packageinstaller/permission/service/PermissionControllerServiceImpl.java @@ -34,7 +34,6 @@ 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; @@ -570,9 +569,6 @@ 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 fde630b5..bee43205 100644 --- a/src/com/android/packageinstaller/role/service/RoleControllerServiceImpl.java +++ b/src/com/android/packageinstaller/role/service/RoleControllerServiceImpl.java @@ -19,6 +19,7 @@ package com.android.packageinstaller.role.service; import android.app.role.RoleControllerService; import android.app.role.RoleManager; import android.content.pm.ApplicationInfo; +import android.os.AsyncTask; import android.os.Process; import android.os.UserHandle; import android.util.ArrayMap; @@ -29,6 +30,7 @@ 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; @@ -168,6 +170,9 @@ public class RoleControllerServiceImpl extends RoleControllerService { } } + AsyncTask.execute( + () -> Utils.updateUserSensitive(getApplication(), Process.myUserHandle())); + return true; } |