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

Commit f0a36003 authored by Zhou Song's avatar Zhou Song Committed by Automerger Merge Worker
Browse files

audio: return accurate frame count when track is flushed am: 8735d0dd 

Original change: https://android-review.googlesource.com/c/platform/frameworks/av/+/1644226

Change-Id: I25326e6f25cbcc4bcdde99b01891d52ff1df98c7
parents c59d0c4e 8735d0dd

media/libaudioclient/AudioTrackShared.cpp

+2 −5
Original line number Diff line number Diff line
@@ -900,11 +900,8 @@ size_t AudioTrackServerProxy::framesReady() 
    }

    audio_track_cblk_t* cblk = mCblk;



    int32_t flush = cblk->u.mStreaming.mFlush;

    if (flush != mFlush) {

        // FIXME should return an accurate value, but over-estimate is better than under-estimate

        return mFrameCount;

    }

    flushBufferIfNeeded();



    const int32_t rear = getRear();

    ssize_t filled = audio_utils::safe_sub_overflow(rear, cblk->u.mStreaming.mFront);

    // pipe should not already be overfull

services/audioflinger/Tracks.cpp

+5 −0
Original line number Diff line number Diff line
@@ -1038,6 +1038,11 @@ status_t AudioFlinger::PlaybackThread::Track::start(AudioSystem::sync_event_t ev 
        // initial state-stopping. next state-pausing.

        // What if resume is called ?



        if (state == FLUSHED) {

            // avoid underrun glitches when starting after flush

            reset();

        }



        if (state == PAUSED || state == PAUSING) {

            if (mResumeToStopping) {

                // happened we need to resume to STOPPING_1