commit e9f5c354556a5f3d04c73c918b4559290421fa94
author Josh Wu <joshwu@google.com> Wed Mar 24 20:15:43 2021 +0800
committer Josh Wu <joshwu@google.com> Wed Mar 31 14:34:06 2021 +0800
tree 0a4e85fe5d6d769113a62c67db35f42f933bec34
parent b6f7ab5cf9833509754c7b46e2929160e1831551

A2DP: Log BLUETOOTH_A2DP_PLAYBACK_STATE_CHANGED

Enable BLUETOOTH_A2DP_PLAYBACK_STATE_CHANGED Result metrics logged in stack

Tag: #feature
Bug: 183473902
Test: atest BluetoothInstrumentationTests
Change-Id: I9e9b7d1d0e6fd094978ccaad9c16a0e2306e7882

system/common/metrics.cc

diff --git a/system/common/metrics.cc b/system/common/metrics.cc
index 10eedc8..1849612 100644
--- a/system/common/metrics.cc
+++ b/system/common/metrics.cc
@@ -678,6 +678,29 @@
   }
 }
 
+void LogA2dpPlaybackEvent(const RawAddress& address, int playback_state,
+                          int audio_coding_mode) {
+  std::string obfuscated_id;
+  int metric_id = 0;
+  if (!address.IsEmpty()) {
+    obfuscated_id = AddressObfuscator::GetInstance()->Obfuscate(address);
+    metric_id = MetricIdAllocator::GetInstance().AllocateId(address);
+  }
+  // nullptr and size 0 represent missing value for obfuscated_id
+  android::util::BytesField bytes_field(
+      address.IsEmpty() ? nullptr : obfuscated_id.c_str(),
+      address.IsEmpty() ? 0 : obfuscated_id.size());
+  int ret = android::util::stats_write(
+      android::util::BLUETOOTH_A2DP_PLAYBACK_STATE_CHANGED, bytes_field,
+      playback_state, audio_coding_mode, metric_id);
+  if (ret < 0) {
+    LOG(WARNING) << __func__ << ": failed to log for " << address
+                 << ", playback_state " << playback_state
+                 << ", audio_coding_mode " << audio_coding_mode << ", error "
+                 << ret;
+  }
+}
+
 void LogReadRssiResult(const RawAddress& address, uint16_t handle,
                        uint32_t cmd_status, int8_t rssi) {
   std::string obfuscated_id;

system/common/metrics.h

diff --git a/system/common/metrics.h b/system/common/metrics.h
index a36a64c..c8973f3 100644
--- a/system/common/metrics.h
+++ b/system/common/metrics.h
@@ -360,6 +360,16 @@
                               int num_dropped_encoded_bytes);
 
 /**
+ * Log A2DP playback state changed event
+ *
+ * @param address A2DP device associated with this event
+ * @param playback_state audio playback state
+ * @param audio_coding_mode audio codec encoding mode
+ */
+void LogA2dpPlaybackEvent(const RawAddress& address, int playback_state,
+                          int audio_coding_mode);
+
+/**
  * Log read RSSI result
  *
  * @param address device associated with this event