diff options
author | Ronghua Wu <ronghuawu@google.com> | 2014-11-20 21:13:08 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2014-11-20 21:13:08 +0000 |
commit | 7966142a096c1d5d7386a42eaccc9aed5b4344b3 (patch) | |
tree | 15a1da95e59cfc2506f73f10dab7fdcec432b4a8 /media | |
parent | f4ea51fda9396d7ee851b1d4d220384210ea2336 (diff) | |
parent | f83408b41bbd796b7923d719e7e3799ddc7acaff (diff) | |
download | frameworks_av-7966142a096c1d5d7386a42eaccc9aed5b4344b3.zip frameworks_av-7966142a096c1d5d7386a42eaccc9aed5b4344b3.tar.gz frameworks_av-7966142a096c1d5d7386a42eaccc9aed5b4344b3.tar.bz2 |
Merge "NuPlayer: invalid anchor time when render is flushed." into lmp-mr1-dev
Diffstat (limited to 'media')
-rw-r--r-- | media/libmediaplayerservice/nuplayer/NuPlayerDriver.cpp | 7 | ||||
-rw-r--r-- | media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp | 1 |
2 files changed, 8 insertions, 0 deletions
diff --git a/media/libmediaplayerservice/nuplayer/NuPlayerDriver.cpp b/media/libmediaplayerservice/nuplayer/NuPlayerDriver.cpp index e09567a..d050c78 100644 --- a/media/libmediaplayerservice/nuplayer/NuPlayerDriver.cpp +++ b/media/libmediaplayerservice/nuplayer/NuPlayerDriver.cpp @@ -310,6 +310,13 @@ status_t NuPlayerDriver::stop() { } status_t NuPlayerDriver::pause() { + // The NuPlayerRenderer may get flushed if pause for long enough, e.g. the pause timeout tear + // down for audio offload mode. If that happens, the NuPlayerRenderer will no longer know the + // current position. So similar to seekTo, update |mPositionUs| to the pause position by calling + // getCurrentPosition here. + int msec; + getCurrentPosition(&msec); + Mutex::Autolock autoLock(mLock); switch (mState) { diff --git a/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp b/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp index 42288a3..faf115e 100644 --- a/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp +++ b/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp @@ -1039,6 +1039,7 @@ void NuPlayer::Renderer::onFlush(const sp<AMessage> &msg) { Mutex::Autolock autoLock(mLock); syncQueuesDone_l(); setPauseStartedTimeRealUs(-1); + setAnchorTime(-1, -1); } ALOGV("flushing %s", audio ? "audio" : "video"); |