vp8decoder: Drain the output queue on EOS/finish

The finish() virtual method was flushing the queue, instead push the
remaining buffers. It is not required to reset in finish() unlike
drain(). This a regression causing last frame to always be lost.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/merge_requests/2333>

diff --git a/gst-libs/gst/codecs/gstvp8decoder.c b/gst-libs/gst/codecs/gstvp8decoder.c
index d3f2aa1..2a9f834 100644 (file)
--- a/gst-libs/gst/codecs/gstvp8decoder.c
+++ b/gst-libs/gst/codecs/gstvp8decoder.c
@@ -279,7 +279,7 @@ gst_vp8_decoder_finish (GstVideoDecoder * decoder)
 
   GST_DEBUG_OBJECT (self, "finish");
 
-  gst_vp8_decoder_reset (self);
+  gst_vp8_decoder_drain_output_queue (GST_VP8_DECODER (decoder), 0);
 
   return GST_FLOW_OK;
 }