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

Commit 07b96980 authored by Satya Krishna Pindiproli's avatar Satya Krishna Pindiproli Committed by Steve Kondik
Browse files

mediaplayerservice: reset mBytesWritten in AudioSink flush and stop 

For non-offload cases, when repeat mode is set to repeat track,
post a seek operation, progress bar does not move properly after
playback of current track is complete.

When audioSink is re-opened to replay the track (some apps do not
set the looping flag due to which track is re-opened),
mAudioSink::getFramesWritten() returns an invalid value due to which
calculation of positionUs in getCurrentPosition goes awry.

Reset mBytesWritten in MediaPlayerService in flush() and stop()
so that the correct number of frames written is returned when
queried by NuPlayerRenderer.

CRs-Fixed: 866102
Change-Id: Ic974c168062328565548ae35e6db50348e86b3f0
parent eae6ae14

media/libmediaplayerservice/MediaPlayerService.cpp

+2 −0
Original line number Diff line number Diff line
@@ -1965,12 +1965,14 @@ ssize_t MediaPlayerService::AudioOutput::write(const void* buffer, size_t size) 
void MediaPlayerService::AudioOutput::stop()

{

    ALOGV("stop");

    mBytesWritten = 0;

    if (mTrack != 0) mTrack->stop();

}



void MediaPlayerService::AudioOutput::flush()

{

    ALOGV("flush");

    mBytesWritten = 0;

    if (mTrack != 0) mTrack->flush();

}