glcontext: fix get_current_gl_api on x11/nvidia drivers

They require to get_proc_address some functions through the
platform specific {glX,egl}GetProcAddress rather than the default
GL library symbol lookup.

glcontext: consolidate get_proc_address function definition

Pass the GstGLAPI directly.

glcolorconvertelement: propagate failure to convert buffer upstream

Rather than just silently continuing

glcontext/glx: try creating a context with the highest version

nvidia drivers return the exact version in glGstString (GL_VERSION)
we request on creation so start with the highest known version and
work our way down.

glcontext: track sharedness with a cookie

The previous approach of traversing the other_context weak ref tree was
1. Less performant
2. Incorrect for context destruction removing a link in the tree

Example of 2:
c1 = context_create (NULL)
c2 = context_create (c1)
c3 = context_create (c2)
context_can_share (c1, c3) == TRUE
context_destroy (c2)
unref (c2)
context_can_share (c1, c3) returns FALSE when it should be TRUE!

This does not remove the restriction that context sharedness can only
be tracked between GstGLContext's.

glcontext: use the debug object variant for completeness

gtkgstglwidget: Cleanup unused private member

new_buffer has been moved to base class. Also cleanup
the properties comment, which are also all moved into
the base class.

gtksink: "widget" must be access from main thread

Document that "widget" property must be accessed from the
main thread (where GTK is running). This is the same for
state transition on these elements. It is very natural to
do so un GTK applications.

gtkglsink: Don't leak vertex array and buffers

This is now possible since reset is always called from the
main thread.

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

gtkgstbasewidget: Fix black frame on resize

This is solved by only applying the new format when the next
buffer is to be rendered and on the GTK thread.

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

gtkgstbasewidget: Pass already parsed VideoInfo

As the base sink already parse the caps into VideoInfo it
makes sense to pass in VideoInfo to the widget instead.

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

gtkglsink: Port to GstGtkBaseSink base class

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

gtksink: Port to GstGtkBaseSink

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

gtkbasesink: Create a base class

This contains all the common code between the gtkglsink and
gtksink.

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

gtkglsink: Port to GtkGstBaseWidget

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

gtksink: Port to GtkGstBaseWidget

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

gtk: Add GtkGstBaseWidget

This is a "pseudo" base class. Basically it's a shared instance
and class structure and a shared set of function between the
two widget. It cannot have it's own type like normal base class
since the one instance will implement GtkGLArea while the other
implements GtkDrawingAreay. To workaround this, the parent instance
and class is a union of both.

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

uploadelement: Protect against NULL pointer

I notice that if you stop the pipeline during a renegotiation
the upload may be NULL while an allocation query is being run.
In that scenario, returning FALSE to the allocation query is the
best thing.

gtkgstglwidget: Remove unused gl_caps

gtksink: Create a window if the widget is unparented

The same way as it's now done with the gtkglsink, create a top
level window if the widget is not parented.

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

gtksink: Ensure the copy pasted code remains the same

Move back the default property at the same place they are in the
other sink. This helps when using a diff viewer to synchronized
this unfortunate copy paste.

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

gtk: Fix race between queue_draw and destroy

In GTK dispose can be called before the last ref is reached. This
happens when you close the container window. The dispose will be
explicitly called, and destroyed notify will be fired. This patch
fixes this race by properly tracking the widget state.

In the sink, we now set the widget pointer to NULL, so the widget
will properly get created again if you set your pipeline to NULL
state after the widget was destroy, and set it back to PLAYING.

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

dashdemux: remove typefinder, move it to typefindfunctions in -base

Otherwise this plugin will be loaded pretty much whenever
there's anything to typefind. And the code was pretty much
copy'n'paste from -base anyway.

dashdemux: default timescale is 1

default timescale value should be 1, not 0.

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

dashdemux: free prop_string only after it's no longer needed

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

Update mailing list address from sourceforge to freedesktop

gtkgstwidget: Add missing break in get_property

gtksinks: Remove undefined private structure

The classes contains a private structure which are not defined,
hence unused.

glupload: memcpy on raw data upload

Anything else requires keeping track of the GstVideoFrame mapping
across possible multiple buffers to ensure correct data pointer
usage.

glbasebuffer: add some debug and zero the data pointers on init

mpegtsmux: don't wait for data on sparse inputs like metadata or subtitle streams

mpegtsmux: add basic support for asynchronous KLV metadata streams

This is defined in SMPTE Rp 217. In this case the
metadata PES packets carry no timestamps of their
own and no Metadata Access Unit Wrappers are used.

vtdec: handle non-consecutive GstBuffer input without copying

CMBlockBuffer offers a model similar to GstBuffer, as it can
consist of multiple non-consecutive memory blocks.

Prior to this change, what we were doing was:

 1) Incorrect:

   CMBlockBufferCreateWithMemoryBlock does not copy the data,
   but we gst_buffer_unmap'd right away.

 2) Inefficient:

   If the GstBuffer consisted of non-contiguous memory blocks,
   gst_buffer_map resulted in malloc / memcpy.

With this change, we construct a CMBlockBuffer out of individual mapped
GstMemory objects. CMBlockBuffer is made to retain the GstMemory
objects (through the use of CMBlockBufferCustomBlockSource), so the
original GstBuffer can be unref'd.

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

applemedia/avsample: fix unconditional use of OSX 10.10 API

We can just not use the error/status properties

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

configure: try harder to find the right Qt5 build tools

First check for Qt build tools in the host_bins directory
from the Qt5Core pkg-config file. This fixes the situation
where both Qt4 and Qt5 are installed but the global moc/uic/etc.
are the Qt4 version, which would result in build failures.

configure/qt: Fix build without Qt5X11Extras

mpdparse: g_slice_new() can't return NULL

It will abort() if allocation fails.

mpdparser: Don't allow network access when reading the XML data

qt: don't error out if Qt X11 could not be found

PCK_CHECK_MODULES default error-if-not-found implementation errors
so override it with a simple notice that we couldn't find the library.

new qt5 qml GL video sink

Very much in the same spirit as the Gtk GL sink

Two things are provided
1. A QQuickItem subclass that renders out RGBA filled GstGLMemory
   buffers that is instantiated from qml.
2. A sink element that will push buffers into (1)

To use
1. Declare the GstGLVideoItem in qml with an appropriate
   objectName property set.
2. Get the aforementioned GstGLVideoItem from qml using something like

QQmlApplicationEngine engine;
engine.load(QUrl(QStringLiteral("qrc:/main.qml")));

QObject *rootObject = engine.rootObjects().first();
QQuickItem *videoItem = rootObject->findChild<QQuickItem *> ("videoItem");

3. Set the videoItem on the sink

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

bluez: Fix make dist while bluez >= 5 is not available

mpegtsbase: Fix previous commit

We only want to do a hard reset of the observations if we're working
with TIME segments in push mode. For BYTE segment we want to keep
the observations (in order to do seeks in push-mode).

mpegtsbase: Flush more on DISCONT buffers in push mode

When in push mode, we want to discard all previous observations from the
mpegtspacketizer when we get a DISCONT buffer.

This avoids trying to calculate bogus timestamps (estimating them using old
PCR observations).

We only do a hard reset in push-mode. In pull-mode we still need the observations
(in order to seek properly)

gitignore: Add new generated files

bluez: Include new file in dist'ed files

Fixes make distcheck and tarball generation

gtk: add to the generic/states test

GstGtkGLSink: Ensure widget has a toplevel parent

Checking for a parent is not enough, it must have a toplevel one.
If widget has no toplevel parent then add it in a GtkWindow, that
make it usable from gst-launch-1.0.

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

GstGtkGLSink: Post error if widget gets destroyed

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

GstGtkGLSink: fix possible warning in finalize

If the element is finalized before going in READY state
the widget could still be NULL.

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

mpdparser: Fix some memory leaks in the MPD parser and unit test

dashdemux: tests: added unit tests to test all functions

Added unit tests for all functions. Code coverage:
Overall coverage rate:
  lines......: 83.8% (1941 of 2316 lines)
  functions..: 100.0% (141 of 141 functions)

pngparse: fix a GstCaps object leak

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

tsdemux: remove unnecessary check

This is not public API, use g_assert() instead of
g_return_if_fail(), so that it's compiled out in
releases. It's only called from our code, with &foo.

tsdemux: add support for KLV metadata streams

There's no timestamps for these streams though, we
might want to make some up based on the last/next
video PTS or so.

Revert "Revert "dashdemux: fixed gst_mpd_client_advance_segment to return GST_FLOW_EOS""

This reverts commit 4875ddf5855a26f349df7b385b68eb692e314bfa.

This was based on a misunderstanding of the code.

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

gl/eagl: Don't call anything synchronously from the main thread

This will deadlock if the main thread is the one who creates the GstGLContext.
All things we call from the main thread should be possible from any thread.

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

bluez: remove unnecessary goto

All goto fail happen before ret is set. ret must be NULL, and the only
thing the fail statement block does is return NULL. Replacing the jumps to
do this return directly.

CID #1311329

dashdemux: remove redundant else statement

Else statement is redundant when the if block finishes with an return.

androidmedia: Add support for H265/HEVC

vtdec: fix refcount error in error handling

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

adaptivedemux: Explicitly check for CUSTOM_ERROR when converting errors to EOS

We use things like CUSTOM_SUCCESS* in other places, and those are not errors
but should just be handled like OK.

Revert "dashdemux: fixed gst_mpd_client_advance_segment to return GST_FLOW_EOS"

This reverts commit dfe37ffc59fdf5f4075696976d98ec8d6db73138.

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

mpdparser: Fix off-by-one in has-next-segment calculation

adaptivedemux: If downloading the last fragment fails, consider this as EOS and not ERROR

Sometimes the last fragment does not exist because of rounding errors with the
durations. Just finish the stream gracefully instead of erroring out instead.

download: only start a download transfer for sysmem caps features

dashdemux: tests: added indentation in xml strings to improve readability

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

dashdemux: added parsing of bitstreamSwitching@AdaptationSet

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

dashdemux: regrouped functions in gstmpdparser.h file

Moved gst_mpd_client_get_next_segment_availability_end_time and
gst_mpd_client_add_time_difference functions to be grouped with
functions from the same category.

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

dashdemux: tests: corrected code layout

deleted the empty line between gst_mpd_parse call and its check so that
the assert is next to the function it is testing

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

mpegvideometa: add meta transform function

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

glsyncmeta: transform func: return FALSE if not supported or failed

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

dashdemux: fixed gst_mpd_client_advance_segment to return GST_FLOW_EOS

Fixed gst_mpd_client_advance_segment to return GST_FLOW_EOS if the
new index is out of range.

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

android: add missing egl.h include

gltransformation: correct vao usage

keep the vao bound after uploading the new vertex data

fixes a mesa GL error "no vertex array object bound" on caps changes

gl: consolidate egl header includes to egl-only headers

They may conflict with other headers.

gtkglsink: Release the widget lock when trying to get the GL context

Otherwise we might be waiting for the lock on the main loop (for
example in the ->render vmethod) and thus we will deadlock.

videoaggregator: Remove pixel-aspect-ratio field from the caps returned by getcaps

Avoiding not negotiated error while negotiating as we anyway force 1/1
as output

curlsshsink: remove redundant else statement

Else statement is redundant when the if block finishes with an return.

dashdemux: fixed segment start when specified by SegmentTimeline

@t attribute in segment Timeline is relative to the period start.
Corrected the code to take this into account when computing segment
timestamp.

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

dashdemux: wrong segment duration computed by gst_mpd_client_setup_representation

Corrected computation of segment start and duration.

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

dashdemux: corrected search for audio languages

Corrected the initialisation of mimeType in
gst_mpdparser_get_list_and_nb_of_audio_language: the variable is used
in a loop, so it must be set to NULL at the beginning of each iteration.

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

Automatic update of common submodule

From f74b2df to 9aed1d7

glvideomixer, glmixer: Add description and klass

build: Fix broken BlueZ configure check

The previous commit left an extra set of brackets, and missed removing
the explicit dbus check (which is no longer needed since we're using
gdbus).

build: remove support for bluez4

bluez: refactor to use glib and add connection state tracking

pcapparse: fix another regression

Introduced by c4c9fe60b pcapparse: Take buffer directly from the adapter

Using gst_adapter_take_buffer_fast() can lead to buffers that are
made up of multiple memories with the first memory smaller than the
RTP header size, which violates assumptions GstRtpBaseDepayloader
makes, namely that the complete RTP header will be in the first
memory. This leads to such packets being dropped when feeding
them from pcapparse to RTP depayloaders. Use take_buffer() so
we get buffers with a single memory.

tests: pcapparse: add unit test for frames with eth padding

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

pcapparse: fix regression when handling packets with eth padding

Introduced by c4c9fe60b pcapparse: Take buffer directly from the adapter

Flush any trailing bytes after the payload from the adapter as well,
otherwise we'll read a bogus packet size from the adapter next and
then everything goes downhill from there.

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

openh264dec: Unref video frames and codec state in all cases

gl: initialize output params to 0 before calling gl functions

The client side API of the Chromium's GPU Process has asserts
in debug mode that check that output params are initialized to 0.

gl: Don't leak pool if set_config failed

glfilter: Don't cache buffer pool

Caching and sharing to multiple element the same pool prevents
renegotiation from passthrough to not passthrough.

openh264dec: Uninitializing the decoder can't fail and will always return 0 == cmResultSuccess

... but when we cast cmResultSuccess to a gboolean it will be FALSE, and make
GstVideoDecoder::stop() fail for no reason.

glimagesink: use g_clear_error instead of g_error_free

replace g_error_free with g_clear_error, as it internally
checks if error variable is valid or not.

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

amc: fix check whether we managed to acquire a buffer

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

srtpdec: Add config for the replay window size

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

dashdemux: corrected parsing of %u in segment templates

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

dashdemux: corrected recognition of $$$ in segment templates

fixed recognition of $$ after a $ that ends an identifier, eg $Time$$$

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

androidmedia: Call initialize_classes() also when we got the Java VM from the app

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