diff options
author | Andy Hung <hunga@google.com> | 2014-12-19 19:57:48 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2014-12-19 19:57:48 +0000 |
commit | d4b056da7cbf1adb2a5f668e4425def3754a9c09 (patch) | |
tree | 2f8a28d6432e8a6607fc55391e93031cbec82d97 /media | |
parent | 89ee55cb01f6e230990eb344718627c8fcbec349 (diff) | |
parent | 3491232a7c0d953fa021f6a81baee64c44f364f3 (diff) | |
download | frameworks_av-d4b056da7cbf1adb2a5f668e4425def3754a9c09.zip frameworks_av-d4b056da7cbf1adb2a5f668e4425def3754a9c09.tar.gz frameworks_av-d4b056da7cbf1adb2a5f668e4425def3754a9c09.tar.bz2 |
Merge "NuPlayer: Fix failure to stop audio playback" into lmp-mr1-dev
Diffstat (limited to 'media')
-rw-r--r-- | media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp b/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp index 21b74ee..d6a828b 100644 --- a/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp +++ b/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp @@ -638,10 +638,13 @@ bool NuPlayer::Renderer::onDrainAudioQueue() { mAudioQueue.erase(mAudioQueue.begin()); entry = NULL; - // Need to stop the track here, because that will play out the last - // little bit at the end of the file. Otherwise short files won't play. - mAudioSink->stop(); - mNumFramesWritten = 0; + if (mAudioSink->needsTrailingPadding()) { + // If we're not in gapless playback (i.e. through setNextPlayer), we + // need to stop the track here, because that will play out the last + // little bit at the end of the file. Otherwise short files won't play. + mAudioSink->stop(); + mNumFramesWritten = 0; + } return false; } |