summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Raj Goparaju <rajgoparaju@google.com> 2023-12-09 14:20:21 -0800
committer Raj Goparaju <rajgoparaju@google.com> 2023-12-09 14:20:21 -0800
commit5db107c2a0c5e53c454ae26bbca5c9ba2f50cc54 (patch)
treea6f8bd55d4a9675272cc09932e3827d3281db9f4
parent90471152caddaa4c1116b5cb0389c1d47148d81d (diff)
Remove fade manager aconfig from media solutions repo
The flag was initially defined in media_solutions namespace whereas the correct namespace is media_audio. Move the flag to the audio repo while maintaining the same naming convention. This commit removes the flag definition from media scope and updates import to audio. Bug: 186905459 Bug: 307354764 API-Coverage-Bug: 308666800 Test: atest -c FadeManagerConfigurationUnitTest Change-Id: I4f8701330f5e61aa5601db99f51c1b963eaed107
-rw-r--r--AconfigFlags.bp5
-rw-r--r--media/java/android/media/FadeManagerConfiguration.java2
-rw-r--r--media/java/android/media/flags/fade_manager_configuration.aconfig8
-rw-r--r--media/tests/AudioPolicyTest/src/com/android/audiopolicytest/FadeManagerConfigurationUnitTest.java2
4 files changed, 3 insertions, 14 deletions
diff --git a/AconfigFlags.bp b/AconfigFlags.bp
index 22f736e367c3..f46b188882a7 100644
--- a/AconfigFlags.bp
+++ b/AconfigFlags.bp
@@ -429,10 +429,7 @@ java_aconfig_library {
aconfig_declarations {
name: "com.android.media.flags.bettertogether-aconfig",
package: "com.android.media.flags",
- srcs: [
- "media/java/android/media/flags/media_better_together.aconfig",
- "media/java/android/media/flags/fade_manager_configuration.aconfig",
- ],
+ srcs: ["media/java/android/media/flags/media_better_together.aconfig"],
}
java_aconfig_library {
diff --git a/media/java/android/media/FadeManagerConfiguration.java b/media/java/android/media/FadeManagerConfiguration.java
index 337d4b0a916c..03b63dbd4649 100644
--- a/media/java/android/media/FadeManagerConfiguration.java
+++ b/media/java/android/media/FadeManagerConfiguration.java
@@ -16,7 +16,7 @@
package android.media;
-import static com.android.media.flags.Flags.FLAG_ENABLE_FADE_MANAGER_CONFIGURATION;
+import static android.media.audiopolicy.Flags.FLAG_ENABLE_FADE_MANAGER_CONFIGURATION;
import android.annotation.FlaggedApi;
import android.annotation.IntDef;
diff --git a/media/java/android/media/flags/fade_manager_configuration.aconfig b/media/java/android/media/flags/fade_manager_configuration.aconfig
deleted file mode 100644
index 100e2235a7a8..000000000000
--- a/media/java/android/media/flags/fade_manager_configuration.aconfig
+++ /dev/null
@@ -1,8 +0,0 @@
-package: "com.android.media.flags"
-
-flag {
- namespace: "media_solutions"
- name: "enable_fade_manager_configuration"
- description: "Enable Fade Manager Configuration support to determine fade properties"
- bug: "307354764"
-} \ No newline at end of file
diff --git a/media/tests/AudioPolicyTest/src/com/android/audiopolicytest/FadeManagerConfigurationUnitTest.java b/media/tests/AudioPolicyTest/src/com/android/audiopolicytest/FadeManagerConfigurationUnitTest.java
index fb6bd489d5d0..f105ae9cc33e 100644
--- a/media/tests/AudioPolicyTest/src/com/android/audiopolicytest/FadeManagerConfigurationUnitTest.java
+++ b/media/tests/AudioPolicyTest/src/com/android/audiopolicytest/FadeManagerConfigurationUnitTest.java
@@ -16,7 +16,7 @@
package com.android.audiopolicytest;
-import static com.android.media.flags.Flags.FLAG_ENABLE_FADE_MANAGER_CONFIGURATION;
+import static android.media.audiopolicy.Flags.FLAG_ENABLE_FADE_MANAGER_CONFIGURATION;
import static org.junit.Assert.assertThrows;