Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 07ee5d1f authored by Tom O'Neill's avatar Tom O'Neill
Browse files

Removing a test provider clears its enabled state

- Bug: 11446702

Change-Id: I56ad9235a99772079193553c0f28bd8d74b3d8e3
parent 8cf5eee3
Loading
Loading
Loading
Loading
+7 −0
Original line number Diff line number Diff line
@@ -2227,6 +2227,13 @@ public class LocationManagerService extends ILocationManager.Stub {
    public void removeTestProvider(String provider) {
        checkMockPermissionsSafe();
        synchronized (mLock) {

            // These methods can't be called after removing the test provider, so first make sure
            // we don't leave anything dangling (cf b/11446702).
            clearTestProviderEnabled(provider);
            clearTestProviderLocation(provider);
            clearTestProviderStatus(provider);

            MockProvider mockProvider = mMockProviders.remove(provider);
            if (mockProvider == null) {
                throw new IllegalArgumentException("Provider \"" + provider + "\" unknown");