From 40ac0217fc0126b2ad7979f6244b27022f4ceb4a Mon Sep 17 00:00:00 2001 From: Becca Hughes Date: Fri, 16 Dec 2022 17:44:37 +0000 Subject: [PATCH] Remove QUERY_ALL_APPS permission for listEnabledProviders We don't actually need this permission because we read the setting and we are getting pushback from settings team about this permission so I removed it and everything still works. That function is stil gated by the other permission. Test: manual & make Bug: 253157366 Change-Id: Ifdd2bcd52e9f38167aa1e087da492396f31df7ad --- core/java/android/credentials/CredentialManager.java | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/core/java/android/credentials/CredentialManager.java b/core/java/android/credentials/CredentialManager.java index d02360998651d..8b614d87a0da6 100644 --- a/core/java/android/credentials/CredentialManager.java +++ b/core/java/android/credentials/CredentialManager.java @@ -208,11 +208,7 @@ public final class CredentialManager { * @param callback the callback invoked when the request succeeds or fails * @hide */ - @RequiresPermission( - allOf = { - android.Manifest.permission.LIST_ENABLED_CREDENTIAL_PROVIDERS, - android.Manifest.permission.QUERY_ALL_PACKAGES - }) + @RequiresPermission(android.Manifest.permission.LIST_ENABLED_CREDENTIAL_PROVIDERS) public void listEnabledProviders( @Nullable CancellationSignal cancellationSignal, @CallbackExecutor @NonNull Executor executor,