diff options
| author | 2022-09-08 17:30:38 +0000 | |
|---|---|---|
| committer | 2022-09-08 17:30:38 +0000 | |
| commit | 4dc6c54ddfa9f935732eb1c52ee1ba019f9562bb (patch) | |
| tree | af344c771a90d90d9a6360e088b2397702955d2b | |
| parent | fe4b37a3acf50fe1ceb18913251f4d7c8d89118a (diff) | |
| parent | 5b78fb706979fabe261923fb1398d3f3624f1ca6 (diff) | |
Merge "AudioService: fix initial spatial audio enable state."
| -rw-r--r-- | services/core/java/com/android/server/audio/SpatializerHelper.java | 22 |
1 files changed, 9 insertions, 13 deletions
diff --git a/services/core/java/com/android/server/audio/SpatializerHelper.java b/services/core/java/com/android/server/audio/SpatializerHelper.java index e4fdea742c5b..8a9d78b4ae28 100644 --- a/services/core/java/com/android/server/audio/SpatializerHelper.java +++ b/services/core/java/com/android/server/audio/SpatializerHelper.java @@ -748,33 +748,29 @@ public class SpatializerHelper { if (enabled) { throw (new IllegalStateException("Can't enable when uninitialized")); } - return; + break; case STATE_NOT_SUPPORTED: if (enabled) { Log.e(TAG, "Can't enable when unsupported"); } - return; + break; case STATE_DISABLED_UNAVAILABLE: case STATE_DISABLED_AVAILABLE: if (enabled) { createSpat(); onRoutingUpdated(); - break; - } else { - // already in disabled state - return; - } + // onRoutingUpdated() can update the "enabled" state based on context + // and will call setDispatchFeatureEnabledState(). + } // else { nothing to do as already disabled } + break; case STATE_ENABLED_UNAVAILABLE: case STATE_ENABLED_AVAILABLE: if (!enabled) { releaseSpat(); - break; - } else { - // already in enabled state - return; - } + setDispatchFeatureEnabledState(false, "setSpatializerEnabledInt"); + } // else { nothing to do as already enabled } + break; } - setDispatchFeatureEnabledState(enabled, "setSpatializerEnabledInt"); } synchronized int getCapableImmersiveAudioLevel() { |