commit b75de9d4ba6725104342b5376a1047ff1c9b2765
author Ronghua Wu <ronghuawu@google.com> Wed Oct 07 16:52:19 2015 -0700
committer Ronghua Wu <ronghuawu@google.com> Tue Oct 27 16:47:34 2015 -0700
tree 94245eca48717e67225a5fc4c5ecfc371c794e12
parent acd2186aa239efa58b202823083c1b8dbc1c8a16

nuplayer: don't resume if player is reseting

Bug: 24712426
Change-Id: I58cca0bde708a5c3b073ed77e18b62fe81d6c017

media/libmediaplayerservice/nuplayer/NuPlayer.cpp

diff --git a/media/libmediaplayerservice/nuplayer/NuPlayer.cpp b/media/libmediaplayerservice/nuplayer/NuPlayer.cpp
index c4f20da..3ce0c7d 100644
--- a/media/libmediaplayerservice/nuplayer/NuPlayer.cpp
+++ b/media/libmediaplayerservice/nuplayer/NuPlayer.cpp
@@ -188,6 +188,7 @@
       mPlaybackSettings(AUDIO_PLAYBACK_RATE_DEFAULT),
       mVideoFpsHint(-1.f),
       mStarted(false),
+      mResetting(false),
       mSourceStarted(false),
       mPaused(false),
       mPausedByClient(false),
@@ -1139,6 +1140,8 @@
         {
             ALOGV("kWhatReset");
 
+            mResetting = true;
+
             mDeferredActions.push_back(
                     new FlushDecoderAction(
                         FLUSH_CMD_SHUTDOWN /* audio */,
@@ -1221,7 +1224,8 @@
 }
 
 void NuPlayer::onResume() {
-    if (!mPaused) {
+    if (!mPaused || mResetting) {
+        ALOGD_IF(mResetting, "resetting, onResume discarded");
         return;
     }
     mPaused = false;
@@ -1922,6 +1926,7 @@
     }
 
     mStarted = false;
+    mResetting = false;
     mSourceStarted = false;
 }