summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Andy Hung <hunga@google.com> 2022-09-08 17:30:38 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> 2022-09-08 17:30:38 +0000
commit4dc6c54ddfa9f935732eb1c52ee1ba019f9562bb (patch)
treeaf344c771a90d90d9a6360e088b2397702955d2b
parentfe4b37a3acf50fe1ceb18913251f4d7c8d89118a (diff)
parent5b78fb706979fabe261923fb1398d3f3624f1ca6 (diff)
Merge "AudioService: fix initial spatial audio enable state."
-rw-r--r--services/core/java/com/android/server/audio/SpatializerHelper.java22
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() {