summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPhilip P. Moltmann <moltmann@google.com>2019-09-19 12:38:42 -0700
committerPhilip P. Moltmann <moltmann@google.com>2019-09-19 19:53:47 +0000
commit570755eacdf4ff9241f56b1b4ad9456e194cce50 (patch)
treeffe14e81558281651593c621cf1146e5a727d143
parent36f401ded229f78db2e21006059af83f7c3cb2cd (diff)
downloadandroid_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.java4
-rw-r--r--src/com/android/packageinstaller/role/service/RoleControllerServiceImpl.java5
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;
}