videoaggregator: push the caps event as soon as we receive it
authorMatthew Waters <ystreet00@gmail.com>
Thu, 7 Aug 2014 09:54:36 +0000 (19:54 +1000)
committerTim-Philipp Müller <tim@centricular.com>
Sat, 2 Dec 2017 15:10:25 +0000 (15:10 +0000)
Along with the required mandatory dependent events.

Some elements need to perform an allocation query inside
::negotiated_caps().  Without the caps event being sent prior,
downstream elements will be unable to answer and will return
an error.

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

libs/gst/base/gstaggregator.c

index 9274600..0868d53 100644 (file)
@@ -277,19 +277,6 @@ _check_all_pads_with_data_or_eos (GstAggregator * self,
   return FALSE;
 }
 
-/**
- * gst_aggregator_set_src_caps:
- * @self: The #GstAggregator
- * @caps: The #GstCaps to set later on the src pad.
- *
- * Sets the caps to be used on the src pad.
- */
-void
-gst_aggregator_set_src_caps (GstAggregator * self, GstCaps * caps)
-{
-  gst_caps_replace (&self->priv->srccaps, caps);
-}
-
 static void
 _reset_flow_values (GstAggregator * self)
 {
@@ -351,6 +338,20 @@ _push_mandatory_events (GstAggregator * self)
 }
 
 /**
+ * gst_aggregator_set_src_caps:
+ * @self: The #GstAggregator
+ * @caps: The #GstCaps to set later on the src pad.
+ *
+ * Sets the caps to be used on the src pad.
+ */
+void
+gst_aggregator_set_src_caps (GstAggregator * self, GstCaps * caps)
+{
+  gst_caps_replace (&self->priv->srccaps, caps);
+  _push_mandatory_events (self);
+}
+
+/**
  * gst_aggregator_finish_buffer:
  * @self: The #GstAggregator
  * @buffer: the #GstBuffer to push.