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

Commit 28f03c7d authored by Soonil Nagarkar's avatar Soonil Nagarkar
Browse files

Remove confusing delay messages 

There's no need to allow for OFF -> OFF changes, as it only produces
confusing logs. Also add a log for when a provider request changes so
it's clearer in the logcat.

Test: atest LocationProviderManagerTest

Change-Id: I88e0e5ec05cab5c73be9e1833fc8a8bf0ca2aff3
parent c0bcd9af

services/core/java/com/android/server/location/provider/LocationProviderManager.java

+8 −0
Original line number Diff line number Diff line
@@ -2105,6 +2105,11 @@ public class LocationProviderManager extends 
    @Override

    protected boolean registerWithService(ProviderRequest request,

            Collection<Registration> registrations) {

        if (!request.isActive()) {

            // the default request is already an empty request, no need to register this

            return true;

        }



        return reregisterWithService(ProviderRequest.EMPTY_REQUEST, request, registrations);

    }
@@ -2179,6 +2184,9 @@ public class LocationProviderManager extends 
        }



        EVENT_LOG.logProviderUpdateRequest(mName, request);

        if (D) {

            Log.d(TAG, mName + " provider request changed to " + request);

        }

        mProvider.getController().setRequest(request);



        FgThread.getHandler().post(() -> {