commit 1eb88a92ab3fb0a24237bc76ff932e1a9b66bcf9
author Wonsik Kim <wonsik@google.com> Mon Mar 29 20:44:04 2021 -0700
committer Wonsik Kim <wonsik@google.com> Tue Mar 30 18:09:31 2021 -0700
tree adf972b95ac346c98db7a26248cc8b67d1419d03
parent 2f844ad433af0da7749eff0df2bc7ede9d353e44

CCodec: use known default format for bytebuffer mode decoder

Bug: 181386257
Test: manual
Change-Id: Iaadc1eacd9b4f87d14d9bdef62844a90cd829888

media/codec2/sfplugin/CCodec.cpp

diff --git a/media/codec2/sfplugin/CCodec.cpp b/media/codec2/sfplugin/CCodec.cpp
index 60c5901..b203d24 100644
--- a/media/codec2/sfplugin/CCodec.cpp
+++ b/media/codec2/sfplugin/CCodec.cpp
@@ -996,7 +996,15 @@
                 // needed for decoders.
                 if (!(config->mDomain & Config::IS_ENCODER)) {
                     if (surface == nullptr) {
-                        format = flexPixelFormat.value_or(COLOR_FormatYUV420Flexible);
+                        const char *prefix = "";
+                        if (flexSemiPlanarPixelFormat) {
+                            format = COLOR_FormatYUV420SemiPlanar;
+                            prefix = "semi-";
+                        } else {
+                            format = COLOR_FormatYUV420Planar;
+                        }
+                        ALOGD("Client requested ByteBuffer mode decoder w/o color format set: "
+                                "using default %splanar color format", prefix);
                     } else {
                         format = COLOR_FormatSurface;
                     }