oggdemux: Don't forget to reacquire lock when needed

Fixup to ef93130cf04143fcf675b05ce0d19eddfbac4e75

I overlooked the issue. There is a case when the lock is released
and we need to reacquire it

libgstvideo.def: add new definitions

videoencoder: add qos property

This new property control if the encoder base class should gather QoS
stats and if subclasses should use them by dropping late frames.

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

gst-plugins-base: gstaudiochannels: Handled buffer mapping failure

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

oggdemux: Don't drop sticky events

Previous commit was wrong. We should still send all events to the
pad (so that sticky events get attached to it and sent when pad
gets added).

oggdemux: Improve handling of EOS without source pads

We might have a chain to use, but it might not have any active pads

Properly detect that and send an error message on EOS

Revert "videoencoder: flush encoder in transition PAUSED->READY"

This reverts commit 877664a414a466cfcc71c79d28c470722408c9a7.

Revert "videodecoder: flush decoder in transition PAUSED->READY"

This reverts commit 6e9edc3031935ce8d6640d9774a7c093d20d4366.

Revert "audioencoder: flush encoder in transition PAUSED->READY"

This reverts commit 2dcdd13512e788797d03f9990994eecdd339ca0e.

Revert "audiodecoder: flush decoder in transition PAUSED->READY"

This reverts commit e7cf4c058ddd466a797623b5ddf162c263d07059.

win32: update exports file

oggdemux: Error out on EOS if we have no chains to use

There are not active and pending chains, if we get EOS we need to
inform the user via an error message

oggdemux: Don't double lock

The lock was already taken just before this block and is released after

videoencoder: flush encoder in transition PAUSED->READY

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

videoencoder: remove the lock from gst_video_encoder_flush

The lock is already taken before calling the flush method and can lead to
deadlock for some encoders that need to take the same lock from another
thread while flushing

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

videodecoder: flush decoder in transition PAUSED->READY

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

audioencoder: flush encoder in transition PAUSED->READY

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

audiodecoder: flush decoder in transition PAUSED->READY

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

videoencoder: implement QoS

It allows encoders to detect and drop input frames which are already
late to increase the chance of the pipeline to catch up.

The QoS logic and code is directly copied from gstvideodecoder.c.

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

videoencoder test: properly name the encoder variable

The element is an encoder so calling it 'dec' makes things confusing.

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

doc: Add some missing/malformed Since markers

audioconvert: allow empty mix matrix

When an empty mix matrix is passed, audio-channel-mixer
will now generate a (potentially truncated) identity matrix,
this replicates the behaviour of audiomixmatrix in first-channels
mode.

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

Reorder win32 defs

discoverer: output channel positions

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

discoverer: output whether the uri is live

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

API: gst_discoverer_info_get_live

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

API: gst_discoverer_audio_info_get_channel_mask

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

opusenc: Allow unpositioned channels.

The input will be treated as a set of mono channels,
and the channel-mapping-family will be set to 255
(no defined channel meaning)

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

audio-converter: remove unused mix_matrix private field

tests: appsink: fix compiler warning and typo in struct name

elements/appsink.c:624:3: warning: missing braces around initializer
with gcc 4.8.4

rtsp: Start implementing support for RTSP 2.0

Properly handle protocol version in the connection

Add the following headers types:
  * Pipelined-Request
  * Media-Properties
  * Seek-Style
  * Accept-Ranges

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

meson: remove vs_module_defs

GST_EXPORT should handle it.

meson: Add some missing args and dependencies in the gir generation

Use proper GtkDoc notation for NULL/FALSE/TRUE

meson: Add mssing GstBase-1.0 include in the gir generation

audio: Fix unit test after changed GstAudioStreamAlign constructor signature

audio: Add stream align API for getting timestamp at discont and number of samples since discont

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

audio: Add reverse playback support to GstAudioStreamAlign

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

audio: Add helper object for audio discontinuity detection and sample alignment

This is the same code that is in decklinkaudiosrc, audioringbuffer,
audiomixer and various other places. Have it once instead of copying it
everywhere.

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

alsasink: Fix Memory leak in payload not succuss case

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

audioconvert: [API]: expose mix-matrix property.

This obsoletes audiomixmatrix

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

[API]: GST_AUDIO_CONVERTER_OPT_MIX_MATRIX

Taken from audiomixmatrix, credits to Vivia Nikolaidou

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

[API]: gst_audio_channel_mixer_new_with_matrix

+ Refactor previous constructor to call on that new constructor

+ Reimplement is_passthrough to strictly check whether the matrix
  is an identity matrix, comparing channel-masks was incorrect:
  the mixer may be remixing from a list of positions to the same
  list of positions, but ordered differently, and reciprocally,
  the mixer may be remixing from a list of positions to another
  list of positions identically ordered

+ Remove unused tmp field, must have been a refactoring leftover

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

audioconvert: refactor format removal.

remove_format_info was a bit confusing to read, this removes
it in favor of standard gst_caps_map_in_place calls.

This no longer simplifies the resulting caps, but I
consider this should be the job of basetransform.

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

appsink: on drain wait for buffers to be consumed

So that an upstream element can claim all buffers to return to its buffer pool.

Added unit test 'test_query_drain'
  make elements/appsink.check

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

appsink: also clear preroll buffer in _pull_sample

If someone calls gst_app_sink_try_pull_sample they are
probably no longer interested in any preroll samples.

Useful if the user has not registered a preroll appsink callback.

Also added unit test 'test_do_not_care_preroll'
  make elements/appsink.check
that fails without this patch.

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

appsink: unref preroll buffer upon pull

There is no reason for appsink to hang onto the preroll buffer.
If needed, the application can just keep a ref on this buffer
after calling gst_app_sink_try_pull_preroll.

Also added unit test 'test_pull_preroll'
  make elements/appsink.check

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

appsink: rename GstBuffer *preroll to preroll_buffer

priv->preroll can be confused with basesink_class->preroll

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

tests: playbin-complex: skip test that needs oggdemux if it's not available

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

Request minimum buffer even if need_pool is FALSE

When tee is used, it will not request a pool, but still it wants to
know how many buffers are required.

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

video-filter: Support allocation pool with pool object

This is used to indicate upstream the requirement in buffers
while no buffer pool can be provided. In this case, only
configure the pool with caps/size/min/max if we have caps,
which we only parsed when there was no allocation pool.

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

video/affinetransformationmeta: use the provided doc comment for the struct

playbin: ref_sink() sink rather than pointer to sink

Automatic update of common submodule

From 29046b8 to 3f4aa96

videoutilsprivate: fill more video fields.

__gst_video_element_proxy_caps is called by
__gst_video_element_proxy_getcaps with caps set to the caps
allowed downstream. As we didn't set colorimetry or
chroma-site on the resulting caps, upstream considered it
possible to use whatever values it wanted, leading to
not negotiated errors later on.

As the description for that function is:
"Takes caps and copies its video fields to tmpl_caps",
it seems legitimate to set these fields there.

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

appsrc: handle duration query only if the property was set

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

decodebin3/urisourcebin: Switch to actual EOS events internally

Use the intended sequence for re-using elements:
* EOS
* STREAM_START if element is to be re-used

This avoids having elements (such as queue/multiqueue/queue2) not
properly resetting themselves.

When delaying EOS propagation (because we want to wait until all
streams of a group are done for example), we re-trigger them by
first sending the cached STREAM_START and then EOS (which will
cause elements to re-set themselves if needed and accept new
buffers/events).

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

meson: hide symbols by default unless explicitly exported

pbutils: sprinkle more GST_EXPORT

libs: add some more missing GST_EXPORT

examples/snapshot: Does not need GTK but only gdk-pixbuf

audioencoder: also adjust sample count upon discont to avoid ts overflow

Only adjusting the base_ts might lead to a negative ts and as such integer
overflow into a huge timestamp which then propagates into the granulepos
and so on.  Instead, resync to incoming buffer timestamp using both base_ts
and sample count rather than only base_ts.

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

meson: fix a few meson warnings

examples: fix memory leaks in appsrc and appsrc2 examples

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

playbin: Make sure to ref_sink() any sinks before calling activate_sink()

It is forwarding messages to the playbin bus, thus forwarding messages
that contain a floating reference to the application. This generally
makes bindings unhappy, we must not leak floating references to them.

xmptag: Stop parsing GPS coordinate if sscanf() fails

CID 1139610

appsrc: fix doc typos

appsrc: Add out annotations to gst_app_src_get_latency()

audioconvert: Consider channel=1, channel-mask=0x0 as mono

channels=1 is always mono, having it 'unpositioned' does not make
sense.

This fixes pipeline such as:

  gst-validate-1.0 audiotestsrc ! audio/x-raw,channels=2,rate=44100,layout=interleaved ! audioconvert ! audioresample ! audio/x-raw, rate=44100, channels=1 ! avenc_mp2 ! fakesink

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

videotimecode: Add GST_VIDEO_TIME_CODE_INIT macro

videofilter: Unmap input frame if mapping output frame failed

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

videodecoder: Make sure we have an actually writable buffer when modifying metadata

avviddec keeps references to the buffers internally for example, in
which case we need to do a shallow copy of the buffer.

videometa: Don't crash if adding the timecode meta to a buffer failed

ssaparse: Fix buffer leak in error case

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

adder: comment and formatting cleanups

Log a few more details. Update method comments. Remove some extra blank lines.

videorate: flush remaining buffers on SEGMENT_DONE

Just as we do on EOS.

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

fft: don't generate g-i files for bindings

The g-i stuff for this helper lib was never usable from bindings
anyway and there are problems with the latest gobject-introspection,
so we might just as well remove the g-i integration entirely for
this lib.

decodebin3: Remove FIXME and do remove_input_stream() only for the corresponding parsebin

Do not remove other parsebin's input streams. It will cause unexpected
removal of any input streams in multi-parsebin use case.

Basically, the purpose of blocking buffers is similar to checking
no-more-pads of chain/group. That is, it gives hint to know the timing
to remove old (EOSed) streams of the parsebin and to add/reuse slots
for new input streams. But, that doesn't mean that we need to remove
other parsebin's EOSed stream. Each parsebin has most likely its
own streaming thread and therefore EOSed time can be much different.
(i.e., much early EOS of subtitle only parsebin)

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

parsebin: Ensure StreamType and Caps of GstStream object before exposing it

The final StreamType and Caps might not be set yet on GstStream at exposing the pads.

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

playbin3: Delay linking text output until video stream is shown

We are not sure that which stream's collection arrives first
when there are multiple parsebins such as adaptive streaming.

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

tests: appsrc: fix leaks in new unit test

encodebin: fix possible pad ref leak in error code path

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

playbin3: We only care about source pads going away

decodebin3: use lock macro where applicable

decodebin3: Protect fields related to streams handling with the SELECTION_LOCK

Fields related to stream handling (input_streams,
output_streams, slots, guint slot_id) where used totally unprotected
until know.

This lead to several races, especially playing back RTSP streams.

To protect those fields, the OBJECT_LOCK can not be used as we sometimes
need to be able to post message on the bus while holding it.

decodebin3 already has a lock to manage stream selection, and in the end
it makes sense to protect all the stream management fields with the same
lock which is why we reuse the SELECTION_LOCK here.

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

decodebin3: Protect dbin->collection usage

Use the selection lock to protect dbin->collection access

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

tests: adder: set all properties at once

tests: adder: add helper to create buffers

Keeps repeated code out of the test and syncs it with the audiomixer test.

meson: add translations

adder: fix docs typo

adder: document caps negotiation raciness and workaround

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

playback example: Prettify time reporting for big values

When dealing with streams/contents which have large duration, it is
more user-friendly to show more details in the high values (hours or days)
than in the microseconds.

This patch will use the following formatting schemes:
* Below 1hour   : MM:SS.SSS
* Below 24hours : HHhMMmSSs
* Above         : DDdHHhMMm

decodebin3: Push EOS to output stream if they are all drained

decodebin3 checks input streams and pushes EOS if all input streams
are EOSed. If not, fake EOS is pushed to the corresponding slot.

When adaptivedemux is used with multi-track configuration,
adaptivedemux never ever push EOS to non-selected track
because streaming thread for the slot stops with not-linked flow return.
So, decodebin3 should generate EOS itself to finish playback.

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

urisourcebin: Push EOS if slot is still eos state

linked input of slot can be old input, so urisourcebin should check
eos state to figure out whether it's new one or not.
If not, urisourcebin never ever forwards EOS to downstream at the end
of presentation, because the old input is still there without removal

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

decodebin3: Don't send duplicated stream-start event

group-id in stream-start event might be updated in
parse_chain_output_probe (). This cause duplicated stream-start
twice with identical stream-id and seq-num, but only group-id is
different. Although there is no change, stream-start event will
be followed by the first buffer.

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

adder: sync some fixes from the audiomixer test

typefind: Detect JPEG2000 codestreams

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

meson: find python3 via python3 module

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

convertframe: Fix leak in case of vcrop is disabled

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

test-appsrc: Test state when blocked in caps Event

In GStreamer 1.12 and older, the GstBaseSrc live lock used to be held while
create() virtual function was called. As appsrc pushes serialized event in
that virtual function, we ended up with some deadlock while setting the
state to NULL. This test simulates this situation.

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