mpdparser: Fix leak

The error: location takes care of freeing new_representation

CID #1405027

mpdparser: Run gst-indent

meson: add missing gstglrenderbuffer.h header

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

meson: decklink: fix meson configure error on macOS

"meson encountered an error in file
sys/decklink/meson.build, line 33, column 2:
Invalid use of addition: must be str, not list"

Also remove nonsensical linker flags on windows.

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

faceoverlay: Port to GStreamer 1.x

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

faceoverlay: Revert deletion

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

dashdemux/mpdparser: Fix wrong false sanity check for manifests with nested SegmentTemplate nodes

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

Automatic update of common submodule

From 39ac2f5 to 60aeef6

dashdemux: Handle current SIDX before current position in normal mode like in keyunit-only trick mode

Otherwise we'll get into an infinite loop here. Now this is still not
correct and will cause a clean error, but at least it won't hang forever
anymore.

meson: gl: set default value of 0 for glconf vars

meson's configure_file emits only a comment like /* #undef ... */
for values which are unset in the configuration_data. For
gstglconfig.h, this differs from the autotools build where the
preprocessor definitions are always either 0 or 1. So loop over a
list of variables to set to zero as default.

Also sync up the gstglconfig.h.meson file with the additional
macros defined by the autotools build.

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

gstaggregator: fix event use after free

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

mxfmux: fix index entry leak

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

tests: fix leak in videoframe-audiolevel test

also add braces for the else case

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

glcontext: fix display leak

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

gldisplay: fix list leak

Windows aren't always removed in time, and it turns out to be
very, very hard to remove a window in a way that's not racy and
not deadlocky. Since the window itself doesn't leak, freeing
the list on object destruction is enough.

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

mpegts: fix invalid memory access in language descriptor

Also add a couple sanity asserts/returns

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

mpdparser: Report error if Representation is invalid

Spec "5.3.5 Representation" is saying that
id and bandwidth attributes are mandatory fields.

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

adaptivedemux: Retry downloading a fragment immediately if any in live streaming

At the moment that demux is waiting manifest update, the target sequence
of fragment was advanced already. So, checking stream_has_next_fragment()
means looking for the next fragment of target fragment.
This might cause unexpected buffering if each fragment has
large duration and manifest is listing only limited number of fragments.

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

dashdemux: Fix SEEK of multi-period On-Demand profile

For each period, media presentation is the relative to the
period-start time. So SIDX seek position should be target seek
position minus period-start. Also, if presentationTimeOffset
is defined, the value should be compensated

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

ttmlparse: push segment events downstream

Without this, subtitles will stop after seeking.

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

ttmlparse: don't filter out untimed whitespace

Significant whitespace in elements that don't have begin/end values
should inherit timing from its parent, or if no its parents have no
timing, from the document's Root Temporal Extent. Currently, such
whitespace is removed, which is not spec-compliant. Fix this by
retaining whitespace in content nodes, and assigning a Root Temporal
Extent of 24 hours to any significant whitespace whose parents have no
associated timing.

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

ttmlparse: don't leak caps.

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

ttmlrender: improve function comments/descriptions

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

ttmlrender: relocate type definitions to head of file

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

ttml: correctly implement lineHeight behaviour

The specified behaviour in TTML when lineHeight is "normal" is different
from the behaviour when a percentage is given. In the former case, the
line height is a percentage (the TTML spec recommends 125%) of the largest
font size that is applied to the spans within the block; in the latter
case, the line height is the given percentage of the font size that is
applied to the block itself.

The code doesn't correctly implement this behaviour; this patch fixes
that.

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

ttmlrender: apply correct line height to blocks with multiple text sizes

In TTML, the height of every line in a block is determined by lineHeight
and fontSize style attributes, and should be the same for each line in
that block, regardless of whether different sized text appears on
different lines. Currently, a single PangoLayout is used to lay out all
the text in a block; however, pango will vary the line height in a
layout depending on the size of text used in each line, which is not
compliant with TTML.

This patch makes ttmlrender lay out the lines in a block itself, rather
than using a PangoLayout to do the work. The code still uses a
PangoLayout to render the text of each element, but the overall layout
of the text in a block is now controlled by ttmlrender itself. By doing
this, ttmlrender is able to ensure that the height of each line in a
block is correct.

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

ttmlrender: add facility to stitch images in both directions

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

ttmlrender: extend UnifiedElement type to include font metrics

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

ttmlrender: extend UnifiedBlock type

Include a reference to the GstSubtitleStyleSet of the represented block
and a string containing the concatenated text from all elements.

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

ttml: add refcounting to GstSubtitleStlyeSet type

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

ttmlrender: rename TextRange type and its fields.

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

ttmlrender: store PangoLayout in object state

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

ttmlrender: put all functions in gst_ttml_render namespace

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

gl: Remove duplicate typedef of GstGLFramebufferClass

The GstGLFramebufferClass struct is typedeffed in
gstgl_fwd.h, and having a duplicate elsewhere is
breaking the cerbero build on my OSX machine,
even though it seems to be working in CI.

tests: dash: don't use deprecated g_object_newv()

videoaggregator: Do not mix the same buffer twice when EOS.

When entering this code path, we know that:

We received EOS on this pad.
We consumed all its buffers.

In any case, we want to replace vaggpad->buffer with NULL,
otherwise we will end up mixing the same buffer twice.

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

Release 1.11.90

Update .po files

po: Update translations

win32: Update libgsturidownloader.def

adaptivedemux: add some reasons on warning reporting

directsoundsrc: Correctly calculate segsize and segtotal

segsize should be based on latency-time, and must be a multiple of the
frame size. segtotal should be based on buffer-time and segsize.

This prevents errors caused by outputting buffers that are not a
multiple of the frame size, and actually makes the buffer-time and
latency-time properties do what they're supposed to do.

uridownloader: Use a GWeakRef to the parent to prevent a reference cycle

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

adaptivedemux: set uridownloader's parent

Allows internal http source to request contexts and share cookies
with the pipeline in case the server imposes any restriction based
on cookies.

uridownloader: add new gst_uri_downloader_set_parent

If set, the parent is used to proxy need-context messages from
uridownloader's http source in order to get cookies/headers
from the pipeline.

Based on a patch from Philippe Normand

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

examples: add audiomixmatrix test to .gitignore

glwindow/cocoa: fix warnings reported by XCode 8.1.0

gstglwindow_cocoa.m:345:16: error: unused variable 'window' [-Werror,-Wunused-variable]
  GstGLWindow *window = GST_GL_WINDOW (window_cocoa);
               ^
gstglwindow_cocoa.m:445:11: error: unused variable 'external_view' [-Werror,-Wunused-variable]
  NSView *external_view = (__bridge NSView *)priv->external_view;
          ^

audiointerleave: don't overflow channel map with >64 channels

When there are more than 64 channels, we don't want to exceed the
bounds of the ordering_map buffer, and in these cases we don't want to
remap at all. Here we avoid doing that.

Based on a patch originally for plugins-good/interleave in
https://bugzilla.gnome.org/show_bug.cgi?id=780331

gldisplay: Fix assert caused by missing debug category

meson: Fix mplex plugin file name

Meson was creating libgstmplex2.so which didn't match the plugin name
'mplex'.

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