Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 9195d57f authored by Wei Jia's avatar Wei Jia Committed by Android (Google) Code Review
Browse files

Merge "NuPlayer: set video surface directly when NuPlayer has not started yet." into mnc-dev

parents 9872cdce 6b7d2ed4

media/libmediaplayerservice/nuplayer/NuPlayer.cpp

+5 −2
Original line number Diff line number Diff line
@@ -632,10 +632,13 @@ void NuPlayer::onMessageReceived(const sp<AMessage> &msg) { 


            ALOGD("onSetVideoSurface(%p, %s video decoder)",

                    surface.get(),

                    (mSource != NULL && mSource->getFormat(false /* audio */) != NULL

                    (mSource != NULL && mStarted && mSource->getFormat(false /* audio */) != NULL

                            && mVideoDecoder != NULL) ? "have" : "no");



            if (mSource == NULL || mSource->getFormat(false /* audio */) == NULL

            // Need to check mStarted before calling mSource->getFormat because NuPlayer might

            // be in preparing state and it could take long time.

            // When mStarted is true, mSource must have been set.

            if (mSource == NULL || !mStarted || mSource->getFormat(false /* audio */) == NULL

                    // NOTE: mVideoDecoder's mSurface is always non-null

                    || (mVideoDecoder != NULL && mVideoDecoder->setVideoSurface(surface) == OK)) {

                performSetSurface(surface);