commit c78ff99f29698b6b714dc8c71a43af9c6f323f6b
author Sungtak Lee <taklee@google.com> Mon Sep 19 23:02:16 2022 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Mon Sep 19 23:02:16 2022 +0000
tree b948a81313235f712674969f1f662130c8822755
parent f89a300395b9e03f6611242f5e78b6d861e568e8
parent c0fb0446de0a4fbd80a23531163fad5ddb56934c

CCodec: Delay stop using surface until the end of release() am: 58c3812982 am: c0fb0446de

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/av/+/19980354

Change-Id: Ib51f1cf9e1a1d7e17742ba2137f2028361e50c9b
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

media/codec2/sfplugin/CCodec.cpp

diff --git a/media/codec2/sfplugin/CCodec.cpp b/media/codec2/sfplugin/CCodec.cpp
index 124a893..72301db 100644
--- a/media/codec2/sfplugin/CCodec.cpp
+++ b/media/codec2/sfplugin/CCodec.cpp
@@ -1973,7 +1973,6 @@
     }
 
     mChannel->reset();
-    mChannel->stopUseOutputSurface();
     // thiz holds strong ref to this while the thread is running.
     sp<CCodec> thiz(this);
     std::thread([thiz, sendCallback] { thiz->release(sendCallback); }).detach();
@@ -1994,6 +1993,7 @@
         comp = state->comp;
     }
     comp->release();
+    mChannel->stopUseOutputSurface();
 
     {
         Mutexed<State>::Locked state(mState);