diff options
author | android-build-team Robot <android-build-team-robot@google.com> | 2019-07-31 03:12:56 +0000 |
---|---|---|
committer | android-build-team Robot <android-build-team-robot@google.com> | 2019-07-31 03:12:56 +0000 |
commit | ae9992a42d7ac4342c4f3a33e829633d995950a5 (patch) | |
tree | db0a8e6ab4df96ebd12f345aea384fc4a71185cc | |
parent | 639382a3a2f73ff953cf2b28d058674c7a085aa3 (diff) | |
parent | 1f748546449f810ca3a9ae49a71ca8f7efd6ed6c (diff) | |
download | android_packages_apps_PackageInstaller-ae9992a42d7ac4342c4f3a33e829633d995950a5.tar.gz android_packages_apps_PackageInstaller-ae9992a42d7ac4342c4f3a33e829633d995950a5.tar.bz2 android_packages_apps_PackageInstaller-ae9992a42d7ac4342c4f3a33e829633d995950a5.zip |
Snap for 5767086 from 1f748546449f810ca3a9ae49a71ca8f7efd6ed6c to qt-qpr1-release
Change-Id: I47d05f297e9011f131a4680b793f728c1f69c265
-rw-r--r-- | src/com/android/packageinstaller/permission/service/PermissionControllerServiceImpl.java | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/src/com/android/packageinstaller/permission/service/PermissionControllerServiceImpl.java b/src/com/android/packageinstaller/permission/service/PermissionControllerServiceImpl.java index 6427ba36..d846ce09 100644 --- a/src/com/android/packageinstaller/permission/service/PermissionControllerServiceImpl.java +++ b/src/com/android/packageinstaller/permission/service/PermissionControllerServiceImpl.java @@ -67,9 +67,8 @@ import java.util.function.IntConsumer; /** * Calls from the system into the permission controller. * - * All methods are called async beside the backup related method. For these we force to use the - * async-task single thread executor so that multiple parallel backups don't override the delayed - * the backup state racily. + * All reading methods are called async, and all writing method are called on the AsyncTask single + * thread executor so that multiple writes won't override each other concurrently. */ public final class PermissionControllerServiceImpl extends PermissionControllerService { private static final String LOG_TAG = PermissionControllerServiceImpl.class.getSimpleName(); @@ -210,8 +209,8 @@ public final class PermissionControllerServiceImpl extends PermissionControllerS public void onRevokeRuntimePermissions(@NonNull Map<String, List<String>> request, boolean doDryRun, int reason, @NonNull String callerPackageName, @NonNull Consumer<Map<String, List<String>>> callback) { - AsyncTask.THREAD_POOL_EXECUTOR.execute(() -> callback.accept( - onRevokeRuntimePermissions(request, doDryRun, reason, callerPackageName))); + AsyncTask.execute(() -> callback.accept(onRevokeRuntimePermissions(request, doDryRun, + reason, callerPackageName))); } private @NonNull Map<String, List<String>> onRevokeRuntimePermissions( @@ -403,7 +402,7 @@ public final class PermissionControllerServiceImpl extends PermissionControllerS @Override public void onRevokeRuntimePermission(@NonNull String packageName, @NonNull String permissionName, @NonNull Runnable callback) { - AsyncTask.THREAD_POOL_EXECUTOR.execute(() -> { + AsyncTask.execute(() -> { onRevokeRuntimePermission(packageName, permissionName); callback.run(); }); @@ -503,9 +502,8 @@ public final class PermissionControllerServiceImpl extends PermissionControllerS public void onSetRuntimePermissionGrantStateByDeviceAdmin(@NonNull String callerPackageName, @NonNull String packageName, @NonNull String unexpandedPermission, int grantState, @NonNull Consumer<Boolean> callback) { - AsyncTask.THREAD_POOL_EXECUTOR.execute(() -> callback.accept( - onSetRuntimePermissionGrantStateByDeviceAdmin(callerPackageName, packageName, - unexpandedPermission, grantState))); + AsyncTask.execute(() -> callback.accept(onSetRuntimePermissionGrantStateByDeviceAdmin( + callerPackageName, packageName, unexpandedPermission, grantState))); } private boolean onSetRuntimePermissionGrantStateByDeviceAdmin(@NonNull String callerPackageName, @@ -568,7 +566,7 @@ public final class PermissionControllerServiceImpl extends PermissionControllerS @Override public void onGrantOrUpgradeDefaultRuntimePermissions(@NonNull Runnable callback) { - AsyncTask.THREAD_POOL_EXECUTOR.execute(() -> { + AsyncTask.execute(() -> { onGrantOrUpgradeDefaultRuntimePermissions(); callback.run(); }); |