commit c35577ee86001c1bc9253da590f948871006ffdb
author Venkatarama Avadhani <venkatarama.avadhani@ittiam.com> Mon Sep 11 08:47:33 2023 +0530
committer Venkatarama Avadhani <venkatarama.avadhani@ittiam.com> Thu Nov 02 05:25:59 2023 +0000
tree 404d7a818e9f264401c1408dd1deabb8d2767531
parent 0092cae047fa307eeed414624aca20a8adf06465

MetadataRetriever: Use RGBA_8888 as default color format based on a flag

When the flag mediametadataretriever_default_rgba8888 is set, use
RGBA_8888 as the color format in getFrameAtTime call.

Add the unit test to check the intended behaviour.

Test: adb shell setprop persist.device_config.aconfig_flags.\
media_solutions.com.android.media.playback.flags.\
mediametadataretriever_default_rgba8888 true
Test: adb shell device_config put media_solutions \
com.android.media.playback.flags.mediametadataretriever_default_rgba8888 \
true
Test: atest com.android.mediaframeworktest.unit.MediaMetadataRetrieverTest
Bug: 298965955

Change-Id: I8452e75e07bcfcd2483ad2da971806147f5a9067

AconfigFlags.bp

diff --git a/AconfigFlags.bp b/AconfigFlags.bp
index 0265431..45f6570 100644
--- a/AconfigFlags.bp
+++ b/AconfigFlags.bp
@@ -199,6 +199,24 @@
     defaults: ["framework-minus-apex-aconfig-java-defaults"],
 }
 
+// Media
+aconfig_declarations {
+    name: "android.media.playback.flags-aconfig",
+    package: "com.android.media.playback.flags",
+    srcs: ["media/jni/playback_flags.aconfig"],
+}
+
+cc_aconfig_library {
+    name: "android.media.playback.flags-aconfig-cc",
+    aconfig_declarations: "android.media.playback.flags-aconfig",
+}
+
+java_aconfig_library {
+    name: "android.media.playback.flags-aconfig-java",
+    aconfig_declarations: "android.media.playback.flags-aconfig",
+    defaults: ["framework-minus-apex-aconfig-java-defaults"],
+}
+
 // VCN
 aconfig_declarations {
     name: "android.net.vcn.flags-aconfig",