subtitleoverlay: fix leak of element reference through pad block

If the pad block never happens because there is no data flow at all, the
callback is never fired and the reference is never released. This causes a
reference cycle between the pad and element, so valgrind is not very vocal
about it (memory is still reachable).

audiodecoder: having gather queue contents implies some draining is in order

... which ensures e.g. processing and sending last fragment of reverse playback
downstream at EOS.

vorbisdec: do not try to read past the buffer array

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

vorbisdec: only finish header packet frame if received in-stream

... rather than scaring audiodecoder with a frame extracted from caps.

Fixes #662108 (partially).

x(v)imagesink: make it more clean that "synchronous" props are not for avsync

baseaudiosink: fix unused variable compiler warning if debugging in core is disabled

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

subtitleoverlay: fix event unref in (rare) error case

decodebin2: fire drained signal where appropriate

This will allow playbin2 to send its about-to-finish signal.
Taken out (apparently by mistake) by the EOS rewrite in july.

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

oggdemux: do not retry seeking indefinitely

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

videotestsrc: fix LDADD missing GST_LIBS

vorbisenc: only push header buffers following initial events

audioencoder: fix compile warning

tests: vorbisenc: adjust discontinuity checking to audioencoder behaviour

... which still detects gaps and marks DISCONT, depending on configuration,
but may come up with somewhat different timestamps when crossing the gap.

tests: vorbisdec: properly configure audiodecoder when requiring perfect ts

tests: vorbisdec: remove empty header buffer check

... as empty buffers are discarded, and header buffers are now
also optionally retrieved from caps anyway.

audioencoder: only resync to upstream upon discont in perfect ts mode

... as documented, where discont is marked here if tolerance has been
exceeded.

audiodecoder: fix timestamp tolerance handling

audiodecoder: handle empty input by discarding

vorbisdec: port to audiodecoder

audioencoder: make upstream queries MT-safe

audiodecoder: make upstream queries and events MT-safe

vorbisenc: port to audioencoder

tests: actually test what we said we would

All tests were testing the default sine wave

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

audiotestsrc: add missing break

And make violet noise usable

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

playsink: fix caps negotiation through the new convenience bins

The bins' getcaps was bypassing the inner elements, and thus
failing to account for the caps transformations they allow,
which caused YUV video pipelines to fail with ximagesink, which
does not support YUV, even though the convenience bin includes
a colorspace converter for just this purpose.

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

playbin2: fix mismatch between video/ and video/x-dvd-subpicture

The new code was checking for a prefix, and would find video/
first. Check in two passes, first checking for a perfect match,
and falling back to a prefix check if nothing was found.

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

encodebin: Re-enable parsers

Re-enable parsers in encodebin to allow more passthrough scenarios
to work. Specially the ones that require changing 'stream formats'.
i.e. h264 in mkv to mpegts.

playsink: Add audio- and text-sink props

auditestsrc: indent fix

playsink: Add video-sink property

The video-sink property allows manual specification via g_object_set ()
of the video sink element to be used.

playbin2: Minor cleanup of decoder-sink compatibility checking code

playbin2: Make sure that the decoders we plug are compatible with the fixed sink

The fact that a decoder is not compatible with the fixed sink
is currently happenning in the case where we have hardware accelerated
video decoders on the system (especially vaapi elements that are actually plugged),
and the user is providing a sink that doesn't support the surface.

A simple example that shows how it used to crash on a system where gstreamer-vaapi
is installed:
    gst-launch playbin2 video-sink=xvimagesink uri=/codec/supported/by/vaapi

What we are now doing in this case, is avoid using the accelerated
decoder and plug a "normal" decoder instead (if avalaible).

This commit doesn't handle the case where we have hardware accelerated
demuxing.

encoding-profile: add a function to create a profile from a discoverer info

Only A/V streams are added at the moment, there does not seem to be
a similar way to add other streams (eg, subtitles).

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

alsasrc: fail gracefully when ALSA does not give timestamps

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

decodebin2: Use a TIME limit for pre-rolling in live streams and not in non-live streams

Fixes bug #647769 for real.

textoverlay: add YV12 support

Basically the same as I420, just with chroma planes swapped.

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

encodebin: Fix typo on formatter adding condition

The condition is if the muxer doesn't have tag setter *and* isn't
a formatter itself. Any of those two conditions makes the muxer
good enough to not need a formatter.

audiodecoder: really push pending events

audioencoder: remove more tags from upstream tag events such as bitrate tags

We want to remove all codec specific tags.

videotestsrc: Fix compiler warning on 64 bit mingw-w64

Fixes bug #660304.

playbin2: Fix compiler warnings on 64 bit mingw-w64

Fixes bug #660301.

audioencoder: only got_data if we really got some

... which avoids going loopy with casual subclass.

audioencoder: really push pending events

audioencoder: send tag event after pending events

... which probably includes a pending newsegment event.

audioencoder: protect pending_events with proper lock

audioencoder: clean up some documentation

docs: minor docs fix

docs: Adjust for GstAudioEncoder API changes

win32: Adjust for GstAudioEncoder API changes

audioencoder: Improve set_frame_sample_{min,max} documentation

audiodecoder: Fix thread safety issues if both pads have different streaming threads

audiodecoder: Delay sending of serialized events to finish_frame()

Revert "audioencoder: Use GST_BOILERPLATE instead of custom GObject boilerplate code"

This reverts commit 11e375486e07cfa0686a97b5cf6110909b3a828c.

GST_BOILERPLATE() can't define an abstract type and
G_DEFINE_ABSTRACT_TYPE() does not pass the class struct to
the instance_init function and there's no way to get the
class struct of the current type in instance_init().

audioencoder: Add support for requesting a minimum and maximum number of samples per frame

This extends the special case of a fixed number of samples per frame
that was supported before already.

audioencoder: Fix thread safety issues if both pads have different streaming threads

audioencoder: Delay sending of serialized events to finish_frame()

This makes sure that the caps are already set before any serialized
events are sent downstream.

audioencoder: Use GST_BOILERPLATE instead of custom GObject boilerplate code

audioencoder: add some tag handling convenience help

audioencoder: provide CODEC/AUDIO_CODEC handling

audioencoder: filter AUDIO_CODEC/CODEC tags from passing tag events

typefindfunctions: backport some const-ifications from 0.11 branch

To keep code identical as much as possible between the two branches,
for easier merging.

typefindfunctions: fix indentation

encodebin: Avoid unnecessary read only caps copy

audioencoder: proxy some more optional downstream caps fields to upstream

audioencoder: changed is verily the opposite of equal

audioencoder: prevent crashing when comparing to a freshly inited GstAudioInfo

audio: some more accessor macros for GstAudioInfo

audiodecoder: fix documentation typo

videorate: Add tests for the max-rate case

videorate: Print which caps didn't match up

videorate: Add a max-rate property

In various use-case you want to dynamically change the framerate (e.g.
live streams where the available network bandwidth changes). Doing this
via capsfilters in the pipeline tends to be very cumbersome and racy,
using this property instead makes it very painless.

videorate: Add test for caps negotiation

videorate: Add more strict caps negotiation

When in drop-only mode we can never provide a framerate that is higher
then the input, so let the caps negotiation reflect this.

videorate: don't unref event we don't own

http://bugzilla.gnome.org/show_bug.cgi?id=659562

decodebin2: Only check if this is a discarded type if we have fixed caps

For unfixed caps we will get here again later when the caps are fixed.

decodebin2: Only call autoplug-continue with fixed caps

With unfixed caps we can't reliably decide if the final caps
are going to be "raw" (e.g. supported by a sink) or not.

We will get here again later when the caps are fixed.

decodebin2: Fix unit test by strictly implementing parser behaviour instead of relying on basetransform

oggstream: only use information from skeleton if we have nothing better

The codec setup headers are a lot more likely to have correct information,
especially as it's easy to remux a skeleton in a file where streams don't
have the same parameters (I've even seen a file with two skeletons).

Still, this is useful in the case we have a codec we can't decode, so we
can at least (theoretically) convert granpos to time, so we discard this
information if the codec setup has already provided it.

This fixes playback on (at lesat) the original archive.org encoding of
"The Night of the Living Dead" (now replaced by another encoding).

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

discoverer: Don't use gtk-doc /* < ... > */ style comments for signals

The /*< ... >*/ style is only used for public|protected|private,
signal comments use /* signals */. This prevents the some code
parsers/binding generators to be confused by the comment.

subtitleoverlay: Get the target of the video sinkpad, not the target sinkpad in the video setcaps handler

decodebin2: Initialize variable correctly

If subdrained isn't initialized to FALSE then a chain might think
that its group is drained when in fact it's not and this can cause
a switch too early or even cause a deadlock.

decodebin2: Rewrite EOS-handling code

This is now really threadsafe and improves switching
between different groups.

decodebin2: Fix non-prerolling pipelines and not-linked errors if a parser is available but no decoder

Fixes bug #658846.

rtspdefs: add RTCP-Interval header

subtitleoverlay: Implement support for switching between raw and non-raw video streams

textoverlay: Protect against accessing the NULL parent of the pads during shutdown

Fixes bug #658901.

oggdemux: remove superfluous check in newsegment event handler

If we get a newsegment event from upstream, we can be quite
sure we're not operating pull-based.

oggdemux: minor printf format fix

oggdemux: fix wedge when seeking twice quickly in push mode

This could happen when testing with navseek, and pressing
right and left at roughly the same time. The current chain
is temporarily moved away, and this caused the flush events
not to be sent to the source pads, which would cause the
data queues downstream to reject incoming data after the
seek, and shut down, wedging the pipeline.

Now, I can't really decide whether this is a nasty steaming
hack or a good fix, but it certainly does fix the issue, and
does not seem to break anything else so far.

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

oggdemux: implement push mode seeking

This patch implements seeking in push mode (eg, over the net)
in Ogg, using the double bisection method.
As a side effect, it also fixes duration determination of network
streams, by seeking to the end to check the actual duration.

Known issues:
- Getting an EOS while seeking stops the streaming task, I can't
  find a way to prevent this (eg, by issuing a seek in the event
  handler).
- Seeking twice in a VERY short succession with playbin2 fails
  for streams with subtitles, we end up pushing in a dataqueue
  which is flushing. Rare in normal use AFAICT.
- Seeking is slow on slow links - byte ranges guesses could be
  made better, decreasing the number of required requests
- If no granule position is found in the last 64 KB of a stream,
  duration will be left unknown (should be pretty rare)

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

playbin2: fix compiler warning

Remove a check for gchar >= 128

adder: don't access the event after pushing

Fixes valgrind warnings.

Revert "playbin2: autoplug sink if stream is incompatible to the configured one"

This reverts commit b0b4e286c8cde2e79a959a444a2c68e99c3f29c6.

We agreed that the previous (pre-.35) behaviour is broken and a bug and the
current behaviour is correct, deterministic and allows the application to
handle stuff properly while the old behaviour can't be handled properly by
applications and just worked in some applications by luck.

The solution to the problem that was solved by relying on the old, broken
behaviour would be, to make decodebin2/playbin2 more aware of decoders and
improve the autoplugging of decoders by considering the caps supported by the
sink instead of just using something with the highest rank.

See bug #656923.

playbin2: autoplug sink if stream is incompatible to the configured one

Fixes regression since 0.10.33 where sinks that can cope with non raw
caps or custom caps are not autoplugged if there's a sink configured
with the properties video-sink and audio-sink which cannot handle
the stream. This change checks for compatibility on the configured one
and use it if success. Otherwhise it tries with the found factories.

oggdemux: do not propagate discontinuities in sparse streams

The first packet of a sparse stream may arrive after an initial
delay in the stream. If ogg_stream_packetout reports a discontinuity
in a sparse stream, do not propagate it to other streams in the
chain unnecessarily.

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

Revert "playsink: only add text overlay if vido sink also accepts raw caps"

This reverts commit a22faad18a73a27a2a0c903748c1a355df4d8c13. Instead
of disabling subtitles completelly when video stream have custom caps,
just let the sutbtileoverlay cope with them as now it's able to.

subtitleoverlay: gracefully handle non raw video streams

Implement handling of non raw video streams by avoiding colorspace
elements and autoplugging a compatible renderer if available. Fallback
to passthrough if no compatible renderer is found.

playbin2: try to catch malformed URIs

Only log in debug log for now, since the check is a bit
half-hearted, its purpose is mostly to make sure people
use gst_filename_to_uri() or g_filename_to_uri().

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

docs: minor addition to GST_TAG_ID3V2_HEADER_SIZE docs

theoraenc: Fix descriptions of properties