From bb61148802092fe8d9cab02ea798b49aea34a7c5 Mon Sep 17 00:00:00 2001
From: Kevin Tang <zhikait@codeaurora.org>
Date: Sat, 16 Dec 2017 10:59:47 -0800
Subject: [PATCH] GnssLocationProvider metrics race condition fix

setting mLastFixTime = 0 in stopNavigating() introduces race condition where
if a fix is reported back from HAL milliseconds after stopNavigating() is
called, there will be a long TTFF calculated.

Bug: 70747938
Change-Id: I4868fd5ff6a1c072581e7825ac48ff1878833436
---
 .../java/com/android/server/location/GnssLocationProvider.java   | 1 -
 1 file changed, 1 deletion(-)

diff --git a/services/core/java/com/android/server/location/GnssLocationProvider.java b/services/core/java/com/android/server/location/GnssLocationProvider.java
index 55c0f5a95907a..c9f4edfee1989 100644
--- a/services/core/java/com/android/server/location/GnssLocationProvider.java
+++ b/services/core/java/com/android/server/location/GnssLocationProvider.java
@@ -1582,7 +1582,6 @@ public class GnssLocationProvider implements LocationProviderInterface {
             mStarted = false;
             mSingleShot = false;
             native_stop();
-            mTimeToFirstFix = 0;
             mLastFixTime = 0;
 
             // reset SV count to zero