commit 0342ef4098002142c34a647d46c3e5fe935fd6ac
author Ray Essick <essick@google.com> Tue Jan 12 16:57:58 2021 -0800
committer Ray Essick <essick@google.com> Wed Mar 10 09:13:36 2021 -0800
tree 5e86b685a146d7363c7c5c6458bfd0860f7c15c0
parent cb4fc2b79d3deb764609526c46a7058d6867a16a

NuPlayer uses format

have nuplayer pass format when getting codec; mediacodecinfo uses that
to screen the candidate codecs.
This helps with situations such as high min-size hardware codecs getting
chosen for small thumbnail-like video clips.... which then won't decode.

Bug: 71392444
Test: adjust HW codec config out of range, see that we failover to the sw codec.
Change-Id: I572d2dd6d8c1cc040543fdba01a88cc66f447a4e

media/libmediaplayerservice/nuplayer/NuPlayerDecoder.cpp

diff --git a/media/libmediaplayerservice/nuplayer/NuPlayerDecoder.cpp b/media/libmediaplayerservice/nuplayer/NuPlayerDecoder.cpp
index 5b60bbf..2c1f158 100644
--- a/media/libmediaplayerservice/nuplayer/NuPlayerDecoder.cpp
+++ b/media/libmediaplayerservice/nuplayer/NuPlayerDecoder.cpp
@@ -302,7 +302,7 @@
     ALOGV("[%s] onConfigure (surface=%p)", mComponentName.c_str(), mSurface.get());
 
     mCodec = MediaCodec::CreateByType(
-            mCodecLooper, mime.c_str(), false /* encoder */, NULL /* err */, mPid, mUid);
+            mCodecLooper, mime.c_str(), false /* encoder */, NULL /* err */, mPid, mUid, format);
     int32_t secure = 0;
     if (format->findInt32("secure", &secure) && secure != 0) {
         if (mCodec != NULL) {