commit bf8986159e7615cfe2e1403776634e6c211354f8
author Wei Jia <wjia@google.com> Fri Nov 04 16:15:57 2016 -0700
committer Wei Jia <wjia@google.com> Fri Nov 04 16:15:57 2016 -0700
tree 3a53055793d610b2a6eb118c8022926b4cf083a2
parent d592974f4e1d8afc753ada8c10d2b252d8f852af

NuPlayerDecoder: flush audio buffers when audio output format is changed.

Test: compiles
Bug: 32659046
Change-Id: I8e771b8ee22520698427854c1850d8ee7f828765

media/libmediaplayerservice/nuplayer/NuPlayerDecoder.cpp

diff --git a/media/libmediaplayerservice/nuplayer/NuPlayerDecoder.cpp b/media/libmediaplayerservice/nuplayer/NuPlayerDecoder.cpp
index 6974207..d2452af 100644
--- a/media/libmediaplayerservice/nuplayer/NuPlayerDecoder.cpp
+++ b/media/libmediaplayerservice/nuplayer/NuPlayerDecoder.cpp
@@ -711,6 +711,10 @@
             flags = AUDIO_OUTPUT_FLAG_NONE;
         }
 
+        // TODO: This is a temporary fix to flush audio buffers in renderer. The real
+        // fix should be to wait for all buffers rendered normally, then open a new
+        // AudioSink.
+        mRenderer->flush(true /* audio */, false /* notifyComplete */);
         status_t err = mRenderer->openAudioSink(
                 format, false /* offloadOnly */, hasVideo, flags, NULL /* isOffloaed */);
         if (err != OK) {