gl/dispmanx: Fix set_render_rectangle typo

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

mpegtsmux: fix buffer size mismatch in M2TS mode

In M2TS mode, we need an extra 4 bytes in the buffer, so need
to ensure the buffer can contain these. The allocation site
does not know the mode, so this is done in all cases.

meson: Don't set c_std to gnu99

Use the default for each compiler on every platform instead. This
improves our compatibility with compilers that don't have gnu99 as
a c_std.

dvbsrc: fix 'delivery system'-related messages

* Rephrase tune error to be delsys-neutral
* Refer to the actual check in the 'missing sanity check' warnings
* Use "Delivery system" instead of 'delsys'. The
  latter is OK as a shorthand in the code but not
  even a real word

bayer: fix _transform_caps method to preserve fields in given caps

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

waylandsink: Properly configure internal pool

There was a small omission in the code. It still worked, but the pool
configuration may not have been optimimal.

common: sync to latest revision

Fixes accidental common change when commit patch.

mpegtsdemux: Implement efficient program updates

If the parent bin can handle it, only add/remove the new/gone stream
instead of re-adding/re-moving everything

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

mpegtsdemux: Ensure we always get an upstream stream-id

This was a regression.

We only have a upstream-id via STREAM_START if we were in push-mode.
In pull-mode we need to create one.

Note: It would be good to eventually have that method (copied from
gst_pad_get_stream_id_internal()) public in the future

mpegtsdemux: Fix issues with GST_DISABLE_GST_DEBUG

Some variables were hidden or unused

mpegtsdemux: Switch to using GstStream/GstStreamCollection

For each MpegTSBaseStream, we have a GstStream object which
subclasses can extend with information.

For each program a GstStreamCollection is created with all
GstStream from each stream.

mpegtsbase: Remove duplicated code

Just have one function to check for private section streams

adaptivedemux: fix buffer size print format

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

hlsdemux: fix hls demux stream type inheritance

GstHLSDemuxStream inherits GstAdaptiveDemuxStream, not
GstAdaptiveDemux. Fixes memory corruption issues.

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

dvb/parsechannels: parse guard interval as a fraction

Fixes wrong default failover when parsing libdvbv5 formatted
channel configuration files.

dvb/parsechannels: add prototypes and reorganize for clarity

Update my email address

dvb: drop trailing whitespace and c&p leftovers

dvb/parsechannels: parse DVB-T hierarchy from v5 config file

Avoids relying on the H/W default when the information
can be extracted from the configuration file.

dvb/parsechannels: parse DVB-T lp and hp code rates

mpegts: Also clear packetizer on TIME DISCONT

When dealing with TIME-based input, the incoming stream could have
potentially changed completely.

In order to check whether it did or not, we need to re-check all sections
(PAT, PMT...). If it didn't, we will keep using the existing streams/pad,
and if it did we will act as if there was a program switch.

Fixes HLS streaming with decodebin3/playbin3

adaptivedemux: Calculate values before queue2

In order to calculate the *actual* bitrate for downloading a fragment
we need to take into account the time since we requested the fragment.

Without this, the bitrate calculations (previously reported by queue2)
would be biased since they wouldn't take into account the request latency
(that is the time between the moment we request a specific URI and the
moment we receive the first byte of that request).

Such examples were it would be biased would be high-bandwith but high-latency
networks. If you download 5MB in 500ms, but it takes 200ms to get the first
byte, queue2 would report 80Mbit/s (5Mb in 500ms) , but taking the request
into account it is only 57Mbit/s (5Mb in 700ms).

While this would not cause too much issues if the above fragment represented
a much longer duration (5s of content), it would cause issues with short
ones (say 1s, or when doing keyframe-only requests which are even shorter)
where the code would expect to be able to download up to 80Mbit/s ... whereas
if we take the request time into account it's much lower (and we would
therefore end up doing late requests).

Also calculate the request latency for debugging purposes and further
usage (it could allow us to figure out the maximum request rate for
example).

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

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

opencv: Fix memory leak by gst_caps_to_string

gst_caps_to_string function returned allocated memory.
So, It should be freed using g_free function.

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

glwindow: don't use g_thread_join() to join the navigation thread

Using g_thread_join() in _finalize() handlers may result in a deadlock
joining the current thread when the last reference is held by a signal
handler.

e.g.:

error 'Resource deadlock avoided' during 'pthread_join (pt->system_thread, NULL)'

The backtrace looks like this:
[...]
g_thread_join ()
gst_gl_window_finalize ()
gst_gl_window_x11_finalize ()
g_object_unref ()
g_value_unset ()
g_signal_emit_valist ()
g_signal_emit ()
gst_gl_window_send_mouse_event ()
gst_gl_window_mouse_event_cb ()
g_main_dispatch ()
[..]
g_main_loop_run ()
gst_gl_window_navigation_thread ()
g_thread_proxy ()
start_thread ()
clone ()

gluploadelement: fix leak of upload library object

When only linking the element, the upload object will be created from
_transform_caps() but will never be unreffed as the only case is in _stop().

Add an unref if non-NULL to a new finalize handler for this case.

vkdisplay: hold a weakref on the list of windows

It's possible that the window may have been destroyed when a winsys
event comes in for it.

Fixes an assertion in make -C tests/check generic/states.check

vkdevice: only unref the gstcontext query if non-NULL

Fixes an assertion in make -C tests/check generic/states.check

tests/gl: make sure all GL commands are executed on the GL thread

e.g. the final glGetError() must also be completed on the GL thread

gl/win32: remove egl implementation

It hasn't been used in ages and is dead code.

tsdemux: Add conditions to identify ATSC EAC3

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

dvb/parsechannels: make parsing errors delsys-neutral

DVB is a group of delivery standards but we support
several others. While at this, make message formating
more congruent.

meson: Setup pre-commit hooks when configuring

meson: update version

player: Remove Return documentation for function returning void

Release 1.9.90

Update .po files

po: Update translations

waylandsink: Actually use buffer pool config after setting it up

CID: 1373420

fix for https://bugzilla.gnome.org/show_bug.cgi?id=771871

a2dpsink: Drop some dead-code

Left over from the last cleanup.

CID: 1373418

fdkaacenc: fix accessing freed memory

The buffer data is not always copied in _Fill, and will be
read in _DecodeFrame. We unmap at the end of the function,
whether we get there via failure or early out, and keep a
ref to the buffer to ensure we can use it to unmap the
memory even after _finish_frame is called, as it unrefs
the buffer.

Note that there is an access beyond the allocated buffer,
which is only apparent when playing from souphttpsrc (ie,
not from filesrc). This appears to be a bug in the bit
reading code in libfdkaac AFAICT.

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

fdkaacdec: avoid memory corruption on decoding error

The buffer size is expected to be in multiples of the sample size,
not in bytes.

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

fdkaacenc: fix buffer leak

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

fdkaacenc: set framed=true on src caps

This fixes muxing in MPEG TS.

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

gl/dispmanx: Implements set_render_rectangle to adjust the position of window

We cannot set the x, y coordinate of the video frame at the dispmanx at
this point. We need to teach dispmanx backend to understand about
set_render_rectangle API to draw a video with other UI.

This patch keeps the current behavior which places video frame at the
center of the display if there is no set_render_rectangle call to the
dispmanx window.

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

gl: add necessary padding bytes to all public structs

fdkaacenc: set transmux on the fdkaac lib

Not doing so will fail to decode in a simple fdkaacenc ! fdkaacdec
pipeline, though would work if this goes through a file.

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

fdkaacdec: do not error out of out of sync return

The docs say we should continue feeding in data and decoding

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

fdkaac: fix error with AOT_MP2_AAC_LC removed from libfdkaac API

AOT_MP2_AAC_LC is a "pseudo AOT" which got removed after 0.1.4,
and maps to AOT_AAC_LC.

Remove mpegversion 2 from th caps to match.

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

meson: bluez: fix build

decklink: Update to SDK 10.8

OSX and Linux only, Windows needs someone on Windows to convert the .idl
files to C source code.

fdkaac: fix mixup setting interleaved output

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

player: Only report the initial duration if the query was successful

h265parse: Fix parsing of multiple NALs in the hvcC box

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

bluez: dist new header file

Fixes distcheck.

a2dpsink: Don't try to set device caps on avdtpsink

We can't actually configure the transport, so we should only be working
with whatever we get.

avdtpsink: Chain event handling up to basesink

bluez: Clean up and make a2dpsink functional again

Was crashing, and had a lot of cruft (like a capsfilter and a fakesink)
that was no longer required.

avdtpsrc: Add support for AVRCP metadata

Metadata from AVRCP is emitted as tags, and the duration from AVRCP is
used in queries by avdtpsrc.

openjpegdec: Properly offset and shift for all formats when converting

And while at it, also clean up some code.

jpeg2000parse: Don't pass NULL strings to the colorspace / sampling parsing functions

They assert on NULL.

waylandsink: Don't leak GValues in getcaps

waylandsink: Update our window size on configure event

This is specific to when the waylandsink is not being embedded. In
this patch we pass the render lock to the window so it can safely
call gst_wl_window_set_render_rectangle() with the new size.

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

waylandsink: Remove atomic operation

We already take the render lock from the wlqueue thread in some other
place which indicates that there is no use of this atomic instead of
a proper locking mechanism.

waylandsink: Do not pre-configure proposed pool

Upstream must configure the pool before using it. Pre-configuring
the proposed pool could hide bugs in upstream elements.

waylandsink: Properly draw black border in absence of viewporter

When we don't have a viewporter (scaling support), we can't use the
1x1 scaleup image trick. Instead, we need to allocate a buffer with
the same size as the area that need to have black background.

dx9screencapsrc: throw error for invalid screen index

Currently dx9screencapsrc prints a verbose warning in case the screen
index is out of range for the current number of detected monitors. This
value is then dropped.

However there is no initial indication (beside the console print) if it
worked or not. This may result in capturing an unwanted screen as it
would capture the last set index that was not rejected.

This patch sets the index regardless. Instead, the element throws an
error when it tries to run or getting caps for an invalid index.

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

gldisplay: also free the GWeakRef when removing dead contexts

Otherwise we leak GWeakRef's.

Found with make -C tests/check libs/gstglcontext.valgrind

wayland/Makefile.am: add missing libs

libgstwayland is missing a dependency on $(GST_PLUGINS_BASE_LIBS), so add it.

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

waylandsink: Destroy viewporter when done

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

build: add missing $(GST_PLUGINS_BASE_LIBS)

The linker flags for several plugins (dashdemux, mssdemux, hlsdemux) are missing
$(GST_PLUGINS_BASE_LIBS), so add them so they can find -lgsttag and -lgstapp.

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

waylandsink: Port to vmeta and GstVideoFrame

This add support for non-standard strides to be used. Note that
some extra work is needed for multi-plane format which may have
a different GstMemory object per plane. This is not currently a
problem since SHM interface is limited to 1 memory.

waylandsink: Don't propose multiple time the same buffer pool

The buffer pool API does not allow multiple of owner. This otherwise
lead to error when renegotiation take place. Aso consider the
allocation query "need_pool" boolean.

waylandsink: Promote debug trace into error trace

Otherwise those error may go unseen making debugging much
harder.

waylandsink: Factor-out the pool creation

waylansink: Implement _show_frame()

This fixes the property show-preroll-frame expose by the VideoSink base
class and removes redundant code.

rtponviftimestamp: Update unit tests after changing default value of D-bit

The D bit shall always be set to true for the first buffer.

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

decklink: Print the Decklink API error return values in debug output in all places

wayland: fix distcheck some more

rtponviftimestamp: Change default value of D-bit

The default value of D-bit is changed to TRUE so discontinuity
is set for initial request and seek request as well.

Only set the e_bit flag for the CUSTOM_DOWNSTREAM event if
a cached buffer exists.

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

configure: fix --disable-external

conditional "USE_VULKAN" was never defined

glcolorbalance: reconfigure on passthrough changes

Fixes an assertion when moving from passthrough to non-passthrough

Without an explicit reconfigure, glfiter won't have created the GL
resources such as the FBO, GL bufferpool, etc and basetransform will
allocate sysmem buffers instead.

player: Remove some leftovers from external GstPlayer

wayland: fix distcheck when wayland-scanner is not installed

waylandsink: Make wl_viewporter optional

This makes the viewporter interface optional. The end result is
obviously far from optimal, though it greatly helps testing on older
compostitors or gnome-wayland. We can make it strictly needed later when
this new interface get widely adopted.

wayland: Update from scaler to viewporter protocol

Signed-off-by: Scott D Phillips <scott.d.phillips@intel.com>
https://bugzilla.gnome.org/show_bug.cgi?id=767671

player: Fix unit test for minor API change

player: Remove boolean return from set_subtitle_uri()

It can't fail synchronously and is inconsistent with set_uri().

player: Don't change state to PLAYING immediately if a seek is pending

We first have to finish the seek (in PAUSED) and move to PLAYING once
the seek is actually finished (unless a new one is pending then).

configure: Depend on gstreamer 1.9.2.1

dvbsuboverlay: map with READWRITE for overlay blend

Just as in basetextoverlay [1], the frame to blend with the subtitles overlay
should be mapped with flags GST_MAP_READWRITE, because
gst_video_overlay_composition_blend() does both operations.

1. https://cgit.freedesktop.org/gstreamer/gst-plugins-base/commit/?id=1396f804

Signed-off-by: Víctor Manuel Jáquez Leal <vjaquez@igalia.com>
https://bugzilla.gnome.org/show_bug.cgi?id=771382

kmssink: update libdrm dependency to 2.4.55

The latest libdrm feature supported by kmssink is the universal plane, which
was added in 2.4.55. Let's update the dependency to that version.

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

tsdemux: handle a NULL name in DVB short event descriptor

Resolves the warning:

GStreamer-WARNING **: Trying to set NULL string on field 'title' on taglist.

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

adaptivedemux: Also never have the SINK flag set on adaptivedemux

Revert "decklinkvideosink: Scale down scheduled frame times to milliseconds"

This reverts commit 3b7e0d7de32f0728259d601daa13b9eeec3dd3ce.

It was a bug in the driver and is supposed to be fixed with 10.8 and newer.

adaptivedemux: fix typo in new API

Fixes supressed -> suppressed typo in previous commit

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

adaptivedemux: prevent to propagate source flag to itself

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

Automatic update of common submodule

From b18d820 to f980fd9

Automatic update of common submodule

From 2c31690 to b18d820

meson/gl: use separate deps for gl and glx

e.g. passing with_gl_api=gles2 would still build the glx code but not be
linking against the libGL library which is where the glX* functions are
located and would result in a linker error.

Solved by checking for the libGL library if either opengl or glx may be
needed and then disabling the corresponding deps as requested.