From 140dca43f35d5438d8c5300bd150e9fb69beea59 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Sebastian=20Dr=C3=B6ge?= <sebastian.droege@collabora.co.uk>
Date: Tue, 15 Feb 2011 17:46:22 +0100
Subject: [PATCH] playbin2: Optimize autoplug-continue handler a bit

Don't build merge the caps of all sinks but check them one-by-one
until one supports the caps. Also get reffed caps from the sinkpads
instead of a writable copy and add debug output if a sink claims to
support ANY caps.
---
 gst/playback/gstplaybin2.c | 119 ++++++++++++++++++++++++---------------------
 1 file changed, 64 insertions(+), 55 deletions(-)

diff --git a/gst/playback/gstplaybin2.c b/gst/playback/gstplaybin2.c
index 5fce8ab..54b9c1d 100644
--- a/gst/playback/gstplaybin2.c
+++ b/gst/playback/gstplaybin2.c
@@ -2930,79 +2930,88 @@ static gboolean
 autoplug_continue_cb (GstElement * element, GstPad * pad, GstCaps * caps,
     GstSourceGroup * group)
 {
-  GstCaps *stopcaps = NULL;
   gboolean ret = FALSE;
-  GstElement *text_sink;
-  GstPad *text_sinkpad = NULL;
+  GstElement *sink;
+  GstPad *sinkpad = NULL;
+  GstCaps *sinkcaps = NULL;
 
-  text_sink =
-      (group->playbin->text_sink) ? gst_object_ref (group->
-      playbin->text_sink) : NULL;
-  if (text_sink)
-    text_sinkpad = gst_element_get_static_pad (text_sink, "sink");
+  GST_PLAY_BIN_LOCK (group->playbin);
+  GST_SOURCE_GROUP_LOCK (group);
 
-  if (text_sinkpad) {
-    stopcaps = gst_pad_get_caps_reffed (text_sinkpad);
-    gst_object_unref (text_sinkpad);
+  if ((sink = group->playbin->text_sink))
+    sinkpad = gst_element_get_static_pad (sink, "sink");
+  if (sinkpad) {
+    sinkcaps = gst_pad_get_caps_reffed (sinkpad);
+    gst_object_unref (sinkpad);
 
     /* If the textsink claims to support ANY subcaps,
-     * go the save way and only use the plaintext caps */
-    if (gst_caps_is_any (stopcaps)) {
+     * go the safe way and only use the plaintext caps */
+    if (gst_caps_is_any (sinkcaps)) {
       GST_WARNING_OBJECT (group->playbin, "Text sink '%s' accepts ANY caps",
-          GST_OBJECT_NAME (text_sink));
-      gst_caps_unref (stopcaps);
-      stopcaps = gst_static_caps_get (&sub_plaintext_caps);
+          GST_OBJECT_NAME (sink));
+      gst_caps_unref (sinkcaps);
+      sinkcaps = gst_static_caps_get (&sub_plaintext_caps);
     }
   } else {
-    stopcaps = gst_subtitle_overlay_create_factory_caps ();
+    sinkcaps = gst_subtitle_overlay_create_factory_caps ();
   }
+  ret = !gst_caps_can_intersect (caps, sinkcaps);
+  gst_caps_unref (sinkcaps);
+  sinkcaps = NULL;
+  if (!ret)
+    goto done;
 
-  if (text_sink)
-    gst_object_unref (text_sink);
-
-  GST_PLAY_BIN_LOCK (group->playbin);
-  GST_SOURCE_GROUP_LOCK (group);
-  if (group->playbin->audio_sink) {
-    GstCaps *audio_sinkcaps;
-    GstPad *audio_sinkpad = NULL;
-
-    audio_sinkpad =
-        gst_element_get_static_pad (group->playbin->audio_sink, "sink");
-    if (audio_sinkpad) {
-      audio_sinkcaps = gst_pad_get_caps (audio_sinkpad);
-      gst_object_unref (audio_sinkpad);
-
-      stopcaps = gst_caps_make_writable (stopcaps);
-      if (!gst_caps_is_any (audio_sinkcaps))
-        gst_caps_merge (stopcaps, audio_sinkcaps);
-      else
-        gst_caps_unref (audio_sinkcaps);
+  if ((sink = group->playbin->audio_sink)) {
+    sinkpad = gst_element_get_static_pad (sink, "sink");
+    if (sinkpad) {
+      sinkcaps = gst_pad_get_caps_reffed (sinkpad);
+      gst_object_unref (sinkpad);
+
+      /* If the audio sink claims to support ANY caps go
+       * the safe way and only use the raw audio caps
+       * that decodebin2 already has */
+      if (gst_caps_is_any (sinkcaps)) {
+        GST_WARNING_OBJECT (group->playbin, "Audio sink '%s' accepts ANY caps",
+            GST_OBJECT_NAME (sink));
+        gst_caps_unref (sinkcaps);
+        sinkcaps = NULL;
+      }
     }
   }
+  if (sinkcaps) {
+    ret = !gst_caps_can_intersect (caps, sinkcaps);
+    gst_caps_unref (sinkcaps);
+    sinkcaps = NULL;
+  }
+  if (!ret)
+    goto done;
 
-  if (group->playbin->video_sink) {
-    GstCaps *video_sinkcaps;
-    GstPad *video_sinkpad = NULL;
-
-    video_sinkpad =
-        gst_element_get_static_pad (group->playbin->video_sink, "sink");
-    if (video_sinkpad) {
-      video_sinkcaps = gst_pad_get_caps (video_sinkpad);
-      gst_object_unref (video_sinkpad);
-
-      stopcaps = gst_caps_make_writable (stopcaps);
-      if (!gst_caps_is_any (video_sinkcaps))
-        gst_caps_merge (stopcaps, video_sinkcaps);
-      else
-        gst_caps_unref (video_sinkcaps);
+  if ((sink = group->playbin->video_sink)) {
+    sinkpad = gst_element_get_static_pad (sink, "sink");
+    if (sinkpad) {
+      sinkcaps = gst_pad_get_caps_reffed (sinkpad);
+      gst_object_unref (sinkpad);
+
+      /* If the video sink claims to support ANY caps go
+       * the safe way and only use the raw video caps
+       * that decodebin2 already has */
+      if (gst_caps_is_any (sinkcaps)) {
+        GST_WARNING_OBJECT (group->playbin, "Video sink '%s' accepts ANY caps",
+            GST_OBJECT_NAME (sink));
+        gst_caps_unref (sinkcaps);
+        sinkcaps = NULL;
+      }
     }
   }
+  if (sinkcaps) {
+    ret = !gst_caps_can_intersect (caps, sinkcaps);
+    gst_caps_unref (sinkcaps);
+  }
+
+done:
   GST_SOURCE_GROUP_UNLOCK (group);
   GST_PLAY_BIN_UNLOCK (group->playbin);
 
-  ret = !gst_caps_can_intersect (stopcaps, caps);
-  gst_caps_unref (stopcaps);
-
   GST_DEBUG_OBJECT (group->playbin,
       "continue autoplugging group %p for %s:%s, %" GST_PTR_FORMAT ": %d",
       group, GST_DEBUG_PAD_NAME (pad), caps, ret);
-- 
2.7.4