commit d8913a420b0c37e3ec87decdcb3513fd4200c75a
author Eric Laurent <elaurent@google.com> Fri Jul 08 13:35:46 2022 +0200
committer Andy Hung <hunga@google.com> Tue Sep 06 19:40:04 2022 -0700
tree c1576b7187a161def6078b5379a8c871e0ad5a2e
parent b387fb45d2e339afee274cdf9db4bcde9765e149

Spatializer: fix head pose sensor enabling according to latency mode

Fix logic in Spatializer::checkSensorsState_l() to allow disabling
of head pose sensor if low latency mode is not supported.

Bug: 238408306
Test: spatial audio playback on A2DP wihtout low latency support
Merged-In: I3ff1bb83adc899ef020b635cc64fa04ab03e44f7
Change-Id: I3ff1bb83adc899ef020b635cc64fa04ab03e44f7

services/audiopolicy/service/Spatializer.cpp

diff --git a/services/audiopolicy/service/Spatializer.cpp b/services/audiopolicy/service/Spatializer.cpp
index b420199..9bba479 100644
--- a/services/audiopolicy/service/Spatializer.cpp
+++ b/services/audiopolicy/service/Spatializer.cpp
@@ -861,8 +861,8 @@
     bool lowLatencySupported = mSupportedLatencyModes.empty()
             || (std::find(mSupportedLatencyModes.begin(), mSupportedLatencyModes.end(),
                     AUDIO_LATENCY_MODE_LOW) != mSupportedLatencyModes.end());
-    if (mSupportsHeadTracking && mPoseController != nullptr && lowLatencySupported) {
-        if (mNumActiveTracks > 0 && mLevel != SpatializationLevel::NONE
+    if (mSupportsHeadTracking && mPoseController != nullptr) {
+        if (lowLatencySupported && mNumActiveTracks > 0 && mLevel != SpatializationLevel::NONE
             && mDesiredHeadTrackingMode != HeadTrackingMode::STATIC
             && mHeadSensor != SpatializerPoseController::INVALID_SENSOR) {
             mPoseController->setHeadSensor(mHeadSensor);