basesink: Make sure we never drop the preroll buffer

This is cosmetic as 'late' should never be set during preroll (in pause).
Though code may evolve in the future, so this is good for preventing
potential bugs.

https://bugzilla.gnome.org/show_bug.cgi?id=772468

diff --git a/libs/gst/base/gstbasesink.c b/libs/gst/base/gstbasesink.c
index 97e0573..aef33fd 100644 (file)
--- a/libs/gst/base/gstbasesink.c
+++ b/libs/gst/base/gstbasesink.c
@@ -3476,15 +3476,15 @@ gst_base_sink_chain_unlocked (GstBaseSink * basesink, GstPad * pad,
       }
     }
 
-    if (G_UNLIKELY (late))
-      goto dropped;
-
     /* We are about to prepare the first frame, make sure we have prerolled
      * already. This prevent nesting prepare/render calls. */
     ret = gst_base_sink_do_preroll (basesink, obj);
     if (G_UNLIKELY (ret != GST_FLOW_OK))
       goto preroll_failed;
 
+    if (G_UNLIKELY (late))
+      goto dropped;
+
     if (!is_list) {
       if (bclass->prepare) {
         ret = bclass->prepare (basesink, GST_BUFFER_CAST (obj));