commit ad6516d9247bd0a76a393810b041c3e4094f0e36
author Andreas Huber <andih@google.com> Wed Feb 03 10:23:26 2010 -0800
committer Andreas Huber <andih@google.com> Wed Feb 03 10:23:26 2010 -0800
tree 9ddbeb15c8c205420a6deb72d1a8ea01c1db7f32
parent 7b7a83a6840d34ab39f1a3c8ab81946bb4404c14

AudioPlayer didn't properly account for the latency when calculating the current media time.

related-to-bug: 2414900

media/libstagefright/AudioPlayer.cpp

diff --git a/media/libstagefright/AudioPlayer.cpp b/media/libstagefright/AudioPlayer.cpp
index efe7ebb..01578c1 100644
--- a/media/libstagefright/AudioPlayer.cpp
+++ b/media/libstagefright/AudioPlayer.cpp
@@ -249,7 +249,8 @@
                         kKeyTime, &mPositionTimeMediaUs));
 
             mPositionTimeRealUs =
-                ((mNumFramesPlayed + size_done / mFrameSize) * 1000000)
+                -mLatencyUs
+                + ((mNumFramesPlayed + size_done / mFrameSize) * 1000000)
                     / mSampleRate;
 
             LOGV("buffer->size() = %d, "
@@ -297,6 +298,10 @@
 int64_t AudioPlayer::getMediaTimeUs() {
     Mutex::Autolock autoLock(mLock);
 
+    if (mPositionTimeMediaUs < 0 || mPositionTimeRealUs < 0) {
+        return 0;
+    }
+
     return mPositionTimeMediaUs + (getRealTimeUsLocked() - mPositionTimeRealUs);
 }
 
@@ -307,7 +312,7 @@
     *realtime_us = mPositionTimeRealUs;
     *mediatime_us = mPositionTimeMediaUs;
 
-    return mPositionTimeRealUs != -1 || mPositionTimeMediaUs != -1;
+    return mPositionTimeRealUs != -1 && mPositionTimeMediaUs != -1;
 }
 
 status_t AudioPlayer::seekTo(int64_t time_us) {