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

Commit 8420073f authored by Andy Hung's avatar Andy Hung
Browse files

NuPlayerRenderer: Allow immediate pause on next track. 

Prior code required audio data to have been decoded first.

Bug: 25904185
Change-Id: I52fa65b2ed53d29ddd534c2bf3070626d54bd633
parent 6411cb68

media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp

+6 −6
Original line number Diff line number Diff line
@@ -1547,10 +1547,9 @@ void NuPlayer::Renderer::onPause() { 
    mDrainAudioQueuePending = false;

    mDrainVideoQueuePending = false;



    if (mHasAudio) {

    // Note: audio data may not have been decoded, and the AudioSink may not be opened.

    mAudioSink->pause();

    startAudioOffloadPauseTimeout();

    }



    ALOGV("now paused audio queue has %zu entries, video has %zu entries",

          mAudioQueue.size(), mVideoQueue.size());
@@ -1561,8 +1560,9 @@ void NuPlayer::Renderer::onResume() { 
        return;

    }



    if (mHasAudio) {

    // Note: audio data may not have been decoded, and the AudioSink may not be opened.

    cancelAudioOffloadPauseTimeout();

    if (mAudioSink->ready()) {

        status_t err = mAudioSink->start();

        if (err != OK) {

            ALOGE("cannot start AudioSink err %d", err);