diff options
author | Tej Singh <singhtejinder@google.com> | 2020-02-26 23:46:29 -0800 |
---|---|---|
committer | Tej Singh <singhtejinder@google.com> | 2020-03-05 16:43:02 -0800 |
commit | 72a70a8a3c56624997df4eca952073f37ef87cb7 (patch) | |
tree | 1c427cf6770bbd27c9fcce9aa9a18bdd6275a9d2 /location | |
parent | 995e2de829564fb5ba895e5fe245ab81a980ffc0 (diff) |
Address Puller API Feedback
1. Rename registerPullAtomCallback to setPullAtomCallback
2. Rename unregisterPullAtomCallback to clearPullAtomCallback
3. Add getters to PullAtomMetadata
4. Change Ns to Millis (when I tried to make it Nanos, I received a
built time error saying to prefer millis unless we need the precision.
We do not need the precision, so I changed it).
5. Fix out of order params.
I did not change usePooledBuffer to setPooledBuffer because I think use
is more appropriate for our use case.
Test: make
Test: atest PullAtomMetadataTest
Test: atest GtsStatsdHostTestCases
Bug: 149475498
Change-Id: Ib07aa57a6e02c77917fe0e65a3d4a77c00ce8565
Diffstat (limited to 'location')
-rw-r--r-- | location/java/com/android/internal/location/gnssmetrics/GnssMetrics.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/location/java/com/android/internal/location/gnssmetrics/GnssMetrics.java b/location/java/com/android/internal/location/gnssmetrics/GnssMetrics.java index 1ac4b4b29fbf..323bba3fdd71 100644 --- a/location/java/com/android/internal/location/gnssmetrics/GnssMetrics.java +++ b/location/java/com/android/internal/location/gnssmetrics/GnssMetrics.java @@ -624,7 +624,7 @@ public class GnssMetrics { private void registerGnssStats() { mPullAtomCallback = new StatsPullAtomCallbackImpl(); - mStatsManager.registerPullAtomCallback( + mStatsManager.setPullAtomCallback( FrameworkStatsLog.GNSS_STATS, null, // use default PullAtomMetadata values BackgroundThread.getExecutor(), mPullAtomCallback); |