[gst-cvs] gstreamer: gstpad: Return pad template in get_caps if pad is not negotiable

[<bi...@ke...>]

Module: gstreamer
Branch: master
Commit: 7460321a600438966d7152ab2b4318be48eadce0
URL:    http://cgit.freedesktop.org/gstreamer/gstreamer/commit/?id=7460321a600438966d7152ab2b4318be48eadce0

Author: Edward Hervey <bi...@bi...>
Date:   Mon May 17 15:06:37 2010 +0200

gstpad: Return pad template in get_caps if pad is not negotiable

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

---

 gst/gstpad.c                  |    8 ++++++++
 tests/check/gst/gstghostpad.c |    6 ++++++
 2 files changed, 14 insertions(+), 0 deletions(-)

diff --git a/gst/gstpad.c b/gst/gstpad.c
index 5cd434f..e376857 100644
--- a/gst/gstpad.c
+++ b/gst/gstpad.c
@@ -2181,6 +2181,14 @@ gst_pad_get_caps_unlocked (GstPad * pad)
 
   GST_CAT_DEBUG_OBJECT (GST_CAT_CAPS, pad, "get pad caps");
 
+  /* If the parent element is in GST_STATE_NULL, return the template caps */
+  if (G_UNLIKELY (!GST_PAD_IS_NEGOTIABLE (pad) && GST_PAD_PAD_TEMPLATE (pad))) {
+    GST_CAT_DEBUG_OBJECT (GST_CAT_CAPS, pad,
+        "parent element is in GST_STATE_NULL, using pad templates");
+    result = gst_caps_ref (GST_PAD_TEMPLATE_CAPS (GST_PAD_PAD_TEMPLATE (pad)));
+    goto done;
+  }
+
   if (GST_PAD_GETCAPSFUNC (pad)) {
     GST_CAT_DEBUG_OBJECT (GST_CAT_CAPS, pad,
         "dispatching to pad getcaps function");
diff --git a/tests/check/gst/gstghostpad.c b/tests/check/gst/gstghostpad.c
index b78538a..a752550 100644
--- a/tests/check/gst/gstghostpad.c
+++ b/tests/check/gst/gstghostpad.c
@@ -591,6 +591,9 @@ GST_START_TEST (test_ghost_pads_new_from_template)
   /* check template is properly set */
   fail_unless (GST_PAD_PAD_TEMPLATE (ghostpad) == ghosttempl);
 
+  /* Set pad negotiable so the getcaps call goes through */
+  gst_pad_set_negotiable (ghostpad, TRUE);
+
   /* check ghostpad caps are from the sinkpad */
   newcaps = gst_pad_get_caps (ghostpad);
   fail_unless (newcaps != NULL);
@@ -645,6 +648,9 @@ GST_START_TEST (test_ghost_pads_new_no_target_from_template)
 
   fail_unless (gst_ghost_pad_set_target ((GstGhostPad *) ghostpad, sinkpad));
 
+  /* set pad negotiable so the getcaps call goes through */
+  gst_pad_set_negotiable (ghostpad, TRUE);
+
   /* check ghostpad caps are now from the target pad */
   newcaps = gst_pad_get_caps (ghostpad);
   fail_unless (newcaps != NULL);