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.

meson: add build files for vulkan

meson: add build files for the gtk plugin

meson: add some starting build files for GL

Currently only works on linux with egl/glx + wayland/x11 but the general
principals have been layed out for adding the other GL platforms/winsys'.

kmssink: Scale up to the screen dimension

In most display sink, the logic is to use as much as possible
of the given window. In this case, the window is the screen,
hence it's logical to scale up.

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

kmssink: Fix selection of source region

The source region was scaled for display before being passed
to drmModeSetPlane, which resulted in a portion of the video
being cropped. While when crop meta was present, the rectangle
was not centered since we where using unscaled width/height.

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

gstplayer: remove leftover update_interval api

old: gst_player_set_position_update_interval
new: gst_player_config_set_position_update_interval

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

vkupload: add video/x-raw to the VulkanBuffer uploader

Allows upstream elements that use our memory to passthrough without copying.

vkutils: set the last element of the layer array to NULL

g_strv* functions require this to not read off the end of the array.

gl/examples: Really remove references to $(GST_PLUGINS_GL_*).

These seem to have been missed in the previous commit to this file.

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

gl: Remove duplicate GL_CFLAGS in Makefile.am.

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

kmssink: override stride if defined in driver

Some kms drivers demands specific pitches over the ones calculated by
GstVideoInfo. For example, intel driver demands strides round up 64.

This patch queries the driver for the prefered pitch and overwrites it
in the pool's GstVideoInfo structure.

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

aggregator: Use the event_full function for GstAggregatorPads

Allowing us to tell GstPad why we are failing an event, which might
be because we are 'flushing' even if the sinkpad is not in flush state
at that point.

qmlglsink: check qt_context_ first in GstQSGTexture::bind()

When start qmlglsink app, it will set NULL buffer to GstQSGTexture
in which case that qt_context_ will be a random value and cause
gst_gl_context_activate() fail.

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

build/vulkan: fix winsys detection based on the previously set variables

The X11/Wayland winsys checks weren't being used to enable the vulkan element.
Use them.

srtpenc: prevent deadlock in create_session

Temporarily release the object lock which is needed to post error
messages on the bus.

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

meson: Fix building with meson 0.34 when using subprojects

When using subproject meson.source_root() returns the root
directory of the toplevel project which break build.

Upstream meson bug: https://github.com/mesonbuild/meson/issues/722

meson: Bump version to 1.9.2

glupload: create the GstGLUpload object in ::transform_caps

Previously it was created in the init function and destroyed in ::stop, which
lead to segfaults when reusing the element.

Now the upload object is created in ::transform_caps if it is NULL, which is the
earliest we need it. The other vfuncs already bail out if the upload object is
NULL, which means that negotiation wasn't done.

tsdemux: Support ATSC EAC3

EAC3 bit streams shall be identified with a stream_type value of 0x87 when
transmitted as PES streams conforming to ATSC-published standards. It is specified
in ATSC Standard A/52.

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

decklink: Fix-up last commit that was not meant to be pushed yet

decklink: Fix indentation

decklink: Require a clock when going from PAUSED_TO_PLAYING and don't crash if there is none

Also when going from PLAYING_TO_PAUSED, the clock might've been unset in the
meantime, e.g. because the element was removed from its surrounding bin.

Back to development

Release 1.9.2

po: Update translations

applemedia: Conditionally compile qtkitvideosrc

The API was deprecated in 10.9 and apparently does not exist in the SDK for
macOS Sierra anymore.

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

gl: Remove extra duplicate typedef of GstGLFramebuffer

It's triggering a warning building git master on OSX

tests/glupload: support opengl3 properly instead of spewing GL errors

gl/pkgconfig: add the lib include dir to the list of includes

714d8aa in core removed this include dir from it's pkgconfig file.

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

glupload: fix tests and check for the new RECONFIGURE behaviour

The tests were broken since 91fea30, which changed glupload to return
GST_GL_UPLOAD_RECONFIGURE if the texture target in the input buffers doesn't
match the texture-target configured in the output caps.

This commit fixes that and adds more checks for the new behaviour.

Use the new API to post flow ERROR messages on the bus

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

dash: Fix build of unit test

dashdemux: Add properties to select maximum allowed width/height and framerate

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

decklinkaudiosink: Fix compiler warning on OS X

While gint64 and int64_t are always the same, clang does not agree with that.

/Applications/Xcode.app/Contents/Developer/usr/bin/make -C decklink
  CXX      libgstdecklink_la-gstdecklinkaudiosink.lo
gstdecklinkaudiosink.cpp:675:79: error: cannot initialize a parameter of type 'int64_t *' (aka 'long long *') with an rvalue of type 'gint64 *' (aka 'long *')
      ret = buf->output->attributes->GetInt (BMDDeckLinkMaximumAudioChannels, &max_channels);
                                                                              ^~~~~~~~~~~~~
./linux/DeckLinkAPI.h:692:87: note: passing argument to parameter 'value' here
    virtual HRESULT GetInt (/* in */ BMDDeckLinkAttributeID cfgID, /* out */ int64_t *value) = 0;
                                                                                      ^

applemedia: fix compiler warning

decklinkvideosink: Scale down scheduled frame times to milliseconds

Scale down to milliseconds, otherwise at least some hardware has problems
scheduling the frames (or schedules them too slow) and we run out of available
frames.

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

decklinkaudiosink: Add support for 8 and 16 channels

applemedia: changes to make GL memory mappable on CPU on iOS

This commit introduces IOSGLMemory which is a GLMemory that falls back to
GstAppleCoreVideoMemory for CPU access. This is a temporary solution until
IOSurface gets exposed as a public framework on iOS and so we can use
IOSurfaceMemory on both MacOS and iOS.

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

gstglupload: make the GLMemoryUpload method output the correct texture-target

Now when used with video/x-raw as input, the GLMemoryUpload method checks for
->tex_target in input GLMemory(es) and sets the output texture-target
accordingly.

Fixes video corruption with a pipeline like avfvideosrc ! video/x-raw !
glimagesink where on macos avfvideosrc pushes RECTANGLE textures but glupload
was configuring texture-target=2D as output.

meson: Make mpegts lib build when using subprojects

The headers passed as parametter are relative to the build dir
basically "../subproject/gst-plugins-bad/gst-libs/gst/mpegts/XXX.h"
but that does not match what is needed at build time when building as
subproject, also we always add current dir as include_dir so we are
safe including directly.

And link mpegtsdemux against the 'math' library as it is needed.

docs: Expose the opengl plugin and element docs

videoaggregator: Handle if update_caps() returns EMPTY

Don't assume the returned caps from update_caps() is
non-empty.