commit 810eace3c27c6e621523d216fa0b7c23f1a7c748
author Sungtak Lee <taklee@google.com> Mon Apr 12 22:16:44 2021 -0700
committer Sungtak Lee <taklee@google.com> Fri Apr 16 16:36:37 2021 -0700
tree 08c73f9eadf18ff6e5c1085fc2933ac66600f5ee
parent c293b92613d9afde90250400c342ae4f9edd59f3

Codec2 Components: reset OutputBlockPool after use

Since SimpleC2Component is holding the reference of output BlockPool,
destroying output BlockPool and creating a new BlockPool does not work.

Reset the reference of output BlockPool after
use(reset()/stop()/release()).

Bug: 184049126
Test: atest CtsMediaTestCases:android.media.cts.AdaptivePlaybackTest
Change-Id: I75fe1555533d75aa0f742e59992ae033f4454160

media/codec2/components/base/SimpleC2Component.cpp

diff --git a/media/codec2/components/base/SimpleC2Component.cpp b/media/codec2/components/base/SimpleC2Component.cpp
index fb3fbd0..dfad226 100644
--- a/media/codec2/components/base/SimpleC2Component.cpp
+++ b/media/codec2/components/base/SimpleC2Component.cpp
@@ -110,17 +110,20 @@
         }
         case kWhatStop: {
             int32_t err = thiz->onStop();
+            thiz->mOutputBlockPool.reset();
             Reply(msg, &err);
             break;
         }
         case kWhatReset: {
             thiz->onReset();
+            thiz->mOutputBlockPool.reset();
             mRunning = false;
             Reply(msg);
             break;
         }
         case kWhatRelease: {
             thiz->onRelease();
+            thiz->mOutputBlockPool.reset();
             mRunning = false;
             Reply(msg);
             break;