dashdemux: fix memory leak in gst_dash_demux_update_manifest_data

new_client local variable was not freed in case of errors.
Also, the buffer is not unmapped.

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

mpdparser: renamed gst_mpd_client_get_next_segment_availability_end_time to gst_mpd_client_get_next_segment_availability_start_time

The function actually returns the segment availability start time (as defined by the standard).
That is at the end of the segment, but it is called availability start time.

Availability end time is something else (the time when the segment is no longer
available on the server). The function name was misleading.

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

mpdparser: corrected getting segment availability

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

mpdparser: tests: added unit test for getting segment availability when segment timeline is used

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

adaptive_demux: tests: improved validation of pads

When removing a pad, the on_demuxPadRemoved function must find a stream
for that pad.

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

adaptive_demux: tests: fix pad used for AppSink event

on_demuxNewPad registered the on_appsink_event callback on a wrong pad.

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

adaptivedemux: tests: remove unused demux_sent_eos callback

The demux_sent_eos callback is unused in tests. It was also registered on
a wrong pad, so it actually triggered when demux received eos from a
fragment download.

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

adaptivedemux: tests: remove unneeded youtube xmlns from manifests

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

applemedia: vtdec: implement GstElement::set_context

applemedia: vtdec: update copyright

applemedia: avfvideosrc: implement GstElement::set_context

Needed to properly handle gst.gl.app_context(s).

applemedia: avfvideosrc: add myself to the copyright holders

gl: update win32 .def file

glsyncmeta: add compatibility definition for GL_TIMEOUT_IGNORED

glsyncmeta: separate out gpu/cpu waits.

CPU waits are more expensive and are only required if the CPU is ever going to
access the data. GPU waits perform inter-context synchronisation and are cheaper
as they don't require CPU intervention.

adaptivedemux: remove overwritten value

ret is overwitten before used with the return of the subclass seek.

CID #1352044

resindvd: remove commented out include

Commit 211828979b8c10abf3c74d964bc698dbda4d497d removed rsnaudiomunge,
no need for this commented out include for a file that doesn't exist
anymore.

Revert "resindvd: get rid of _stdint.h include"

This reverts commit 0eff6e1991cb24cb65f07c481742782acaa8ec62.

Breaks the build when linking with dvdnav.h

x265enc: get rid of _stdint.h include

It was never needed since it doesn't use any standard integer fixed width
types. Only the Glib types.

resindvd: get rid of _stdint.h include

po: update POTFILES

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

gleffects: identity: add the shader to the hash table

So that we don't recreate it every frame and leak memory.

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

tests: extend the AM_TESTS_ENVIRONMENT from check.mak

To get the CK_DEFAULT_TIMEOUT defined for all tests

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

Automatic update of common submodule

From e97c9bb to b64f03f

gstrtph265depay: keep consistency with rtph264depay

Use gst_rtp_drop_meta() and the same function prototype for
gst_rtp_copy_meta() to keep consistency with the RTP elements in
gst-plugins-good

tinyalsasink: Use glib CLAMP() instead of our own macro

rtph265depay: fix termination of access unit

Only consider the access unit complete when the next-occurring VCL NAL unit
has the first bit after its NAL unit header equal to 1.

adaptivedemux: Update position in stream->segment for new stream segment message.

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

tinyalsasink: Limit period size and count to what the h/w permits

gl: add .def file for msvc builds

Also internalize some API from being exported.

tests: mssdemux: add unit tests

Adds unit tests similar to the ones that we have for DASH and HLS.

Tests:

* manifest parsing finishes successfully
* some queries (duration, seekable, latency)
* seeking with various values and flags

mssdemux: implement snap seeking

Implement snap seek flags handling in stream_seek to allow the
parent class to handle it for us

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

mssdemux: use correct seek position on reverse seeks

Otherwise it was always using the 'start' value, leading to wrong
behavior

mssdemux: only update download position if the seek requests to

Some seeks are only updating the stop position, there is no need
to change the current downloading position.

mssdemux: do not assert on fragment reloads for non-live

It can be used as a resource to verify if the server has updated
something in the Manifest when downloads are failing

tests: dashdemux: add tests for snap flags seeking

Similar to HLS but DASH has the extra issue that it can have
multiple streams so snapping can be tricky as streams usually
won't be aligned.

For now, those tests handle the case of only having a single
stream.

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

dashdemux: implement snap seek handling

Handle snap seeking at the stream_seek method and let superclass
do the rest to support snap seeking

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

adaptivedemux: handle snap seeks

Adaptive demuxers need to start downloading from specific positions
(fragments) for every stream, this means that all streams can snap-seek
to a different position when requested. Snap seeking in this case will
be done in 2 steps:

1) do the snap seeking on the pad that received the seek event and
   get the final position

2) use this position to do a regular seek on the other streams to
   make sure they all start from the same position

More arguments were added to the stream_seek function, allowing better control
of how seeking is done. Knowing the flags and the playback direction allows
subclasses to handle snap-seeking.
And also adds a new return parameter to inform of the final
selected seeking position that is used to align the other streams.

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

adaptivedemux: add utility function to get stream from pad

Simplifies the code a bit and avoid repeating this
common operation

h264parse: remove unused mview_mode value

Since commit b77f8e172a3f0be6be5cb4a72e654253404e694f the new value
assigned to mview_mode hasn't been used. That commit changed the following
"if" check to an "else if", which means the original value of mview_mode
is used.

glquery: remove unnecessary pointer check

All uses of query->context in gstglquery.c assume it exists. We can assume
this as well before unrefing it. Furthermore, gst_object_unref() will just
silently return if it ever were to not exist.

h265parse: avoid sending unnecessary downstream caps queries/events

h265 versions of the following commits:
f352691a04896d0de3381fe8ee85ada948bd6337
try the current caps before querying downstream
72bc7d7f736a10117fedb8e1d4013a4946cfefbc
increase caps equality check for no codec_data

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

h264parse: increase caps equality check for no codec_data

When converting from avc to byte-stream, there will not be any codec_data
in the src caps.  Remove it before the equality check to avoid sending caps
events downstream on every SPS/PPS change.

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

h264parse: try the current caps before querying downstream

If we have a stream that contains an unchanging SPS/PPS for every video frame,
we don't need to to constantly query downstream for it's supported caps if the
current caps are compatible with the negotiated caps.

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

glbasefilter: enable qos by default

Improves the responsiveness of the pipeline when resources are close/above the
limitations of the hardware.

Any subclass that wishes not to enable qos can do so themselves.

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

amcvideodec: pass the correct time value to wait_for_sync

When we are not waiting, we need to pass -1 to signal that we just want to check
that the frame was/n't rendered.  Avoids waiting for frames that will never be
rendered.

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

amcvideodec: advance the ready counter ourselves when render=FALSE

When not rendering the video frame, e.g. when freeing an unreleased sync frame,
we will not receive a frame listener callback.

Reduces the amount of 'on_frame_available miss detected' messages when dropping
frames.

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

decklinkaudiosrc: Fix discont tracking

Don't reset the marker that's tracking disconts until
either the discont disappears or we resync.

Remove ancient and dead cdrom control plugin

This was never even ported to 0.10, and I don't think
it's particularly useful, since it's just a control
interface really. Let's remove it.

docs: remove some leftover 'Since 0.10.x' markers

debugutils: use generic marshaller

opus: fix FEC

FEC may only be used when PLC is enabled on the audio decoder,
as it relies on empty buffers to generate audio from the next
buffer. Hooking to the gap events doesn't work as the audio
decoder does not like more buffers output than it sends.

The length of data to generate using FEC from the next packet
is determined by rounding the gap duration to nearest. This
ensures that duration imprecision does not cause quantization
to 2.5 milliseconds less than available. Doing so causes the
Opus API to fail decoding. Such duration imprecision is common
in live cases.

The buffer to consider when determining the length of audio
to be decoded is the previous buffer when using FEC, and the
new buffer otherwise. In the FEC case, this means we determine
the amount of audio from the previous buffer, whether it was
missing or not (and get the data either from this buffer, or
the current one if the previous one was missing).

gltransformation: reconfigure the src when changing to/from passthrough

Otherwise it's very possible that any GL resources have not been created yet.

glvideoflip: correctly update the output caps on method changes

When changing methods we may need different output caps.

tinyalsasink: fix tinalsa typo

tinyalsasink: fix sample launch line

libs: player: use configured GST_PKG_CONFIG_PATH when invoking the introspection scanner

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

opusdec: fix wrong buffer being checked for missing data

This caused a decoding error if the resulting (wrong) buffer size
was passed to the Opus decoding API.

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

tinyalsasink: Use int type if we support a single rate/channel count

Avoids using an int range if the field we're setting is not actually a
range.

Revert "gl/eagl: try getting a gles3 context"

This reverts commit 96b9666d596be115fd4b446ef846508b7fd05c24.
This reverts commit d11385d167d8843604ea23a05e2dea40b8bbac35.

This breaks the texture sharing with the applemedia elements as
CVOpenGLESTextureCache seems to have an arbitrary restriction on GLES2 only.

glcolorconvert: don't skip subset structures

We may need them to transform into a different set of formats.

Fixes YUV->YUV with two glcolorconverts, e.g:

format=I420 ! glcolorconvert ! glcolorconvert ! format=NV12

glbasefilter: always call gl_start when not called already

Fixes elements transitioning out of passthrough mode using GL resources that
hadn'e been allocated yet.

glcolorconvert: fix RGB,rectangle->planar YUV conversion

Converting to GRAY is no fun.  Another case of normalized vs unnormalized
texture coordinates.

glcolorconvert: fix YUY2/UYVY,rectangle->RGB conversion

1.0 / width does not offset by one pixel in rectangular textures (which use
unnormalized coordinates).

Provide the actual pixel offset as a uniform to the shader.

tinyalsa: Add a new sink plugin using tinyalsa

This uses the tinyalsa library to playback to an ALSA device. Future
work can include using the mmap interface.

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

glcolorconvert: perform better negotiation

1. Correctly describe what we can caps we can transform to/from.
   i.e. no YUV->YUV or GRAY->YUV or YUV->GRAY (except for passthrough).
2. Prefer similar formats and ignore incompatible formats on fixation.

tests: dashdemux: add test for updating stop position

Test that a seek that only updates the stop position works
as expected

dashdemux: only update current reading position when needed

If the seek doesn't set the start/stop position, no need to
reposition the download index.

dashdemux: respect seeking parameter

Instead of using the segment values, use the parameter
requested in the seeking argument

tests: dash: fix dash tests after index fix

8e788f284508612982e8e38336eed676c2eec35a broke one of the
dash mpd tests. This updates and fixes it.

gstglwindow: initialize navigation_loop to NULL

Useful when gst_gl_window.c::gst_gl_window_new is not used.
This is the case when using a custom GstGLWindow.
(ex: GstGLWindowGPUProcess from Chromium)

gl: eagl: fix build

gstglcontext_eagl.m:238:45: error: too few arguments to function call,
expected at least 4, have 3

gldownload: allow video/x-raw as input

...and just passthrough. This is useful for pipelines where downstream must be
non-GL but upstream can optionally be GL.

gl/eagl: try getting a gles3 context

Fallback to a gles2 context if that fails

glvideomixer: fix checker vbo leak

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

tests: fix warning in qml example

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

qt: Fix build for android

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

Remove dead snapshot plugin

Was never ported and doesn't look like
we want it or need it in this form, can
do the same with the libgstvideo sample
conversion utility API now, but better
and in a more flexible way.

rsvg: bump requirement to 2.36.2

Which allows us to simplify the configure check and
some special-casing in the code, and is plenty old
in any case.

Remove dead libfame plugin

Was never even ported to 0.10, and the lib
has been unmaintained for ever and doesn't
even seem to be packaged any more.

Remove dead audiofile plugin

This was never even ported to 0.10.

Remove gsettings plugin which was never ported

Don't think we need this any longer or want to
support it, and clearly no one has been missing
it all these years either, so let's just get rid
of it.

smoothstreaming: remove unused dependency on uridownloader

faad: simplify configure check and require faad >= 2.7

Just check whether LATM is defined which is only available
in 2.7 and later. Allows us to simplify the configure check
a little and we can get rid of some hackish workarounds for
problems with earlier version headers.

dvbsrc: drop unnecessary use of _stdint.h

mxftypes: Generate UUIDs according to RFC4122 version 4 (aka random)

Instead of fully random UUIDs, these should follow a scheme.

mxfmetadata: Properly write video line map data as an array of 32 bit integers

mxfmetadata: DMSchemes in the Preface is a required field

Even if it has 0 entries.

mxful: Fix Content Storage UL

mxfmetadata: Best effort values must be written and if unset must contain the distinguished value

It's invalid to not write them at all.

mxful: Add missing OperationalPattern UL, which fixes an off-by-one with the others

mxfmux: Our body partition is always complete and closed

We have no metadata in it.

mxfmux: Use IndexSID of 2 instead of 1

Some tools complain if essence and index have the same SID.

mxfmux: Rewrite body partition pack on EOS and mark it as complete/closed

amcvideodec: guard against not finding a valid frame in loop

Fixes sporadic crashes on finishing decoding a video.

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

amcvideodec: set our data pointer in java to NULL on close

The frame available callback can be called after deconfiguring the amc codec.
Guard against this by setting the back pointer to NULL on close() and ignoring
any NULL data pointer.

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

applemedia: refactor GL context code

Rework the GL context code. Now both avfvideosrc and vtdec can create an
internal GL context for pushing textures. Both elements will still try to
use/switch to a local context where available (including after RECONFIGURE
events).

glprototypes: fix parameter type of glGenBuffers

The number of buffers should be GLsizei instead of GLuint.

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

gleffects: fix gleffects fisheye shader compile error

On some embedded systems, sqrt() is not supported in the shader,
use the actual value of sqrt(2) instead.

Signed-off-by: Haihua Hu <b55597@freescale.com>
Bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=761271

glvideomixer: par may not exist in the caps

Fixes a critical in the gst-validate tests:

gst_structure_fixate_field_nearest_fraction: assertion 'gst_structure_has_field
(structure, field_name)