From e38efca813478c60b7f55188feabb4c70e8b66d7 Mon Sep 17 00:00:00 2001 From: Soonil Nagarkar Date: Fri, 3 Apr 2020 09:33:07 -0700 Subject: [PATCH] DO NOT MERGE Remove permission guard on getTestProviderCurrentRequests() This permission was only supposed to be in S, and was accidentally cherrypicked into R as part of a larger change. Bug: 153051820 Bug: 153051800 Bug: 152897902 Test: presubmits Change-Id: I1454351a5ad34149a58db945463da28ad75c1d24 --- .../core/java/com/android/server/LocationManagerService.java | 2 -- 1 file changed, 2 deletions(-) diff --git a/services/core/java/com/android/server/LocationManagerService.java b/services/core/java/com/android/server/LocationManagerService.java index 8ccff76058112..a9c3079adcc0a 100644 --- a/services/core/java/com/android/server/LocationManagerService.java +++ b/services/core/java/com/android/server/LocationManagerService.java @@ -2598,8 +2598,6 @@ public class LocationManagerService extends ILocationManager.Stub { @Override @NonNull public List getTestProviderCurrentRequests(String provider) { - mContext.enforceCallingOrSelfPermission(Manifest.permission.READ_DEVICE_CONFIG, null); - LocationProviderManager manager = getLocationProviderManager(provider); if (manager == null) { throw new IllegalArgumentException("provider doesn't exist: " + provider);