commit 802954e7c4d59576a5d369886c9f9ac3076d3249
author Treehugger Robot <treehugger-gerrit@google.com> Fri Jul 24 19:49:28 2020 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Fri Jul 24 19:49:28 2020 +0000
tree 254cfafde44df4350751c0e66c463c6c91420af4
parent 5845a86ead39c86cbb1588a74148ddcb663e7fe2
parent 7054e598f4efa6f7c5c8a3f8f2f03dc4cb7826a4

Merge "BTAudio: report session ended after cleaning up" am: 5ae008679d am: 5373028c3c am: 3c1ec94cd0 am: 7054e598f4

Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1371705

Change-Id: I3c0b2bae461021da37084affa79366380eca50f1

bluetooth/audio/2.0/default/session/BluetoothAudioSession.cpp

diff --git a/bluetooth/audio/2.0/default/session/BluetoothAudioSession.cpp b/bluetooth/audio/2.0/default/session/BluetoothAudioSession.cpp
index d60e732..50119bf 100644
--- a/bluetooth/audio/2.0/default/session/BluetoothAudioSession.cpp
+++ b/bluetooth/audio/2.0/default/session/BluetoothAudioSession.cpp
@@ -90,14 +90,16 @@
 // bluetooth_audio outputs
 void BluetoothAudioSession::OnSessionEnded() {
   std::lock_guard<std::recursive_mutex> guard(mutex_);
-  if (IsSessionReady()) {
-    ReportSessionStatus();
-  }
+  bool toggled = IsSessionReady();
+  LOG(INFO) << __func__ << " - SessionType=" << toString(session_type_);
   audio_config_ = (session_type_ == SessionType::A2DP_HARDWARE_OFFLOAD_DATAPATH
                        ? kInvalidOffloadAudioConfiguration
                        : kInvalidSoftwareAudioConfiguration);
   stack_iface_ = nullptr;
   UpdateDataPath(nullptr);
+  if (toggled) {
+    ReportSessionStatus();
+  }
 }
 
 // invoking the registered session_changed_cb_