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

Commit d30c1cea authored by Yu-Han Yang's avatar Yu-Han Yang
Browse files

Always call stopMeasurementCollection before startMeasurementCollection 

Bug: 253453668
Test: manual
Change-Id: I9f85c6be7a79540fd5fd8b851695c0de1742853d
parent 2d054595

services/core/java/com/android/server/location/gnss/GnssMeasurementsProvider.java

+10 −0
Original line number Diff line number Diff line
@@ -115,6 +115,16 @@ public final class GnssMeasurementsProvider extends 
        if (request.getIntervalMillis() == GnssMeasurementRequest.PASSIVE_INTERVAL) {

            return true;

        }

        // The HAL doc does not specify if consecutive start() calls will be allowed.

        // Some vendors may ignore the 2nd start() call if stop() is not called.

        // Thus, here we always call stop() before calling start() to avoid being ignored.

        if (mGnssNative.stopMeasurementCollection()) {

            if (D) {

                Log.d(TAG, "stopping gnss measurements");

            }

        } else {

            Log.e(TAG, "error stopping gnss measurements");

        }

        if (mGnssNative.startMeasurementCollection(request.isFullTracking(),

                request.isCorrelationVectorOutputsEnabled(),

                request.getIntervalMillis())) {