webrtcechoprobe: return from _read() early if the probe is not configured yet

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

dashdemux: fix typo in implementation notes

s/enveloppe/envelope

webrtcdsp: Add support to build with meson

webrtcdsp: expose voice activity detection.

The element now exposes properties to enable and configure
voice activity detection, and posts "voice-activity" messages
when the return value of stream_has_voice () changes.

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

audiomixmatrix: Don't crash if matrix is unset

Fix a crash ff one set in_channels/out_channels to non-zero, and then
get the matrix property.

audiomixmatrix: Remove property matrix-value-array

This duplicated property is no longer needed as there is now API to
allow bindings access GST_TYPE_ARRAY (see gst_util_get/set/object_array).
Additionnally, Python has proper overrides which will make this looks
like Python. A 2x2 matrix would be set this way:

  element = matrix = Gst.ValueArray(Gst.ValueArray([1.0, -1.0]),
      Gst.ValueArray([1.0, -1.0))

Notice that you need to "cast" each arrays to Gst.ValueArray, otherwise
there is an ambiguity between Gst.ValueArray and Gst.ValueList list type.
Fortunatly, Gst.ValueArray implements the Sequence interface, so it can
be indexed like normal python matrix.

mssdemux: only add live fragments after manifest fragments

A live manifest may have a set (> LookAheadFragmentCount) of fragments
that have already been served and are stored on the server, maybe
indefinitely.  Adding the parsed live fragments after the manifest
fragments breaks duration reporting and the seekable range.

Fix by only adding parsed fragments outside the list of fragments which
assumes that the fragment list in the manifest is accurate enough to not
stray too far off what's in the retrieved data.

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

Add missing include path for gst/glib-compat-prive.h

This fixes build failure in mplex and mpeg2enc plugins and most likely
in kate plugin (untested).

decklink: Use GstQueueArray instead of GQueue

Let's save two allocations and frees per frame.

meson: Build mpeg2enc and mplex when available

kmssink: Fix compiler warning with gcc 7

gstkmssink.c: In function ‘gst_kms_sink_get_input_buffer’:
gstkmssink.c:1102:29: error: ‘mems[0]’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
   kmsmem = (GstKMSMemory *) get_cached_kmsmem (mems[0]);
                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

adaptivedemux: Don't send EOS on pads when there's a pending switch

When there are new pads pending for a bitrate switch, don't allow
EOS through from the old streams. It will be sent when the new pads are
ready, just before the old streams are removed.

This fixes racy bitrate switching with hlsdemux in urisourcebin
where old pads EOS before new pads appear and the entire pipeline can
EOS if those propagate fast enough

player: Make use of GstStreams API with playbin3

Allow use of playbin3 and GstStreams API by setting
the environment variable "GST_PLAYER_USE_PLAYBIN3"

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

decklinkvideosrc: Add read-only "signal" property

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

audiobuffersplit: Install srcpad query function on the srcpad, not the sinkpad

dashdemux: In reverse playback mode, if seeking to the beginning of a fragment, start with the previous one instead

There's no point to start downloading a fragment just to output 1ns.

dashdemux: Implement SNAP_NEAREST seeks

dashdemux: Don't use ABS() for calculating differences

We know which number is bigger.

dashdemux: Only snap seeks forwards if the seek position does not happen to be the exact segment position

dashdemux: Use gst_mpdparser_get_segment_end_time() directly instead of calculating it again

It does the exact same calculations.

mpdparser: Refactor to get rid of one level of indentation in seeking code

Also go out of the segment searching loop once segment->start > ts.
We're not going to find any earlier segment anymore.

dashdemux: Fix snap SIDX seeking and report if we're going outside the index

Instead of just going to the first or last fragment, report if we're
going outside the index. This should never happen unless there's a bug
or the stream is broken.

Allow some possibility for inaccuracies here though.

dvb: camutils: drop macro disabling glib deprecation warnings

No longer needed.

build: declare dependency from gl on allocators

libgstgl has a dependency on libgstallocators starting with:

 ed1e4c1 glupload: Add support for Vivante DirectTexture uploads

The dependency wasn't stated in the Makefile, so the build could
occasionally fail if you got unlucky with the ordering of build
steps.

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

dvb: fix distcheck

Apparently automake automatically dists README files,
but can't handle when they disappear.

Update for 'mad' mp3 decoder removal

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

m3u8: Fix parsing EXT-X-DISCONTINUITY

EXT-X-DISCONTINUITY tag should have no trailing ":" character

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

m3u8: Release m3u8 lock if consistent checking failed

To fix deadlock during live m3u8 update

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

dashdemux: Just reset SIDX parser if the index is invalid for the current segment

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

ttml: Replace assertions

Remove assertions and replace, where necessary, with code that handles
the error cases.

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

dashdemux: If a SIDX seek failed although we're in the correct segment, disable the SIDX usage for this segment

The SIDX apparently does not contain information about the current
segment, so better stop using it instead of using incorrect values.

dashdemux: Keep track of the PTS, not the SIDX index position between representations

There is no guarantee that the index positions are the same between
representations, and assuming this easily causes us to get into invalid
index positions.

dashdemux: When clearing the sidx parser, reset all state

And especially don't keep entry count and index around, we have no
entries anymore after clearing.

dashdemux: Drain buffer at the end of subfragment

Some of streams such as below have tailing boxes at the end of subfragment.
http://dash.akamaized.net/dash264/TestCases/1a/netflix/exMPD_BIP_TC1.mpd

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

dashdemux: Do not advace subfragment after pending SIDX seek done

If a MPD is On-Demand profile and no index described, demux will terminate
download loop after parsing inband SIDX with flow return custom-success.
At this moment, SIDX index is excat target position, but finish_fragment()
might cause re-advancing subfragment depending on MPD structure.

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

dashdemux: Reset SIDX related variables per fragment

SIDX's base offset (i.e., byte offset of SIDX + sidx.first_offset)
mostly vary as per fragment. Also, target SIDX index must be zero for the
new fragment.

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

dashdemux: Two depth seeking for On-Demand profile

Try to find fragment using MPD first, then do refinement to find
target subframgnet using SIDX if possible. Note that, if target fragment
was moved from the previously activated one, we should assume that
the last SIDX is invalid for new fragment.

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

dashdemux: Exception handle in _sidx_seek()

If target seek position is outside of the range of sidx entries,
binary search returns NULL pointer.

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

dashdemux: Add assertion for preventing of access after the end of the sidx entries array

Better crash cleanly here than reading some random numbers from memory.

gl/format: correct return enums in gst_gl_format_from_video_info

In commit

> 956c4d0 gl/format: use our own GL format enum's instead of gstvideo's

the name and return type of gst_gl_format_from_video_info changed,
but some returns of the old type were missed. Here they are
updated to the correct type.

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

srtp test: Remove deprecation disabling

No deprecated APIs are used.

dashdemux: Advance subfragment only if any exist

SIDX based playback is not restricted to SegmentBase, but it possible
with SegmentList/SegmentTemplate. In the latter case, each fragment
has its own SIDX box and might be subdivided into subfragment.
So, demux should not assume that the end of subfragment is the end
of stream. Moreover, should try advance subfragment only if there
are remaining subfragments.

With additional fixes by Sebastian Dröge <sebastian@centricular.com>

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

kmssink: adjust memory offset calculation for dmabuf buffers

The data in the dmabuf fd may not start from byte 0, therefore
we need to inform DRM about this additional offset.

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

glupload: adjust memory offset calculation for dmabuf buffers

The data in the dmabuf fd may not start from byte 0, therefore
we need to inform EGL about this additional offset.

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

waylandsink: fix memory offset calculation for dmabuf buffers

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

Revert "adaptivedemux: answer duration queries for live streams"

Completely disabling duration reporting with live streams is not cool.

This reverts commit e1b68d9a65ba512a52c3a2b298fa830a445eb451.

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

gl: GL_ARRAY_BUFFER is not a part of VAO state

As a result we need to bind it on every draw in order to have the
correct state in the GL state machine.

applemedia/iosurfacememory: Fix missing variable name change

956c4d0bde4d6707ccae2b4ae6dfba8500e55217 missed a variable name change.
Fix that.

gl/format: use our own GL format enum's instead of gstvideo's

They can describe in more detail (such as component sizes) the requested format.

glupload: fix GValue leak

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

Add old plugins names to cruft list

This helps fixing uninstalled setup. Also fixes some path in plugin xml
files.

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

shmsink: continue polling after EINTR

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

player: Fix setting of external subtitle URI

gst_player_set_uri_internal shouldn't free suburi which maybe set
by user to load external subtitle before start play. It just need
reset playbin's subutri property to NULL no matter if there was a
previous one or not.

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

avfvideosrc: Use AVCaptureDeviceTypeBuiltInDuoCamera instead of DualCamera for now

The former is deprecated in 10.2 but we don't require that SDK yet.

Fix doc generation

This regression was caused by the renaming of plugin-sdp into
plugin-sdpelem. The doc reference needed an update. Also, add the old
xml to the cruft file list.

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

dash/smoothstreaming: Use GST_PLUGIN_LIBTOOLFLAGS

Use the flags defined by configure script instead of always disabling
static plugins.

Rename plugin filesnames to match plugin names

- libgstgtksink.so -> libgstgtk.so
- libgstteletextdec.so -> libgstteletex.so
- libgstcamerabin2.so -> libgstcamerabin.so
- libgstonvif.so -> libgstrtponvif.so (meson only)
- sdp -> sdpelem (avoid clash with libgstsdp)
- gstsiren -> siren
- libgstkmssink.so -> libgstkms.so

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

timecodestamper: Only unref daily jam if not NULL

timecodestamper: Unref daily jam after usage

applemedia/gl: Fix compile issues for OSX

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

gl/eagl: It's -fobjc-arc not -fobj-arc

h264parse: frame->buffer can't be NULL in pre_push_frame()

CID 1402118

tests: h264parse: fix failures due to insertion of au delimiter

Since insertion of aud landed, we need to change some testcases
accroding to the change.

Note that counting frames are changed in parser.c,
due to generated frames, AUD.

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

h264parse: fix some failures in testcases

Move logic of judgement whether or not to insert AU Delimiter before
draining.
Besides, aud_needed flag is reset to TRUE when skip.

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

applemedia: avfvideosrc supports position, device-type, and orientation

Avfvideosrc represents an iphone camera or, on mac, a screencapture session.
The old API allowed you to select an input device by device index only. The new
API adds the ability to select the position (front or back facing) and
device-type (wide angle, telephoto, etc.). Furthermore, you can now specify
the orientation (portrait, landscape, etc.) of the videostream.

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

applemedia/gl: Update code to use ARC

All code interacting with Objective-C objects should now use Automated
Reference Counting rather than manual memory management or Garbage
Collection. Because ARC prohibits C-structs from containing
references to Objective-C objects, all such fields are now typed
'gpointer'. Setting and gettings Objective-C fields on such a
struct now uses explicit __bridge_* calls to tell ARC about
object lifetimes.

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

h264parse: insert AU delimiter only in case of byte-stream

Inserts AU delimeter by default if missing au delimeter from upstream.
This should be done only in case of byte-stream format.

Note that:
We have to compensate for the new bytes added for the AU, otherwise
insertion of PPS/SPS will use wrong offsets and overwrite wrong data.

Also mark the AU delimiter blob const, and use frame->out_buffer for
storing the output to keep baseparse assumptions valid.

Original-Patch-By: Michal Lazo <michal.lazo@mdragon.org>
Helped by Sebastian Dröge <sebastian@centricular.com>

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

videoaggregator: redo src caps negotiation if a sink pad's caps have changed in the meantime

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

gl: Missed one backwards compat define in GST_GL_TYPE_CONTEXT

Continued from 296b4251e3042fe95a9cce1ad7674b708ba92599

hlsdemux: Try to find type at the end of a fragment

hlsdemux tries to find type if given buffer size is large enought to
find type (currently the threshold is 2KB), or EOS in some cases.
However, since there can be small byte fragments such as WebVTT,
demux should try to find type at the end of a fragment

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

voaacenc: limit bitrate property to 320 kbps

This appears to be the internal limit of voaacenc, higher
bitrates will be ignored and 128 kbps output will be produced
instead. Therefore, we might just as well limit the allowed
property values, so that people who try to set higher bitrates
get a big fat warning instead of silently a much lower bitrate.

applemedia: Fix video texture cache build issue on MacOS

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

applemedia: ensure all textures are released before texturecache is released

It was previously possible for videotexturecache to be finalized before all of
its textures. Finalizing outstanding textures in this circumstance leads
to a crash. This patch ensure resources are freed in the proper order.

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

gl: Fixup for last commit

gl: Fix backwards compat GST_GL_TYPE_* functions/macros

These have to be macros instead of inline function as they must behave
more or less like an integer literal, i.e. include the function call.

gl/viv-fb: Don't use deprecated symbols and make macros more consistent

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

audiomixmatrix: Fix negotiation in first-channels mode if downstream does not allow the same number of channels

hlsdemux: Fix checking adaptation_field_control field to parse pcr

The PCR_flag and PCR value is in adaptation_field, not in payload.
The MSB of adaptation_field_control is used as whether adaptation_
field is exist or not.

For the case(PCR in only adaptation_field without payload), we modify
checking condition about adaptation_field_control field.

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

hls: Fix various leak on hlsdemux and m3u8

* default_variant should be freed, it holds ref.
* GstHLSMedia holds GstM3U8 ref.
* gst_m3u8_get_next_fragment() increases ref count.

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

tests: hls: Fix leak on test code

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

dashdemux: fix default suggestedPresentationDelay

When MPD@suggestedPresentationDelay is not present in the MPD,
dashdemux can provide default suggestedPresentationDelay. However
when applying default value of suggestedPresentationDelay, the value
should be subtracted from current time, not added to it. When streams
setup is performed and live point is calculated, we have to go to the
wall clock (current time) minus suggestedPresentationDelay, if we tried
to start with current time plus suggestedPresentationDelay, we would
be asking for future stream, which has not yet been recorded. Also
the value needs to be converted from ms to us.

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

adaptivedemux: answer duration queries for live streams

For duration queries on live streams, adaptivedemux ignores the query.
The problem then is that the query is answered by the downstream
qtdemux element, with the duration of the currently passing fragment.

This commit changes the behaviour of adaptivedemux to answer the duration
queries for live streams, returning GST_CLOCK_TIME_NONE.

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

hlsdemux: Implement adaptivedemux's _stream_seek()

_stream_seek() can be called by adaptivedemux when "restart download" condition.
It's mostly caused by track switching.

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

hlsdemux: Simplify seeking code by using macro

Import an adaptivedemux's macro to minimize code.
Also, this patch considers KEY_UNIT and TRICKMODE_KEY_UNITS
as snap seek.

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

hlsdemux: Early terminate seeking if we don't need to do

Some codes are imported from dashdemux

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

hlsdemux: Add support for EXT-X-DISCONTINUITY-SEQUENCE

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

adaptivemutex: Fix double mutex unlock

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

openexrdec: Also catch other exceptions by reference instead of by value

It would have to be copied otherwise, and is quite big.

CID 1373419

decklinkvideosink: Don't error out if displaying the preroll frame fails

This seems to happen sometimes on some hardware, and is not really
critical as long as the scheduling of the normal frames works fine.

Only post a warning message for this case.

openexrdec: Catch exception by reference instead of by value

It would have to be copied otherwise, and is quite big.

CID 1373419

dashdemux: Fix potential NULL pointer dereference

CID 1399700

hlsdemux: Compare media type of the correct variable that is actually set

CID 1364752

gltestsrc: Fix potential NULL pointer dereference if we fail without GError

Which can happen here if we just propagate an error that happened
elsewhere, e.g. FBO failed.

CID 1364604

glvideomixer/compositor: Correctly error out if calculating DAR fails

CID 1320700

motioncells: Initialize m_changed_datafile class field

CID 1197704

gme: Remove useless comparison of unsigned integer with < 0

CID 206187

glcolorconvert: Check return value of gst_gl_context_check_framebuffer_status()

CID 1401588

gl: Add viv-fb to DIST_SUBDIRS to fix "make distcheck"

gl: Rename gst_gl_get_affine_transformation_meta_as_ndc_ext() to prevent symbol conflict

The same symbol also exists in libgstgl, although marked as private and
internal. This has no effect when doing static linking and there's a
symbol conflict.

glutils: Mark private functions as G_GNUC_INTERNAL