From e91458f13caa92e5c3d15f67f1c0c8b0a23a5c33 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= Date: Tue, 3 Nov 2009 12:47:55 +0100 Subject: [PATCH] playbin2: Set subtitle caps as raw caps for the uridecodebins This will make sure that no subparse is ever plugged and subtitleoverlay, that subpicture streams are handled the same was as subtitles and that subtitle renderers are used if available. Fixes bugs #595123, #570753, #591662, #591706. --- gst/playback/gstplaybin2.c | 45 +++++++++++++++++++++++++++++++++++--- 1 file changed, 42 insertions(+), 3 deletions(-) diff --git a/gst/playback/gstplaybin2.c b/gst/playback/gstplaybin2.c index 7e5625a0a9..0052d4ed58 100644 --- a/gst/playback/gstplaybin2.c +++ b/gst/playback/gstplaybin2.c @@ -233,6 +233,7 @@ #include "gstfactorylists.h" #include "gstinputselector.h" #include "gstscreenshot.h" +#include "gstsubtitleoverlay.h" GST_DEBUG_CATEGORY_STATIC (gst_play_bin_debug); #define GST_CAT_DEFAULT gst_play_bin_debug @@ -253,7 +254,8 @@ typedef struct _GstSourceSelect GstSourceSelect; /* has the info for a selector and provides the link to the sink */ struct _GstSourceSelect { - const gchar *media_list[3]; /* the media types for the selector */ + const gchar *media_list[8]; /* the media types for the selector */ + GstCaps *media_caps; /* more complex caps for the selector */ GstPlaySinkType type; /* the sink pad type of the selector */ GstElement *selector; /* the selector */ @@ -1052,7 +1054,13 @@ init_group (GstPlayBin * playbin, GstSourceGroup * group) group->selector[1].type = GST_PLAY_SINK_TYPE_AUDIO; group->selector[1].channels = group->audio_channels; group->selector[2].media_list[0] = "text/"; - group->selector[2].media_list[1] = "video/x-dvd-subpicture"; + group->selector[2].media_list[1] = "application/x-subtitle"; + group->selector[2].media_list[2] = "application/x-ssa"; + group->selector[2].media_list[3] = "application/x-ass"; + group->selector[2].media_list[4] = "video/x-dvd-subpicture"; + group->selector[2].media_list[5] = "subpicture/"; + group->selector[2].media_list[6] = "subtitle/"; + group->selector[2].media_caps = gst_subtitle_overlay_create_factory_caps (); group->selector[2].type = GST_PLAY_SINK_TYPE_TEXT; group->selector[2].channels = group->text_channels; group->selector[3].media_list[0] = "video/x-raw-"; @@ -1070,6 +1078,11 @@ free_group (GstPlayBin * playbin, GstSourceGroup * group) g_ptr_array_free (group->video_channels, TRUE); g_ptr_array_free (group->audio_channels, TRUE); g_ptr_array_free (group->text_channels, TRUE); + + if (group->selector[2].media_caps) + gst_caps_unref (group->selector[2].media_caps); + group->selector[2].media_caps = NULL; + g_mutex_free (group->lock); if (group->audio_sink) gst_object_unref (group->audio_sink); @@ -2050,6 +2063,10 @@ pad_added_cb (GstElement * decodebin, GstPad * pad, GstSourceGroup * group) if (array_has_value (group->selector[i].media_list, name)) { select = &group->selector[i]; break; + } else if (group->selector[i].media_caps + && gst_caps_can_intersect (group->selector[i].media_caps, caps)) { + select = &group->selector[i]; + break; } } /* no selector found for the media type, don't bother linking it to a @@ -2630,12 +2647,20 @@ activate_group (GstPlayBin * playbin, GstSourceGroup * group, GstState target) gst_element_set_state (uridecodebin, GST_STATE_READY); gst_bin_add (GST_BIN_CAST (playbin), gst_object_ref (uridecodebin)); } else { + GstCaps *rawcaps, *caps; + GST_DEBUG_OBJECT (playbin, "making new uridecodebin"); uridecodebin = gst_element_factory_make ("uridecodebin", NULL); if (!uridecodebin) goto no_decodebin; gst_bin_add (GST_BIN_CAST (playbin), uridecodebin); group->uridecodebin = gst_object_ref (uridecodebin); + + g_object_get (G_OBJECT (uridecodebin), "caps", &rawcaps, NULL); + caps = gst_caps_union (group->selector[2].media_caps, rawcaps); + g_object_set (G_OBJECT (uridecodebin), "caps", caps, NULL); + gst_caps_unref (rawcaps); + gst_caps_unref (caps); } /* configure connection speed */ @@ -2708,6 +2733,8 @@ activate_group (GstPlayBin * playbin, GstSourceGroup * group, GstState target) gst_bin_add (GST_BIN_CAST (playbin), suburidecodebin); group->suburidecodebin = gst_object_ref (suburidecodebin); + g_object_set (G_OBJECT (suburidecodebin), "caps", + group->selector[2].media_caps, NULL); } /* configure connection speed */ @@ -2955,11 +2982,23 @@ gst_play_bin_change_state (GstElement * element, GstStateChange transition) playbin = GST_PLAY_BIN (element); switch (transition) { - case GST_STATE_CHANGE_NULL_TO_READY: + case GST_STATE_CHANGE_NULL_TO_READY: { + guint i; + g_mutex_lock (playbin->elements_lock); gst_play_bin_update_elements_list (playbin); g_mutex_unlock (playbin->elements_lock); + + /* Update subtitle factory caps, might have changed because + * of newly installed plugins */ + for (i = 0; i < 2; i++) { + if (playbin->groups[i].selector[2].media_caps) + gst_caps_unref (playbin->groups[i].selector[2].media_caps); + playbin->groups[i].selector[2].media_caps = + gst_subtitle_overlay_create_factory_caps (); + } break; + } case GST_STATE_CHANGE_READY_TO_PAUSED: GST_LOG_OBJECT (playbin, "clearing shutdown flag"); g_atomic_int_set (&playbin->shutdown, 0);