gst/playback/gstdecodebin.c: Special-case the text/plain media type: we only want...
authorTim-Philipp Müller <tim@centricular.net>
Sat, 16 Dec 2006 13:59:09 +0000 (13:59 +0000)
committerTim-Philipp Müller <tim@centricular.net>
Sat, 16 Dec 2006 13:59:09 +0000 (13:59 +0000)
commita4c0d0b8c39f106d160189b5beb3ab6d0d18da11
tree0b465ca93fe9fe556882959a288e3afcb3a05917
parent95f46c1fee16f9e6ea6523e275e27276cc5c84de
gst/playback/gstdecodebin.c: Special-case the text/plain media type: we only want to recognise it as a 'raw' decoded ...

Original commit message from CVS:
* gst/playback/gstdecodebin.c: (type_found):
Special-case the text/plain media type: we only want to recognise it
as a 'raw' decoded media type if it comes from a demuxer or subtitle
parser, but not if the entire stream is of text/plain type. If the
entire stream is text/plain, we should just error out.
This fixes playback of audio files with lyrics in totem. Totem can't
distinguish between text files and subtitle files and passes any
.txt file with the same basename as the main file to playbin as
suburi, and playbin will then throw a 'subtitle found, but no video
stream' error, which isn't entirely helpful. See #380342.
Also, with this change we'll show a slightly more correct error
message in case totem passes a playlist file to us (although a
custom error message wording instead of the default text would
probably not be a bad idea either).
Same problem also needs to be fixed for playbin+decodebin2.
* tests/check/Makefile.am:
* tests/check/elements/decodebin.c: (src_handoff_cb),
(decodebin_new_decoded_pad_cb), (GST_START_TEST),
(decodebin_suite):
Add simple unit test for decodebin for the above.
ChangeLog
gst/playback/gstdecodebin.c
tests/check/Makefile.am
tests/check/elements/decodebin.c [new file with mode: 0644]