diff options
author | Blake Kragten <kragtenb@google.com> | 2020-01-16 02:44:25 +0000 |
---|---|---|
committer | Blake Kragten <kragtenb@google.com> | 2020-01-15 18:57:33 -0800 |
commit | c7bdb71cc76f35364dc5879e4aef7c672d65c413 (patch) | |
tree | c0ac9a5c6ec05f1e49f12af5576df12e9ac9c953 | |
parent | 7a5f57289636335fa4db326776ab2b8554ac85d1 (diff) |
Revert "GPS Atoms Addition"
Revert "CTS GPS test"
Revert submission 9726785-1574459091307-c480d999
Reason for revert: Pushed atoms will be integrated into the new pulled atom.
Reverted Changes:
I9cbc8e41e: CTS GPS test
If8aed8f47: GPS Atoms Addition
Change-Id: I8d0a0b73d870a8d9fb02975df70f662de4fd1690
-rw-r--r-- | cmds/statsd/src/atoms.proto | 23 | ||||
-rw-r--r-- | location/java/com/android/internal/location/gnssmetrics/GnssMetrics.java | 3 |
2 files changed, 0 insertions, 26 deletions
diff --git a/cmds/statsd/src/atoms.proto b/cmds/statsd/src/atoms.proto index 815e3e668d31..a2214166e6ba 100644 --- a/cmds/statsd/src/atoms.proto +++ b/cmds/statsd/src/atoms.proto @@ -324,8 +324,6 @@ message Atom { 228 [(allow_from_any_uid) = true]; PerfettoUploaded perfetto_uploaded = 229 [(module) = "perfetto"]; VmsClientConnectionStateChanged vms_client_connection_state_changed = 230; - GpsLocationStatusReported gps_location_status_reported = 231; - GpsTimeToFirstFixReported gps_time_to_first_fix_reported = 232; MediaProviderScanEvent media_provider_scan_event = 233 [(module) = "mediaprovider"]; MediaProviderDeletionEvent media_provider_deletion_event = 234 [(module) = "mediaprovider"]; MediaProviderPermissionEvent media_provider_permission_event = @@ -729,27 +727,6 @@ message GpsSignalQualityChanged { optional android.server.location.GpsSignalQualityEnum level = 1; } -/** - * Gps location status report - * - * Logged from: - * /frameworks/base/location/java/com/android/internal/location/gnssmetrics/GnssMetrics.java - */ -message GpsLocationStatusReported { - // Boolean stating if location was acquired - optional bool location_success = 1; -} - -/** - * Gps log time to first fix report - * - * Logged from: - * /frameworks/base/location/java/com/android/internal/location/gnssmetrics/GnssMetrics.java - */ -message GpsTimeToFirstFixReported { - // int32 reporting the time to first fix in milliseconds - optional int32 time_to_first_fix_millis = 1; -} /** * Logs when a sync manager sync state changes. diff --git a/location/java/com/android/internal/location/gnssmetrics/GnssMetrics.java b/location/java/com/android/internal/location/gnssmetrics/GnssMetrics.java index 751bb6a70880..127d00c0afe2 100644 --- a/location/java/com/android/internal/location/gnssmetrics/GnssMetrics.java +++ b/location/java/com/android/internal/location/gnssmetrics/GnssMetrics.java @@ -110,7 +110,6 @@ public class GnssMetrics { * Logs the status of a location report received from the HAL */ public void logReceivedLocationStatus(boolean isSuccessful) { - StatsLog.write(StatsLog.GPS_LOCATION_STATUS_REPORTED, isSuccessful); if (!isSuccessful) { mLocationFailureStatistics.addItem(1.0); return; @@ -127,7 +126,6 @@ public class GnssMetrics { DEFAULT_TIME_BETWEEN_FIXES_MILLISECS, desiredTimeBetweenFixesMilliSeconds)) - 1; if (numReportMissed > 0) { for (int i = 0; i < numReportMissed; i++) { - StatsLog.write(StatsLog.GPS_LOCATION_STATUS_REPORTED, false); mLocationFailureStatistics.addItem(1.0); } } @@ -138,7 +136,6 @@ public class GnssMetrics { */ public void logTimeToFirstFixMilliSecs(int timeToFirstFixMilliSeconds) { mTimeToFirstFixSecStatistics.addItem((double) (timeToFirstFixMilliSeconds / 1000)); - StatsLog.write(StatsLog.GPS_TIME_TO_FIRST_FIX_REPORTED, timeToFirstFixMilliSeconds); } /** |