convertbin: port to 0.11 again

Merge branch 'master' into 0.11

Conflicts:
common
configure.ac
gst-libs/gst/audio/gstbaseaudiosink.c
gst/playback/gstdecodebin2.c
gst/playback/gstplaysinkaudioconvert.c
gst/playback/gstplaysinkaudioconvert.h
gst/playback/gstplaysinkvideoconvert.c
gst/playback/gstplaysinkvideoconvert.h

baseaudiosink: make discont-wait configurable

Now we can configure how much time to wait before deciding that a
discont has happened.

Also, adds getter and setter to allow derived implementations to set
this value upon construction.

Suggestions and several improvements by Havard Graff.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

baseaudiosink: delay the resyncing of timestamp vs ringbuffertime

A common problem for audio-playback is that the timestamps might not
be completely linear. This is specially common when doing streaming over
a network, where you can have jittery and/or bursty packettransmission,
which again will often be reflected on the buffertimestamps.

Now, the current implementation have a threshold that says how far the
buffertimestamp is allowed o drift from the ideal aligned time in the
ringbuffer. This was an instant reaction, and ment that if one buffer
arrived with a timestamp that would breach the drift-tolerance, a resync
would take place, and the result would be an audible gap for the
listener.

The annoying thing would be that in the case of a "timestamp-outlier",
you would first resync one way, say +100ms, and then, if the next
timestamp was "back on track", you would end up resyncing the other way
(-100ms) So in fact, when you had only one buffer with slightly off
timestamping, you would end up with *two* audible gaps. This is the
problem this patch addresses.

The way to "fix" this problem with the previous implementation, would
have been to increase the "drift-tolerance" to a value that was greater
than the largest timestamp-outlier one would normally expect.  The big
problem with this approach, however, is that it will allow normal
operations with a huge offset timestamp vs running-time, which is
detrimental to lip-sync. If the drift-tolerance is set to 200ms, it
basically means that lip-sync can easily end up being off by that much.

This patch will basically start a timer when the first breach of
drift-tolerance is detected. If any following timestamp for the next n
nanoseconds gets "back on track" within the threshold, it has basically
eliminated the effect of an outlier, and the timer is stopped.  If,
however, all timestamps within this time-limit are breaching the
threshold, we are probably facing a more permanent offset in the
timestamps, and a resync is allowed to happen.

So basically this patch offers something as rare as both higher
accuracy, it terms of allowing smaller drift-tolerances, as well as much
smoother, less glitchy playback!

Commit message and improvments by Havard Graff.

Fixes bug #640859.

baseaudiosink: rename some variables

baseaudiosink: use gst_util_uint64_scale_int when appropriate

It's probably safer this way.

baseaudiosink: split drift-tolerance into alignment-threshold

So that drift-tolerance is used for clock slaving resync, and
alignment-threshold is for timestamp drift.

baseaudiosink: trivial comment fixes

Some found by Havard Graff.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

adder: don't ref NULL caps

volume: use new api to check activity of a controller

controller: port to new location and api changes

playback: name conversion elements differently

build: add audio libs (pulled by pbutils) to avoid linking against system version

ringbuffer: store bpf in the right variable

docs: fix some docs

uridecodebin: fix template name

oggdemux: fix somtimes pad

fix pad template names for request pads

subtitleoverlay: Use gst_caps_merge() instead of gst_caps_union()

This keeps the caps order and is more efficient.

playsinkconvertbin: Use gst_caps_merge() instead of gst_caps_union()

This keeps the caps order and is more efficient.

rtpbuffer: Fix compilation issues with gcc 4.6.1

win32: Update for modified API

Really remove all mention of gstnetbuffer

Add missing default include paths to androgenizer call

Fixes building tag/ with Android's NDK

update for request pads change.

net: remove net library, it's now in core

decodebin2: Post all source pads in stream-topology messages as "element-srcpad" values

This allows us to easily get ahold of all pads on a stream-topology message, including
pre-decoder ones, while "pad" only gives us access to the raw pads (as used by discoverer).

decodebin2: Use existing "caps" quark for one of the structure sets

tests: fix netbuffer test

playsinkconvertbin: Don't add identity multiple times

playsink: send flush start/stop event when we switch elements

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

playsink: re-add identity where appropriate

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

playsink: lock the new {set,get}_property functions

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

playsinkconvertbin: Be more consistent with ghostpad targets

Set up targets on READY->PAUSED state change to passthrough by
default. This prevents the targets from being unset on the
first run, while the 'raw' variable would mean that some
target is set.

playsinkconvertbin: No need to remove the identity

The identity element should be handled by the GstBin's cleanup,
removing it on the remove_elements function might remove it
too soon, as this function can be called directly from playsink

playsinkconvertbin: Adding some debug messages

Adds a couple debug messages and some g_assert to make debugging
easier

playsink-videoconvert: Fix warning on build

Remove unused variable

playsink: handle after-the-fact changes in converters/volume booleans

The playsink was nastily poking a boolean in the structure.
Make those booleans properties, so we are told when they change,
and rebuild the conversion bin when they do.

Some cleanup to go with it too.

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

playsink: handle NULL cached caps in getcaps

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

playsink: consider both passthrough and converter caps in getcaps

Since we can switch between both modes.

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

playsink: cache inner converter bin caps

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

playsink: keep both raw and non raw pipelines at all times

and switch between them as needed.

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

playsink: only compare against the media type we expect

ie, audio/x-raw- for audio, video/x-raw- for video.

Add a trailing - to be more specific. I doubt there's anything
like audio/x-rawhide or something, but you never know.

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

playsink: refactor the converter bins since they are almost identical

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

playsink: fix passthrough mode (hopefully)

The code was doing counterintuitive rewiring of pads when the
bin did not contain any elements. We now add an identity element
in that case, which makes it simpler, and should fix the AC3
passthrough mode when using pulseaudio (but I don't see the bug
here so can't test).

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

playsink: handle NULL ghost pad target

For the src pad anyway.

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

Revert "playsinkaudioconvert: Fix warning when there is no target pad yet"

This reverts commit f35c51c14915729f0fdf2b348f351ea7e81027cc.

Better patch coming soon.

oggmux: Remove obsolete #include

docs: add draft for subtitle overlays to design docs

Main purpose is to provide a generic way to make subtitles work on
top of non-raw video (vaapi, vdpau, etc.).

configure: Allow setting GLIB_EXTRA_CFLAGS

Similar to gstreamer commit bb2020b1e794210cf7d44c6626122f611016a620

update for tag API removal

video: Add convenience macros for accessing GstVideoInfo flags

netbuffer: _netaddress_ -> _net_address_

netaddress: updata api

rename meta* -> *meta

alsa: update for new task api

structure: fix for api update

bufferlist: update for new API

Update for pad API changes

GstProbeType, GstProbeReturn and GstActivateMode -> GstPad*

subtitleoverlay: don't include header that's been removed

Merge remote-tracking branch 'origin/master' into 0.11

audioencoder: save audio info parsed in setcaps in encoder context

Otherwise we'll just error out when the first buffer gets pushed.
This is a porting artefact, in 0.10 the infos were allocated on the
heap, now we're doing everything with stack-allocated structs.

ext, gst, gst-libs, tests: update for tag list API changes

audio: remove old C file generated from template

Not sure how this one got pulled into a merge. In 0.10, it was moved away to
gst-template a long time ago. gstaudiofilterexample.c got generated from
gstaudiofiltertemplate.c.

subtitleoverlay: don't use soon-to-be-deprecated gst_filter_run()

configure.ac: Fix build

Merge branch 'master' into 0.11

fix compile for SEEK_TYPE_CUR removal

audiosink: avoid deadlocking audioringbuffer thread

... when it goes into wait for ringbuffer starting just after such
having been signalled.

Fixes #661738.

audiofilter: use BPF for unit_size

Merge branch 'master' into 0.11

audiofilter: fix get_unit_size

audiofilter: init audio info sooner

audio, video: init audio/video format info to UNKNOWN format

This is to prevent e.g. GST_AUDIO_INFO_FORMAT() from crashing on a NULL pointer
dereference when used with an unset info.

typefind: extract SOF marker in jpeg typefinder

The SOF types are defined by http://www.w3.org/Graphics/JPEG/itu-t81.pdf

This is needed to make sure that we plug a jpeg decoder that
can handle the type of JPEG we have (e.g. lossless JPEG)

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

oggmux: port to gstcollectpads2

basertppay: rename caps fields

Make the caps fields for timestamp and seqnum match the element
properties.

See #628773

basedepay: remove old fields

tests: fix compilation

fix compilation

Merge branch 'master' into 0.11

Conflicts:
configure.ac
gst-libs/gst/audio/gstbaseaudiosink.c
gst/audioconvert/channelmixtest.c
gst/playback/gstplaybasebin.c
gst/playback/gstsubtitleoverlay.c
tests/examples/Makefile.am
tests/examples/audio/Makefile.am

overlay: fix compilation

build: Fix build for moved volume subdir

volume: move volume example to audio

audio examples. fix the makefile

volume: make global vars static

audiomix: add a simple audiomix example

playsinkaudioconvert: Fix warning when there is no target pad yet

decodebin2: Link elements before testing if they can reach the READY state

This is made possible by filtering errors. This is required to let
harware accelerated element query the video context. The video context
is used to determine if the HW is capable, and thus if the element is
supported or not.

Fixes bug #662330.

playbasebin: remove avoidable call to gst_object_set_name

oggdemux: remove avoidable call to gst_object_set_name

audioconvert: bury dead test program

interfaces: clean up the use of iface and class/klass

Disable ext/vorbis for the android ndk build

It currently makes the build fail. Idea is to enable
it back again once its building problems get sorted
out.

subtitleoverlay: fix leaks of pad templates and internal proxy pads

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