summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Bookatz <bookatz@google.com> 2018-09-13 11:38:56 -0700
committer Adam Bookatz <bookatz@google.com> 2018-09-13 18:48:10 +0000
commitae6738ed2a86da9dc89f1283cc4a2a94a583e9bd (patch)
treeaf72e25e66cae534ad950ac3ec9eead94dcfe78d
parent375af769e60024aab97418daf11c3c63294678eb (diff)
Unifying source of statsd BleScan calls and cleaning it up.
Bug: 80308558 Test: cts-tradefed run cts-dev -m CtsStatsdHostTestCases Change-Id: I0b4a0b56631b4354b73f0fafea2c0dc7dff248d1
-rw-r--r--cmds/statsd/src/atoms.proto4
-rw-r--r--core/java/com/android/internal/os/BatteryStatsImpl.java4
2 files changed, 2 insertions, 6 deletions
diff --git a/cmds/statsd/src/atoms.proto b/cmds/statsd/src/atoms.proto
index ab9c7e81a88e..18f6a1c28086 100644
--- a/cmds/statsd/src/atoms.proto
+++ b/cmds/statsd/src/atoms.proto
@@ -272,9 +272,9 @@ message BleScanStateChanged {
* Logs reporting of a ble scan finding results.
*
* Logged from:
- * frameworks/base/core/java/com/android/internal/os/BatteryStatsImpl.java
+ * packages/apps/Bluetooth/src/com/android/bluetooth/gatt/AppScanStats.java
*/
-// TODO: Consider changing to tracking per-scanner-id (log from AppScanStats).
+// TODO: Consider also tracking per-scanner-id.
message BleScanResultReceived {
repeated AttributionNode attribution_node = 1;
diff --git a/core/java/com/android/internal/os/BatteryStatsImpl.java b/core/java/com/android/internal/os/BatteryStatsImpl.java
index 94c3933c55f0..10c32a3b7a36 100644
--- a/core/java/com/android/internal/os/BatteryStatsImpl.java
+++ b/core/java/com/android/internal/os/BatteryStatsImpl.java
@@ -5800,8 +5800,6 @@ public class BatteryStatsImpl extends BatteryStats {
for (int i = 0; i < N; i++) {
int uid = mapUid(ws.get(i));
getUidStatsLocked(uid).noteBluetoothScanResultsLocked(numNewResults);
- StatsLog.write_non_chained(StatsLog.BLE_SCAN_RESULT_RECEIVED, ws.get(i), ws.getName(i),
- numNewResults);
}
final List<WorkChain> workChains = ws.getWorkChains();
@@ -5810,8 +5808,6 @@ public class BatteryStatsImpl extends BatteryStats {
final WorkChain wc = workChains.get(i);
int uid = mapUid(wc.getAttributionUid());
getUidStatsLocked(uid).noteBluetoothScanResultsLocked(numNewResults);
- StatsLog.write(StatsLog.BLE_SCAN_RESULT_RECEIVED,
- wc.getUids(), wc.getTags(), numNewResults);
}
}
}