commit 06f1f0de057c925f3c1671c6094256fa5619ce43
author Wonsik Kim <wonsik@google.com> Mon Jul 26 11:13:05 2021 -0700
committer Wonsik Kim <wonsik@google.com> Mon Jul 26 11:15:17 2021 -0700
tree 95bc62e4a1ada12999693cc9dc20ba731db2adaa
parent 96e879f3910775c36e13a6adefd1eaa6fe92f88f

Revert "CCodec: don't drop metadata for output with flag FLAG_DROP_FRAME."

This reverts commit b8fe01e5992d688987637fd7cbbbbe617293a9da.

Bug: 193482724
Bug: 193867146
Test: cts/media/device-small
Change-Id: I00375414c2e0491f4af4681858eae17e0dc43be6

media/codec2/sfplugin/CCodecBufferChannel.cpp

diff --git a/media/codec2/sfplugin/CCodecBufferChannel.cpp b/media/codec2/sfplugin/CCodecBufferChannel.cpp
index 9ac74c2..e9adfc9 100644
--- a/media/codec2/sfplugin/CCodecBufferChannel.cpp
+++ b/media/codec2/sfplugin/CCodecBufferChannel.cpp
@@ -1881,13 +1881,7 @@
         }
     }
 
-    bool drop = false;
-    if (worklet->output.flags & C2FrameData::FLAG_DROP_FRAME) {
-        ALOGV("[%s] onWorkDone: drop buffer but keep metadata", mName);
-        drop = true;
-    }
-
-    if (notifyClient && !buffer && !flags && !drop) {
+    if (notifyClient && !buffer && !flags) {
         ALOGV("[%s] onWorkDone: Not reporting output buffer (%lld)",
               mName, work->input.ordinal.frameIndex.peekull());
         notifyClient = false;