From 064bb6a090471c473e4641e28296854eb9850cf1 Mon Sep 17 00:00:00 2001
From: Sharad Sangle <assangle@codeaurora.org>
Date: Tue, 7 Apr 2015 15:04:55 +0530
Subject: NuPlayer: Don't maintain timeStamp if state is running

  While offload playback is going on, if user seeks
  to new position and after playback for some time,
  pauses till offload tear-down happens then playback
  resumes with old seek position

  The book-keeping of seektime for start of playback
  mStartupSeekTimeUs is also done in running state,
  it should be done only if current state is paused.

Change-Id: I4173a2ee4244bcc6794822dde24c467f0189b84f
---
 media/libmediaplayerservice/nuplayer/NuPlayerDriver.cpp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

(limited to 'media/libmediaplayerservice/nuplayer')

diff --git a/media/libmediaplayerservice/nuplayer/NuPlayerDriver.cpp b/media/libmediaplayerservice/nuplayer/NuPlayerDriver.cpp
index b7af61b..7c71e4e 100644
--- a/media/libmediaplayerservice/nuplayer/NuPlayerDriver.cpp
+++ b/media/libmediaplayerservice/nuplayer/NuPlayerDriver.cpp
@@ -409,7 +409,9 @@ status_t NuPlayerDriver::seekTo(int msec) {
         {
             mAtEOS = false;
             mSeekInProgress = true;
-            mStartupSeekTimeUs = seekTimeUs;
+            if (mState == STATE_PAUSED) {
+               mStartupSeekTimeUs = seekTimeUs;
+            }
             // seeks can take a while, so we essentially paused
             notifyListener_l(MEDIA_PAUSED);
             mPlayer->seekToAsync(seekTimeUs, true /* needNotify */);
-- 
cgit v1.1