discoverer: Don't wait for subtitle streams to preroll

Subtitle streams being parse can cause the pipeline to wait indefinitely
to PREROLL. This makes subtitle streams got to PAUSED even if no data is
available. This should not be a cause for concern as we don't expect to
get much data for subtitle streams other than language tags from the
container.

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

baseaudiosink: start ringbuffer upon going to PLAYING and already EOS

... otherwise we may end up without running clock in PLAYING.

Fixes #636886.

playbin2: set several properties in one go

g_object_set is a varargs function. Save 7 g_obvject_calls (and the overhead of
them) by using it accordingly.

typefindfunctions: fix compiler warning on 32-bit systems

Mark 64-bit interger constant as such to avoid warnings such as:
gsttypefindfunctions.c:2152: error: integer constant is too large for ‘long’ type

configure.ac: export plugin description more platform independent

Fixes #642504.

Automatic update of common submodule

From 1de7f6a to 6aec6b9

tests: add ABI test suite for libs

decodebin2: Only prevent to autoplug the same parser multiple times for the same chain

Parsers are the only element class that are not changing the data and
could lead to an infinite loop. Other element classes like demuxers,
e.g. id3demux, can be used multiple times in a row and sometimes are.

decodebin2: Break the double-factory checking loop immediately if the factory was used already

decodebin2: Don't use the same element multiple times in the same chain

This is going to lead to an infinite loop of this element and can easily
happen with parsers that accept their own src caps on the sinkpad.

decodebin2: Improve detection of raw caps in expose-all-streams=false mode

Previously we only checked against the raw caps but we should also
check against the return value of autoplug-continue. Additionally fix
a thread-safety issue with accessing the raw caps.

video: Add support for r210

gstvideo: Add GST_VIDEO_BUFFER_PROGRESSIVE flag

Maps to GST_BUFFER_FLAG_MEDIA4. The purpose is to explicitly indicate
whether a telecined buffer is progressive or not without having to make
assumptions based on previous buffers.

encodebin: Fix double unref in unit test

checks: add a simple unit test for the source-setup signal

playbin2, uridecodebin: add "source-setup" signal

Add "source-setup" signal for convenience and discoverability. No need
to figure out "notify::source", look up the notify callback signature,
then do an g_object_get() to get the source element..

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

oggmux: Don't handle GstCollectData as GstObject, use the pad instead

encodebin: Fix memory leaks related to request pads

Request pads have to be released by the caller and must be
unreffed after releasing them.

encodebin: Return a new reference of the pad for the "request-pad" signal

The GObject signal code assumes that the signal handlers return a
new reference or copy. Fixes bug #641927.

adder: Fill in offset_end field of outgoing buffers

... rather than leave it as GST_BUFFER_OFFSET_NONE

Fix bug #642942.

playsink: release all chains when going to NULL

Also fixes #642466.

playsink: undo state change side effect on error way out

... to avoid subsequent cleanup disposing an element not in NULL state.

playsink: avoid crashing on the way out when needed chain missing

win32: update .def file for new libgstvideo API

discoverer: handle desc==NULL

It would otherwise be printed as (null) and mess up indentation (no \n).

discoverer: Chain dispose() up to parent class

discoverer: Keep a ref for the async timeout callback

This makes sure we maintain a ref on the discoverer object while the
async timeout callback is alive to prevent a potential crash if the
object is freed while the callback is pending.

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

discoverer: Use g_signal_connect_object instead of g_signal_connect

We want to make sure the discoverer object passed to the various
callbacks doesn't become invalid if a callback is pending and the object
is free'd in the mean time.

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

typefinding: detect raw h.263

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

pbutils: encoding-target: fix error checking in target file loading

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

tests: fix videoscale test by ignoring newly-added 64-bit formats

They probably fail because ffmpegcolorspace can't handle those formats.

sdp: Fix copy/paste error in inrospection part of Makefile

tag: Fix copy/paste error in inrospection part of Makefile

rtsp: Fix copy/paste error in inrospection part of Makefile

audiorate: add skip-to-first property

API: GstAudioRate::skip-to-first

videorate: fix skip-to-first ts setup

... such as avoiding arithmetic mixing counts and ts, although latter
would typically be 0 so far.

Revert "oggmux,adder: Check if collectpads has been freed"

This reverts commit 6d150873e8b4c23d694b0351570de323b1576d76.

Depends on a core commit that was reverted.

oggmux,adder: Check if collectpads has been freed

Core now calls release_pad in finalize, which is usually after
the collectpads has been unreffed.

videoscale: Add 16-bit-channel support

videotestsrc: Add 16-bit-per-channel formats

video: Add ARGB64 and AYUV64

16-bit per channel formats.

video: Add gst_video_format_get_component_depth()

audiotestsrc: each element gets its own instance of GRand, if needed

As a result, pipelines that contain multiple instances of audiotestsrc
with the 'wave' property set to 'white-noise', 'pink-noise', or
'gaussian-noise' will run much faster, since they won't be competing
for access to the global, lock-protected instance of GRand.

Fixes bug #642720.

playbin2: If a sink claims to support ANY caps assume that it only supports the usual raw formats

This should be changed again in 0.11, if a sink really claims to support ANY
caps it should support everything or provide correct caps.

encodebin: Add a audioconverter after the audio resampler.

This allows handling non-native-endianness conversion properly.

playbin2: Use gst_pad_accept_caps() instead of intersecting with the getcaps caps

This might be faster and more accurate in some cases to detect if a
sink supports a format and autoplugging can be stopped.

uridecodebin: Add default handler for autoplug-select

uridecodebin proxies this signal and only the first signal handler
will ever be called from decodebin2, which is uridecodebin's proxy
signal handler.

uridecodebin: Return NULL from the default autoplug-sort handler

...instead of copying the array. Returning NULL will result
in the original factories array to be used and prevents a useless
array copy in most use cases.

decodebin2: Return NULL from the default autoplug-sort handler

...instead of copying the array. Returning NULL will result
in the original factories array to be used and prevents a useless
array copy in most use cases.

uridecodebin: Update autoplug-* signal docs from decodebin2

uridecodebin proxies these signals.

decodebin2: Update documentation of the autoplug-* signals

Add notes about the behaviour if multiple signal handlers are connected.
For most autoplug-* signals only the first signal handler will ever
be invoked.

Also add to the autoplug-sort docs that the signal handler can return NULL
to specify that the order should change and other handlers get the chance
to sort the array.

decodebin2: Keep the original factory list if the sort signal handlers returned NULL

uridecodebin: expose "autoplug-sort" signal

It is a proxy of the decodebin2's one, and was missing
in the previous code.

See bug #642433.

playbin2: Use a recursive mutex for the playbin lock

This lock is taken when activating a group, which could result in
calling the autoplug-continue callback, which also needs this lock
to access the sinks.

See bug #642174.

decodebin2: Disconnect signal handlers when removing a failed element

This prevents crashes later if one of the signals is emitted after the
element was removed from decodebin2 already, which can happen in discoverer.

typefind: Fix mpeg TS detection

theoraenc: move debug category init earlier

oggparse: better detection of delta unit flag

theoraenc: Set speed level while running

Set the theoraenc speed-level property from libtheora's defaults.

The speed-level property, which allows callers to trade of encoding
quality for speed in the libtheora api, has a version-dependent
maximum and default values. Instead of hardcoding the acceptable
range for the theoraenc element's presentation of this setting,
we query the library directly at class initialization time and
set the maximum and default values from that. If the query fails,
we fall back to the previous default setting.

To keep the values reported by gst-inspect (which I'm told use
the spec values from the class) with those available on an\
instantiated element, we remove to setting of enc->speed_level
from the initializer and instead pass G_PARAM_CONSTRUCT to
the property spec flags, asking g_object to set this property
when theoraenc objects are constructed.

NB in theory the maximum speed-level could depend on the actual
video caps. If later versions of libtheoraenc do this, a second
call will need to be made from theora_enc_reset to update the
property, since this function is mostly useful for realtime
adjustment of performance while the pipeline is running.

discoverer: don't leak parent tags

discoverer: improve logging (and reindent)

Add more logging for the tag merging and use the _OBJECT flavour more.

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.

playbin2: Fix handling of non-raw custom sinks

When autoplugging elements in decodebin2, check if
the caps are supported by one of the sink before
continuing autoplugging.

Fixes bug #642174.

decodebin2: Don't leak elements that fail to go to PAUSED after being autoplugged

Fixes bug #642381.

theoraenc: Don't reset the video quality setting the bitrate

libtheora has two encoding modes, CBR, where it tries to hit a target
bitrate and VBR where it tries to achieve a target quality.

Internally if the target bitrate is set to anything other then 0 the
encoding-mode is CBR.

This means that the gstreamer element can leave the video_quality
setting alone as long as the user is tweaking the bitrate. Which has the
nice side-effect that if the user explicitely sets the bitrate to 0
(which is actually the default), the quality value doesn't get reset and
one ends up encoding VBR at quality-level 0...

gdppay: ensure buffer's metadata is writable before setting caps

Automatic update of common submodule

From f94d739 to 1de7f6a

doap: update mailing list location

discoverer: Use nominal bitrate if bitrate tag is unavailable

If the bitrate tag is unavailable, this falls back to the nominal
bitrate tag instead, if that is present.

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

decodebin2: caps can be NULL

Don't use and unref NULL caps.

tagdemux: also push cached events downstream when operating in pull mode

Otherwise, having 2 tagdemux in a row followed by an element operating in
pull mode will make the second tagdemux implictly eat the first tagdemux'
tag event(s).

Fixes (part of) #641047.

oggmux: ensure serialnos are unique

We do that by checking a newly generated one is not already used in
an existing stream, and doing it again if it is.

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

oggmux: free stream map caps when done

oggmux: keep IN_CAPS flag check for header buffers as fallback

In case the ogg mapper doesn't handle all the accepted input formats
(although it really should). Saves us error handling for that case
though. Also log caps properly.

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

oggmux: use oggstream for less brittleness in recognizing headers

Using the IN_CAPS flag for this is brittle, and will fail if either
vorbisparse or vorbistag (which is itself based on vorbisparse) is
inserted between oggdemux and oggmux. Possibly other elements too
(eg, theoraparse, etc).
Using oggstream ensures we Get It Right More Often Than Not.

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

uridecodebin: fix copy-and-paste typo in property docs

oggmux: do not skip a pageno at start

Discontinuities are automatically signalled by oggdemux at the start
of a new stream. When oggmux is yet to output actual data pages,
do not signal these discontinuities in the ogg stream.

This patch may miss some actual discontinuities at the very start of
a stream, but avoids the spurious missing pages when encoding happens
normally.

A better fix might involve finding a way to distinguish between actual
data discontinuities and discontinuities merely marking the start of
a new stream.

Fixes an issue with ogg page numbering (would skip a number for no
reason, which then looks like a packet was lost somewhere) when
re-muxing an ogg stream, e.g. when re-tagging in rhythmbox.

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

theoraenc: clean up property descriptions

Remove "This property requires libtheora version >= 1.1" qualifiers
from property descriptions. They aren't needed any longer now that
we require libtheora >= 1.1.

id3tag: map the ID3v2 TENC frame to GST_TAG_ENCODED_BY

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

tcp: use socklen_t where appropriate rather than specific type

In particular, fixes Cygwin build where socklen_t is defined as int
in line with native win32 api definition.

xmptag: cast argument to isdigit to int

... as that is the specification and fixes compilation on Cygwin:

gstxmptaag.c: In function 'read_one_tag':
gstxmptag.c:1015: error: array subscript has type 'char'

appsink: add buffer fallback in case the application doesn't handle buffer lists

We shouldn't assume the application handles buffer lists, for
ease-of-use reasons and for backwards compatibility reasons.

appsink: send new-buffer-list signal

Send new-buffer-list signal when emit-signals is TRUE

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

uridecodebin: also add https to buffer protocols

HTTPS also needs buffering.

baseaudiosink: trivial cleanups

It seems these stuff was neglected from commmit d8942e2.

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

win32: fix DEFAULT_AUDIOSINK, should be direct*sound*sink

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

typefinding: register H264 typefinder with H264 caps

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

streamsplitter: release pending events refs

Unref pending events when disposing the streamsplitter.

Also refactor a little to replace a for with a g_list_foreach

tests: don't run encodebin test if vorbis or theora plugins aren't available

pbutils: add description for degas images

typefinding: use image/x-icon as media type for ICON files

That's what we've been using so far (e.g. gdkpixbufdec).

snapshot: use a keyframe seek

One would usualy get good quality snapshots quickly. The exact seek position
does not really matter.

snapshot: add a newline to the usage and error output

decodebin2: add comment and whitespace trimming

pbutils: add description for x-annodex

typefind: add typefinder for DEGAS images

This fixes at least one DEGAS image from being misdetected as DTS audio.

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

oggmux: cleanup

Remove a pointless string concatentation, and fix an off-by-one in
packetno in a log.

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

typefind: add detection for windows icon files to get them out of the way

Some of them can otherwise be misdetected for MPEG audio.

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

oggdemux: Remove dead code