diff options
author | Wei Jia <wjia@google.com> | 2015-01-28 03:25:33 +0000 |
---|---|---|
committer | Android Git Automerger <android-git-automerger@android.com> | 2015-01-28 03:25:33 +0000 |
commit | 2ac21a5d35591acc7f7ff4f85fda146ebeaa2551 (patch) | |
tree | 4d5b3052cedb8b4b7878f24f69394861cf3eb322 /media/libmediaplayerservice | |
parent | 3e68c081d905e3164e11797287484beb68747ab5 (diff) | |
parent | 5fcd4d19e95d5237fef46b1dec8b6f649ea64b84 (diff) | |
download | frameworks_av-2ac21a5d35591acc7f7ff4f85fda146ebeaa2551.zip frameworks_av-2ac21a5d35591acc7f7ff4f85fda146ebeaa2551.tar.gz frameworks_av-2ac21a5d35591acc7f7ff4f85fda146ebeaa2551.tar.bz2 |
am 5fcd4d19: am 0bd5274e: am 231406d5: NuPlayerRenderer: return current real time as render time for video frame when paused.
* commit '5fcd4d19e95d5237fef46b1dec8b6f649ea64b84':
NuPlayerRenderer: return current real time as render time for video frame when paused.
Diffstat (limited to 'media/libmediaplayerservice')
-rw-r--r-- | media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp | 31 | ||||
-rw-r--r-- | media/libmediaplayerservice/nuplayer/NuPlayerRenderer.h | 2 |
2 files changed, 18 insertions, 15 deletions
diff --git a/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp b/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp index 80b8493..04e52b8 100644 --- a/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp +++ b/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp @@ -424,7 +424,8 @@ void NuPlayer::Renderer::onMessageReceived(const sp<AMessage> &msg) { onDrainVideoQueue(); - postDrainVideoQueue(); + Mutex::Autolock autoLock(mLock); + postDrainVideoQueue_l(); break; } @@ -437,7 +438,8 @@ void NuPlayer::Renderer::onMessageReceived(const sp<AMessage> &msg) { } mDrainVideoQueuePending = false; - postDrainVideoQueue(); + Mutex::Autolock autoLock(mLock); + postDrainVideoQueue_l(); break; } @@ -766,7 +768,7 @@ int64_t NuPlayer::Renderer::getPendingAudioPlayoutDurationUs(int64_t nowUs) { int64_t NuPlayer::Renderer::getRealTimeUs(int64_t mediaTimeUs, int64_t nowUs) { int64_t currentPositionUs; - if (getCurrentPositionOnLooper( + if (mPaused || getCurrentPositionOnLooper( ¤tPositionUs, nowUs, true /* allowPastQueuedVideo */) != OK) { // If failed to get current position, e.g. due to audio clock is not ready, then just // play out video immediately without delay. @@ -787,7 +789,7 @@ void NuPlayer::Renderer::onNewAudioMediaTime(int64_t mediaTimeUs) { mediaTimeUs, nowUs + getPendingAudioPlayoutDurationUs(nowUs), mNumFramesWritten); } -void NuPlayer::Renderer::postDrainVideoQueue() { +void NuPlayer::Renderer::postDrainVideoQueue_l() { if (mDrainVideoQueuePending || mSyncQueues || (mPaused && mVideoSampleReceived)) { @@ -823,6 +825,7 @@ void NuPlayer::Renderer::postDrainVideoQueue() { if (mAnchorTimeMediaUs < 0) { setAnchorTime(mediaTimeUs, nowUs); + mPausePositionMediaTimeUs = mediaTimeUs; mAnchorMaxMediaUs = mediaTimeUs; realTimeUs = nowUs; } else { @@ -986,16 +989,15 @@ void NuPlayer::Renderer::onQueueBuffer(const sp<AMessage> &msg) { entry.mFinalResult = OK; entry.mBufferOrdinal = ++mTotalBuffersQueued; + Mutex::Autolock autoLock(mLock); if (audio) { - Mutex::Autolock autoLock(mLock); mAudioQueue.push_back(entry); postDrainAudioQueue_l(); } else { mVideoQueue.push_back(entry); - postDrainVideoQueue(); + postDrainVideoQueue_l(); } - Mutex::Autolock autoLock(mLock); if (!mSyncQueues || mAudioQueue.empty() || mVideoQueue.empty()) { return; } @@ -1044,7 +1046,7 @@ void NuPlayer::Renderer::syncQueuesDone_l() { } if (!mVideoQueue.empty()) { - postDrainVideoQueue(); + postDrainVideoQueue_l(); } } @@ -1063,8 +1065,8 @@ void NuPlayer::Renderer::onQueueEOS(const sp<AMessage> &msg) { entry.mOffset = 0; entry.mFinalResult = finalResult; + Mutex::Autolock autoLock(mLock); if (audio) { - Mutex::Autolock autoLock(mLock); if (mAudioQueue.empty() && mSyncQueues) { syncQueuesDone_l(); } @@ -1072,11 +1074,10 @@ void NuPlayer::Renderer::onQueueEOS(const sp<AMessage> &msg) { postDrainAudioQueue_l(); } else { if (mVideoQueue.empty() && mSyncQueues) { - Mutex::Autolock autoLock(mLock); syncQueuesDone_l(); } mVideoQueue.push_back(entry); - postDrainVideoQueue(); + postDrainVideoQueue_l(); } } @@ -1229,18 +1230,20 @@ void NuPlayer::Renderer::onPause() { return; } int64_t currentPositionUs; + int64_t pausePositionMediaTimeUs; if (getCurrentPositionFromAnchor( ¤tPositionUs, ALooper::GetNowUs()) == OK) { - mPausePositionMediaTimeUs = currentPositionUs; + pausePositionMediaTimeUs = currentPositionUs; } else { // Set paused position to -1 (unavailabe) if we don't have anchor time // This could happen if client does a seekTo() immediately followed by // pause(). Renderer will be flushed with anchor time cleared. We don't // want to leave stale value in mPausePositionMediaTimeUs. - mPausePositionMediaTimeUs = -1; + pausePositionMediaTimeUs = -1; } { Mutex::Autolock autoLock(mLock); + mPausePositionMediaTimeUs = pausePositionMediaTimeUs; ++mAudioQueueGeneration; ++mVideoQueueGeneration; prepareForMediaRenderingStart(); @@ -1284,7 +1287,7 @@ void NuPlayer::Renderer::onResume() { } if (!mVideoQueue.empty()) { - postDrainVideoQueue(); + postDrainVideoQueue_l(); } } diff --git a/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.h b/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.h index 23907c8..98fc44d 100644 --- a/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.h +++ b/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.h @@ -212,7 +212,7 @@ private: int64_t getRealTimeUs(int64_t mediaTimeUs, int64_t nowUs); void onDrainVideoQueue(); - void postDrainVideoQueue(); + void postDrainVideoQueue_l(); void prepareForMediaRenderingStart(); void notifyIfMediaRenderingStarted(); |