commit 71cbb9d4e4757033b23a0ba600cba4e279d5c73f
author Harish Mahendrakar <harish.mahendrakar@ittiam.com> Tue May 21 11:21:27 2019 -0700
committer Wonsik Kim <wonsik@google.com> Tue May 28 11:45:29 2019 -0700
tree d6749eac402022f6fe083c9ddbcba1d01ddc3090
parent 88665c482f6503667389c314594ab5e9f990b1f4

CCodecConfig: Fix mapping to bitrate-mode

bitrate-mode was not mapped from c2 params to sdk params

Also fix typecast of bitrate mode while checking config params
for quality/bitrate. It was wrongly typecast to c2 enum

Test: Manual using a test application using AMediaCodec api
Bug: 133241887

Change-Id: I2544a7cbb4001152197b9ec164fe94b1a6da40d3

media/codec2/sfplugin/CCodec.cpp

diff --git a/media/codec2/sfplugin/CCodec.cpp b/media/codec2/sfplugin/CCodec.cpp
index 85c783b..400b02f 100644
--- a/media/codec2/sfplugin/CCodec.cpp
+++ b/media/codec2/sfplugin/CCodec.cpp
@@ -745,11 +745,8 @@
                 return BAD_VALUE;
             }
             if ((config->mDomain & Config::IS_ENCODER) && (config->mDomain & Config::IS_VIDEO)) {
-                C2Config::bitrate_mode_t mode = C2Config::BITRATE_VARIABLE;
-                if (msg->findInt32(KEY_BITRATE_MODE, &i32)) {
-                    mode = (C2Config::bitrate_mode_t) i32;
-                }
-                if (mode == BITRATE_MODE_CQ) {
+                int32_t mode = BITRATE_MODE_VBR;
+                if (msg->findInt32(KEY_BITRATE_MODE, &mode) && mode == BITRATE_MODE_CQ) {
                     if (!msg->findInt32(KEY_QUALITY, &i32)) {
                         ALOGD("quality is missing, which is required for video encoders in CQ.");
                         return BAD_VALUE;