playsinkconvertbin: improve gst_play_sink_convert_bin_getcaps return

If we have the peer caps and a caps filter, return peer_caps +
intersect_first (filter, converter_caps) instead of
intersect_first (filter, peer_caps + converter_caps) and preservers
downstream caps preference order.

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

tests: Refactor RTP basepayloading test into pay/depay parts

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

rtpbasepayload: Let caps event also configure seqnum-offset

Previously the sequence number kept track of by GstRTPBasePayload would
only be set when going from READY to PAUSED state. This meant that a
downstream element that attempted to configure a basepayloader by
setting seqnum-offset e.g. in its sinkpad's caps template would have
trouble configuring the basepayloader. The reason was that the caps
event which arrives with the desired value for seqnum-offset did not
arrive at the basepayloader until caps negotiation took place,
significantly later than the transition from READY to PAUSED.

The result after this patch is that the default value for the
seqnum-offset property, or later set values for this property, will take
effect when going from READY to PAUSED like before. In addition the an
arriving caps event will also affect the basepayloaders configured
sequence number as the event arrives.

rtpbasepayload: Fix payload type property boundary value

The payload type field in an RTP packet header is 7 bits wide, hence the
boundary values ought to be 0x00 and 0x7f, not the previously stated
values 0x00 and 0x80.

rtpbasedepayload: Fix typos in comments

docs: add GstVideoPool to docs

decodebin: If we have a demuxer without dynamic srcpads, just assume no-more-pads

Otherwise we will wait until the multiqueue after the demuxer will
overrun, which is clearly not needed then.

decodebin: Also make sure to not duplicate an element factory after a group

If we are using an adaptive stream demuxer, which outputs a non-container
stream, we are putting another multiqueue after the *parser* following
the adaptive stream demuxer. We do not want to add another instance of
the same parser right after this multiqueue.

decodebin: During pre-rolling always use the auto-preroll limits on multiqueues

Even if we're buffering in the multiqueues.

decodebin: Pass through the seekability information when setting multiqueue limits

decodebin: During exposing of pads don't set the multiqueue limits multiple times to different values

Instead just set them once in the very end to the correct values.

decodebin: Only enable multiqueue buffering once we're pre-rolled

Otherwise we will emit buffering messages not just from the last
multiqueue but also from previous multiqueues... confusing the
application with different percentages during pre-rolling.

decodebin: Make sure that we always have a second multiqueue for adaptive streaming demuxers

For adaptive streaming demuxer we insert a multiqueue after
this demuxer. This multiqueue will get one fragment per buffer.
Now for the case where we have a container stream inside these
buffers, another demuxer will be plugged and after this second
demuxer there will be a second multiqueue. This second multiqueue
will get smaller buffers and will be the one emitting buffering
messages.
If we don't have a container stream inside the fragment buffers,
we'll insert a multiqueue below right after the next element after
the adaptive streaming demuxer. This is going to be a parser or
decoder, and will output smaller buffers.

uridecodebin: Always use buffering in multiqueue for adaptive streams

uridecodebin: Only add a queue2 for buffering for non-adaptive streaming streams

uridecodebin: pass on the buffering property for adaptive streams

Adaptive streams should download its data inside the demuxer, so
we want to use multiqueue's buffering messages to control the
pipeline flow and avoid losing sync if download rates are low;

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

tests: add new unit tests to .gitignore

rtspconnection: New unit test

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

rtspconnection: Remove read child source when POST is disconnected

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

defs: update for new rtspconnection symbols

oggdemux: allow file to go until the end in push mode

When seeking back to original state after duration seeks, let
upstream know that we want the whole file, including the last
byte that wasn't requested on the duration seeks.

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

oggdemux: remove unused instance variable event

It is never set to anything

rtspconnection: allow specifying a certificate database

Two new functions have been added,
gst_rtsp_connection_set_tls_database() and
gst_rtsp_connection_get_tls_database(). The certificate database will be
used when a certificate can't be verified with the default database.

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

rtspconnection: get rid of superfluous whitespaces

encodebin: simplify tests

Also use the profile helper for the ogg profile here.

video: Fix NV12_64Z32 default offset and size

This was a regression introduced by f52fd7a68, where we started using
the stride to encode the dimensions in tiles. This patch simply updates
offset and size calculation as described in the documentation,
part-mediatype-video-raw.txt.

playbin: Keep inputselector around until we release its pads

Otherwise there's an interesting race condition when we destroy
the inputselector (actually it will be destroyed later when its state
change message gets destroyed) and afterwards release its sinkpad.

This is the code path when the last channel is removed from the
input selector.

Gave this warning sometimes, for chained oggs or whenever else
we change decode groups:
GStreamer-CRITICAL **: Padname '':sink_0 does not belong to element inputselector0 when removing

audioconvert: never do mixing for 1->1 channel conversions

MONO and NONE position are the same, for example, but in
general there isn't much to do here for such a conversion.

Fixes problem in audioconvert, which would end up using
a mixmatrix when converting between different mono format
because it thinks MONO positioning is different from
unpositioned channels, which is not the case in this
special case. The mixmatrix would end up being 0.0 so
audioconvert would convert to silence samples.

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

audio: map channels=1,channel-mask=0 to MONO instead of NONE

Fixes problem in audioconvert, which would end up using
a mixmatrix when converting between different mono format
because it thinks MONO positioning is different from
unpositioned channels, which is not the case in this
special case. The mixmatrix would end up being 0.0 so
audioconvert would convert to silence samples.

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

encodebin: refactor tests

Add a new test to demo how to get missing plugin message.
Split some tests that unneccesarily munge unrelated checks into one test.

playsink: Only remove the complete text chain if the text pad goes away

If the text pads does not go away we just set the overlay to silent, which
allows us to immediately re-enable subs later again. However before this
change we also released the streamsynchronizer text pads, which deadlocked
because there was still dataflow going on. Just do this only if we remove
the complete chain.

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

tools: gst-play: add volume control

oggmux: properly flush when seeking at the beginning

Reset all internal status when collect pads forwards a flush-stop
from the pads to be able to start the stream again.

uridecodebin: Don't leak pad references

tests: Don't build disabled plugins' check tests

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

playbin: First try to get the pad's current caps, then query caps

The caps query might give us ANY caps while the pad has fixed caps
configured currently.

playbin: Fix memory leak in autoplugging code

We should not leak element factories ideally.

playbin: Fix memory leak in unit test

subtitleoverlay: Remove unused function

audiosrc: Fix typo in docs

We read *from* the audio device, not to it.

videoscale: Fix compiler warning in unit test

error: implicit conversion from enumeration type
'GstFormat' to different enumeration type 'GstVideoFormat'

videoconvert: Fix compiler warning in unit test

error: implicit conversion from enumeration type
'GstFormat' to different enumeration type 'GstVideoFormat'

playback-test: Fix types for comparisons

Storing a 64 bit integer in a 32 bit integer and then checking
for the error cases might not be ideal.

error: comparison of constant -9223372036854775808 with
expression of type 'guint' (aka 'unsigned int') is always true

oggmux: Fix typo in header include guard

clang does not like this.

alsa: Make clang happy with our g_strdup_vprintf() wrapper

playback-test: allow seeking outside of the range

For download buffer, allow seeking outside of the already downloaded
area.

basetextoverlay: use correct segment for text

video time uses the 'segment' and the text time should use
the 'text_segment'.

If different segments are used for video and text it would
lead to out of sync video/subtitles.

check: add some more checks

Add header and payload length check in case of CSRCs.

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

jsseek: Add missing HAVE_X check

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

tagdemux: Forward TIME seeks upstream too, maybe upstream can handle that

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

docs: doc fixes for audio library

Add sections docs for audiometa. Fix sections docs for audiochannels. Remove old
mixerutil section.

videotestsrc: ensure having caps when setting the buffer pool config

It happens if downstream does not propose a buffer pool.
GST_DEBUG=2 gst-launch-1.0 videotestsrc ! fakesink

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

gst-play: Support non-ASCII tags

By calling setlocale() to get us multi-byte/UTF-8 support.

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

gst-discoverer: Support non-ASCII tags

By calling setlocale() to get us multi-byte/UTF-8 support.

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

Automatic update of common submodule

From d48bed3 to 1a07da9

streamsplitter: push pending events before eos

Push any pending events downstream before pushing eos

tests: audioencoder: add tests analogous to the videoencoder ones

audioencoder: push pending events and tags before EOS

if there are tags or events pending and an EOS is received, push those
events and tags before the EOS.

tests: videoencoder: check that tags are pushed before eos

Check that if a new tag event is received right before eos it
is pushed before the eos

videoencoder: push tags and events before eos

if any tags or events are pending, push them before pushing eos

tests: videoencoder: basic videoencoder base class test

Adds a single test for video encoding

rtpbasepayload: Do cosmetic changes to rtptime calculations

 * Change running time type to guint64
 * Use GST_CLOCK_TIME_NONE() to check for invalid timestamps
 * Name variables so ns-based and hz-based timestamps are evident

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

rtpbasepayload: Expose running-time of payloaded stream

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

rtpbasepayload: Improve documentation for perfect-rtptime

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

rtpbasepayload: Fix typos in documentation for properties

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

decodebin: make it possible to register multiple handlers for autoplug-select

Change the way autoplug-select is accumulated so that it's possible to have
multiple handlers. The handlers keep getting called as long as they keep
returning GST_AUTOPLUG_SELECT_TRY.

One practical example of when this is needed is when hooking into playbin's
uridecodebin, which is perhaps not very elegant but the only way to influence
which streams playbin autoplugs/exposes.

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

rtpbasepayload: Add statistics property

This property allows for an atomically retrieved set of properties that
can e.g. be used to generate RTP-Info headers.

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

uridecodebin: Drop hardcoded list of media suitable for download buffering

Discussion on IRC indicated that the main reason for this list was to
prevent demuxers that can trigger a lot of seeking from using
progressive buffering using queue2 (which due to being seekable triggers
that behaviour).

However given that upstream can indicate seeks are possible but should
be avoided via a scheduling query, this extra whitelisting shouldn't be
necessary for well-behaved demuxers.

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

videoconvert: tweak the scoring algorithm

Make a little table of conversions and manually score them. Use this
info to define better weights for the scoring algorithm.
give separate scores for doing changes and the impact of the change,
This allows us to avoid conversion when we can but still allow fairly
lossless changes.
The old code did not penalize GRAY conversions, PAL conversions were
punished too low and depth conversions too high.

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

video-chroma: don't crash on NULL resamplers

Make dummy resamplers for all cases and only execute the horizontal
resampler instead of crashing.

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

audiobasesink: make _get_time more threadsafe

We call the _get_time function from the provided clock and we don't lock
the sink object for performance reasons. Make sure we only read and
check variables once so that they don't change while we are executing
the code.

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

audioresample: It's HAVE_EMMINTRIN_H, not HAVE_XMMINTRIN_H for SSE2

audioresample: Fix build on x86 if emmintrin.h is available but can't be used

On i386, EMMINTRIN is defined but not usable without SSE so check for
__SSE__ and __SSE2__ as well.

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

configure: Initialize Qt variables

examples: Port Qt examples to Qt5

riff: Fix G726 caps creation

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

discoverer: minor docs fix

Can use a custom main context as well if needed.

videodecoder: Add API to get the currently pending frame size for parsing

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

playbin: Remove unnecessary assignment

Remove duplicated assignment

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

playbin: Insert decoders without GstAVElement information between the other decoders

Otherwise they would be preferred over all decoders independent
of their ranks.

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

playbin: Only put parsers and sinks first, not all non-decoders

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

tests: videodecoder: plug a few leaks

Remove leaks of caps and events references

videodecoder: plug leak when frames are released on subclass stop

They end up stored in the 'pending_events' list and should be
freed after calling stop

gst-play: Handle CLOCK_LOST message

It is necessary for playbin gapless playback when switching
between audio-only and video-only files for example.

streamsplitter: handle ACCEPT_CAPS query correctly

We can accept a caps when one of the downstream peers can accept the
caps. This is not the same as checking a subset of the getcaps
result because parsers might accept broader caps than what their getcaps
function returns (See https://bugzilla.gnome.org/show_bug.cgi?id=677401).

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

tests: audiodecoder: add another test for negotiation with gap event

Check that even if the subclass doesn't call set_output_format, the base
class should use upstream provided caps to fill the output caps that is
pushed before the gap event is forwarded, otherwise it ends again fixating
the rate and channels to 1.

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

audiodecoder: copy rate and channels from input before fixating output caps

For default caps generation when handling gap events that are sent
before any buffer, try to use caps that are closer to what upstream
provided to avoid fixating rate or channels to 1 as default.

So there are the steps:
1) Try to set rate, channels and channel-mask from upstream if provided
2) Fixate the rate and channels to the default rate and channels from
   audio lib
3) Fixate the caps just to be sure everything is fixed
4) If no channel-mask was provided and channels > 2, use a default
   channel-mask (taken from audioconvert code)

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

xvimagesink: don't recreate xvcontext

A xvcontext can be created early in gst_xvimagesink_set_window_handle().
In this case don't recreate, i.e. overwrite it in gst_xvimagesink_open().
Otherwise XEvents won't be handled in the xevent listener thread.

Fixes a regression when setting the window handle on the sink in
the very beginning before changing its state.

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

oggdemux: fix broken seeking reading the whole file

A change in gst_ogg_demux_do_seek caused oggdemux to wait for
a page for each of the streams, including a skeleton stream if
one was present. Since Skeleton only has header pages, that
was never going to end well.

Also, the code was skipping CMML streams when looking for pages,
so would also have broken on CMML streams.

Thus, we change the code to disregard Skeleton streams, as well
as discontinuous streams (such as CMML and Kate). While it may
be desirable to consider Kate streams too (in order to avoid
losing a subtitle starting near the seek point), this may be
a performance drag when seeking where no subtitles are. Maybe
one could add a "give up" threshold for such discontinuous
streams, so we'd get any page if there is one, but do not end
up reading preposterous amounts of data otherwise.

In any case, it is important that the code that determines
the amount of streams to look pages for remains consistent with
the "early out" conditions of the code that actually parses
the incoming pages, lest we never decrease the pending counter
to zero.

This fixes seeking on a file with a skeleton track reading all
the file on each seek.

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

oggdemux: use an adaptive chunksize for performance reasons

Ogg data is read chunk by chunk, and the chunk size used was
originally taken from libvorbisfile. However, this value leads
to poor performance when used on an Ogg file with large pages
(Ogg pages can be close to 64 KB).

We can't just use a larger chunk size, since this will decrease
performance on small page streams, so we use an adaptive scheme
where the chunk size is twice the largest page size we've seen
so far in the stream. For "typical" Ogg/Vorbis, this gives us
almost the same chunk size (a bit lower), and this lets us get
better performance on streams with large pages.

audiodecoder: avoid parsing caps event if it is not used

Saves some cpu

audiodecoder: make sure caps is set before forwarding gap event

Before trying to generate a default fixated caps when handling a gap
event, make sure that the same strategy that is used when handling
a buffer has been attempted. Otherwise audiodecoder will ignore
upstream caps settings such as rate and channels and will likely
end with a caps with channels=1 and rate=1.

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

tests: audiodecoder: check that negotiation works buffers and gaps

Adds 2 tests to verify that output caps are the expected value, reusing
input structure values for both buffers and gaps

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

tests: audiodecoder: add basic playback test for audio decoder

Simple test that just check that audio decoding works as expected

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

videoverlay: Don't mention gconf elements and add a sentence about playbin/playsink

playbin/playsink now implement the video overlay interface

win32: add new API to .def file

videodecoder: only copy chroma_site when known

Only overwrite the chroma-site if we have a valid value in the reference
format.

videoconvert: don't interpolate chroma in I420 -> RGB

Don't try to interpolate the chroma samples, the used algorithm only
works for horizontal cositing. Let's switch to a faster and safer
version until we handle chroma siting correctly in the fastpaths.

videoutils: add some debug

doc: Add new sections introduce for tile format

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