throw SecurityException in isProviderEnabled()

In MR0, we did not allow applications to query enabled status of
location providers they did not have permission to use. Some
applications counted on this behavior, using the thrown
SecurityException to determine whether or not they have permission
to use the specified provider.

Reverting to this behavior fixes the regressions seen in those
applications.

Bug: 7251459
Change-Id: I8b0cfd5862c80f0c831a4ab544c3fa7408bc84a0

Author: Victoria Lease

services/java/com/android/server/LocationManagerService.java

@@ -1293,8 +1293,13 @@ public ProviderProperties getProviderProperties(String provider) {
 
     @Override
     public boolean isProviderEnabled(String provider) {
-        checkPermission();
+        String perms = checkPermission();
         if (LocationManager.FUSED_PROVIDER.equals(provider)) return false;
+        if (ACCESS_COARSE_LOCATION.equals(perms) &&
+                !isProviderAllowedByCoarsePermission(provider)) {
+            throw new SecurityException("The \"" + provider +
+                    "\" provider requires ACCESS_FINE_LOCATION permission");
+        }
 
         synchronized (mLock) {
             LocationProviderInterface p = mProvidersByName.get(provider);