shmsink: fix possible deadlock in _render()/ _allocator_free()

Drop object lock before unrefing memory, otherwise the object
lock might be taken again from the allocator and then things
deadlock.

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

glfilter: First make caps writable in fixate_caps(), then truncate them

Truncating requires writable caps.

glcolorconvert: Caps passed to fixate_caps() are not owned by us and guaranteed to be fixed

So don't try to fixate them, which takes ownership and steals them from
basetransform.

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

Revert "glpool: fix caps refcount issue"

This reverts commit 3bdcdedfa00b87f1db505218789178318a16c743.

gst_caps_replace() takes a new reference already, if there is a problem then
it is elsewhere. And there are a few problems, see
https://bugzilla.gnome.org/show_bug.cgi?id=760696

rtph265depay: fix unneeded sub-buffer creation

We create a sub-buffer just to copy over its metas and then throw it
away immediately, just use the original input buffer directly.

rtph265pay: add "send VPS/SPS/PPS with every key frame" mode

It's not enough to have timeout or event based VPS/SPS/PPS information
sent in RTP packets. There are some scenarios when key frames may appear
more frequently than once a second, in which case the minimum timeout
for "config-interval" of 1 second for sending VPS/SPS/PPS isn't enough.
It might also be desirable in general to make sure the VPS/SPS/PPS is
available with every keyframe (packet loss aside), so receivers can
actually pick up decoding immediately from the first keyframe if
VPS/SPS/PPS is not signaled out of band.

This commit adds the possibility to send VPS/SPS/PPS with every key frame.
This mode can be enabled by setting "config-interval" property to -1. In
this case the payloader will add VPS, SPS and PPS before every key (IDR)
frame.

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

rtph265pay: change config-interval property type from uint to int

This way we can use -1 as special value, which is nicer than MAXUINT.

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

adaptivedemux: replace ghostpad with a standard pad

Handling the ghostpad and its internal pad was causing more issues
than helping because of their coupled activation/deactivation
actions.

As we have to install custom chain,event and query functions it is
better to use a floating sink pad internally in the demuxer and just
use those pad functions to push through a standard pad in the demuxer

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

adaptivedemux: do not flush the input pad

gst_adaptive_demux_stream_clear_eos_and_flush_state() function will do
all the necessary cleaning.

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

videosegmentclip: fail to set_caps if problem in caps structure

gst_video_segment_clip_set_caps () should return FALSE if there was a
problem reading the framerate values from the caps structure.

glquery: provide compat definition for GL_QUERY_RESULT

e.g. android doesn't have it

While we're here move a state specific assertion after the supported check.
Removes an assertion on GLES

gl: add compatibility definition of GLint64

e.g. anrdoid doesn't have it

qml: Mark material dirty when texture buffer is updated

Qt might not redraw the scene otherwise.

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

glpool: fix caps refcount issue

The caps are from an allocation query which are transfer none but were being
treated as transfer full.

glmemory: log the time for glTexSubImage/glReadPixels

gl: add a gstglquery object for arbitrary GL queries

Currently only GL_TIME_ELAPSED and GL_TIMESTAMP are supported

gldebug: implement a delayed debug system

The messages are stored by gst_gl_async_debug_store_log_msg() and output later
by a corresponding store(), output() or an unset()/free().

Some wrapper macros are provided to avoid callers explicitly using __FILE__,
GST_FUNCTION and __LINE__

gldebug: use gst_info_vasprintf in insert_debug_marker()

Allows expansion of GST_PTR_FORMAT and GST_SEGMENT_FORMAT arguments.

gl: add a videoflip element

Behaves exactly the same as the non-GL videoflip element

glfilter: recreate the fbo on caps changes

The width/height may change which requires recreating the depth buffer.

glbasefilter: add a class function on set_caps (from GL)

This is simply a convenience to avoid duplicating the thread marshalling on
set_caps.

gltransformation: always build a valid mvp matrix

The default case is to build an identity matrix.

gltransformation: support negative scales

A scale of -1.0 means to flip the video.

gltransformation: implement passthrough handling

gltransformation: implement navigation events

Reverses the transformation applied through the properties and forwards the
event.

The process for finding the coordinates on the video are as follows:
1. Convert the given pointer_x and pointer_y to model space at the near and far planes
2. Get the equation of the video plane
3. Find where the ray in 1 intersects the plane
4. Profit!

glimagesink: remove reduntant glimagesink from debug logging

gl/x11/navigation: add button press to the selected event mask

Otherwise we won't get the button press events and GstNavigation fails.

glcolorconvert: try harder to passthrough

This makes a pipeline like:

... ! video/x-raw(memory:GLMemory),format=UYVY ! glcolorconvert !
video/x-raw(memory:GLMemory),format={UYVY, NV12} ! ...

passthrough instead of converting UYVY => NV12. The conversion would happen
before this change since the element (and basetransform) transform the src caps
to format={NV12, UYVY} (since NV12 comes first in the glcolorconvert:src
template) and then the default caps fixate func would fixate to NV12. Blah.

Also there's no need to intersect against the template caps in ::transform_caps
since basetransform does that right after calling the vfunc.

gl: implement a colorbalance element

It performs the exact same operation as videobalance but with opengl shaders and
was tested with glvideomixer by comparing frames from videobalance and
glcolorbalance.

glvideomixer: implement glBlendFunc and glBlendEquation

Allows more blending options than just A over B

e.g. frame comparisons are now possible.
  glvideomixer name=m
    sink_0::zorder=0
    sink_1::zorder=1
    sink_1::blend-equation-rgb={subtract,reverse-subtract}
    sink_1::blend-function-src-rgb=src-color
    sink_1::blend-function-dst-rgb=dst-color
  ! glimagesinkelement
  videotestsrc pattern=checkers-4 ! m.sink_0
  videotestsrc pattern=checkers-8 ! m.sink_1

gldebug: add a specific category for the debug spam from the driver

sbc: sbcdec: Fix frame length calculation

SBC frame length calculation wasn't being rounded up to the nearest byte
(as specified in the A2DP 1.0 specification, section 12.9). This could
cause 'stereo' and 'joint stereo' mode SBC streams to have incorrectly
calculated frame lengths.

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

nvenc: update for recent GL api changes

Specifically the GstGLMemoryPBO addition (which was the unadorned GstGLMemory).

applemedia: avfvideosrc: optionally get the GL context from gst_element_set_context

Allow setting the GL context with gst_element_set_context. When available, the
local context obtained via the context query still has the precedence.

applemedia: iosurfacememory: remove NO_SHARE flag

The full memory can be safely shared. Sharing a sub region can't be done and
will fail in the base allocator, triggering a copy.

frei0rmixer: replace accept-caps with caps query

It wants to check if upstream can produce a certain format,
accept-caps might only check if the next element can produce it.

glbasememory: Free the actual memory object when requested

Otherwise we are leaking ~400B on each GstMemory allocation.

Freeing in the base class matches the GObject semantics.

glmemorypbo: don't double read pixels

The optimistic download_transfer was not setting the required flag to not
perform glReadPixels on subsequent map (READ). resulting in glReadPixels
happening twice.

glmemory: expose gst_gl_memory_texsubimage

Removes extremely similar code from glmemorypbo.

glfeature add a specific debug category

Saves having to trawl through the 'default' category when function retrieval
goes wrong.

mpegtsmux: fix reserve bits so they are 1's

The MPEG standard (ISO-13880-1) says the reserve bits need to be set
to one (2.1.64). This is causing transport streams to fail validation
on some systems.

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

opencv: remove check for OpenCV version

After commit 64080e632, configure checks for all the header files that
should be available in OpenCV 2.3 and later. If any of these files isn't
there the OpenCV elements won't be part of the build.

No need to recheck for opencv2/legacy/legacy.hpp again in
gstpyramidsegment.h. Minimum supported OpenCV version must have this header
and configure already checks for it. Removing check.

opencv: always do cvsmooth in place

After the update to new OpenCV API the transform function using an out
buffer is not necessary. We can always do the transformation in place.

opencv: do pyramidsegment's transformation in place

Run the transform function of pyramidsegment in place, reusing the image
data as both source and destination in cvPyrSegmentation. This avoids
copying the image back and forth and the extra memory.

tests: hlsdemux: add test for updating segment stop

Add a test for seeking that only updates stop position and verifies
that start is unmodified

adaptivedemux: fix seeking that just updates stop position

Fixed adaptivedemux seeking without flushing that just wants
to update stop position. This required protecting the segment
variables with a new mutex so that the seeking thread and the
download threads could safely manipulate the segment and
events related to it.

This contention is only locked/unlocked when starting a new
download, when the first fragment of a segment is received and
when seeking so, hopefully, it won't damage performance.

adaptivedemux: fix segment update on seeks

Fixes typo on reverse rate check and also only update the
position when the start/stop was actually changed.

mxftypes: Don't store pointers in stack allocated guints

This works usually in this place, unless the compiler optimizes things in
interesting ways in which case it causes stack corruption and crashes later.

The compiler in question here is clang with -O1, which seems to pack the stack
a bit more and causes writing to the guint as pointer to overwrite map.memory,
which then later crashes during unmapping of the memory.

mxfmetadata: Initialize boolean to FALSE to fix valgrind warning

Seems to be a false warning though.

glsyncmeta: only flush with a shared context

The wait code will flush for us for single context pipelines.

glcontext: implement checking whether a context has been shared

Some operations are unnecessary when running with only a single GL
context.
e.g. glFlush when setting a fence object as the flush happens on wait.

API: gst_gl_context_is_shared

hlsdemux: put boolean values into gboolean variables

And remove superfluous assignments.

player: Only set the pipeline to PLAYING in play() if buffering>=100%

Otherwise the application can break the buffering logic by setting the
pipeline to PLAYING before we buffered enough.

tests: dashdemux: add tests for post-seek segment boundaries check

Checks if the post seek segment is what is expected.

Also makes it easy to add more tests with different seeking flags using the
same functions.

tests: hlsdemux: add tests for seek with reverse rate and snap flags

Add tests to ensure snap flags work as expected for reverse
rates

adaptivedemux: stop reverse playback when we reach the limit

Avoids downloading and pushing a full segment just to get 1 nanosecond
of data. This happens frequently when seeking is done with flags
that adjust to boundaries or when the start is aligned with segment
starts. The later is common when segment durations is a multiple of
a second.

hlsdemux: fix reverse playback seek with snap flags

Properly handle snap flags during reverse seeking. In this case
the before/after are also reversed, so handle those as such.

For example: with a sequence of 1s fragments:

|-- 0 --|-- 1 --|-- 2 --|-- 3 --|

If you seek to 1.5s it is inside fragment 1. With reverse and

snap-before: should play from the end of fragment 1
snap-after: should play from the end of fragment 0

adaptivedemux: correctly track segment.position in reverse playback

For reverse, set position to segment.stop when starting and also
don't set the position to fragment end timestamp when it finishes,
just leave it at the fragment start.

hlsdemux: simplify snap flags checking

Replace:
if (boolean) var = true;

with:
var = (boolean);

hlsdemux: respect keyunit flag for position

Set the segment start position when keyunit flag is active

tests: hls_demux: add tests for seeking segment

Tests that check that the segment sent after a seek is correct.

Allows testing that multiple seeking flags work as expected

tests: adaptive_demux: add function to be able to check demuxer events

Allows writing tests that verify that events are correct.

Useful to monitor and check segments after seeks, for example.

tests: adaptive: update to allow more flexible seeking tests

Allows defining a seek event to be able to change seeking parameters
and create more seeking test scenarios

tests: hlsdemux: fix makefile variable typo

applemedia: avfvideosrc: enable renegotiation

We can actually renegotiate now, so remove old check which disabled it.

glcolorconvert: implement usage of a buffer pool

Saves unnecessary glGenTextures and glDeleteTextures which may have a
non-trivial cost.

glbasememory: fix copying GstGLAllocationParams

Fixes a GST_IS_GL_CONTEXT critical

glmemorypbo: remove unused code

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

applemedia: avfvideosrc: small negotiation fix

Fix negotiation when GLMemory is requested but a context is not available

applemedia: vtenc: accept UYVY on Mac

When doing GLMemory avfvideosrc negotiates UYVY. This change allows avfvideosrc
! tee name=t ! ... ! glimagesink t. ! ... ! gldownload ! vtenc_h264 ! ...
to do GLMemory and 0-copy with the encoder (with the CV meta).

applemedia: avfvideosrc: change texture format from BGRA to NV12 on iOS

Change texture format from BGRA to NV12. This allows a pipeline like avfvideosrc
! tee name=t ! ... ! glimagesink t. ! ... ! gldownload ! vtenc_h264 ! ...  to
negotiate GLMemory. This makes the glimagesink branch much faster (obviously)
and triggers the 0-copy path between avfvideosrc and vtenc (using the CV meta).
Combined this results in a huge perf improvement on iOS (25-30% of CPU time in a
pipeline like the one above).

Note that this doesn't introduce a new shader conversion in the sink, since BGRA
textures had to be copied/converted from format=BGRA,texture-target=RECTANGLE to
format=RGBA,texture-target=2D anyway.

glupload: always add texture-target field to GL caps

1. Various elements/base classes only perform a subset check on accept-caps
2. Some GL elements have texture-target in their pad template
3. When checking subsets, only the caps to check are allowed to contain extra
   fields.  If the 'template' caps have extra fields, the subset fails.
Thus without texture-target on the caps, various accept-caps implementations
were failing.

Also, add some convenience functions for setting and retrieving
texture targets to/from GValue.

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

glupload: passthrough composition caps features

Don't unconditionally add it to any and all caps transformations.

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

player: Remove gst_player_new() and make gst_player_new_full() the normal constructor

In very few cases the simple version was actually needed and having the
parameters hidden by a _full() version caused application that actually needed
it to not use it.

glviewconvert: Fix stereoscopic handling w/ texture-target

The addition of texture-target negotiation broke stereoscopic
handling when not in passthrough mode, so fix that.

sdpdemux: replace duplicated codes to call new base sdp apis

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

rtmp: refer to both elements in the README

rtmpsrc: check for failed RTMP context alloc

Avoids an unlikely crash.

Arguably, if allocation fails we have no chance of
recovering but nonetheless, RTMP_Alloc can fail and
librtmp's RTMP_init() (called next) assumes a non-NULL
pointer is passed without checking.

Additionally, unify exit path on error.

rtmpsink: check for failed RTMP context alloc

Avoids an unlikely crash.

Arguably, if allocation fails we have no chance of
recovering but nonetheless, RTMP_Alloc can fail and
librtmp's RTMP_init() (called next) assumes a non-NULL
pointer is passed without checking.

Additionally, unify exit path on error.

rtmp: correct librtmp log-level mappings

Additionally, move to a switch in _set_debug_level()
to make easier to follow and compare with the
mappings in _log_callback()

codecparsers: vp9: minor clean-up

Remove setting of parser variable to NULL after free,
that makes no sense (and coverity is no doubt going
to complain about it).

codecparsers: vp9: fix macro namespacing and rename GstVp9InterpFilter

codecparsers: vp9: fix doc typos and sprinkle some Since markers

codecparsers: vp9: move license blurb into header and remove aux files

This is just a bunch of lookup tables..

win32: Update libgstcodecparsers.def with the new symbols

codecparsers: Add another G_GNUC_INTERNAL

codecparsers: Spread some G_GNUC_INTERNAL in various places

vp9parser: Fix indentation to make gst-indent happier

vp9parser: Rename symbols to prevent symbol conflicts

Also make clamp() a static function for the same reason and use CLAMP (as
defined by GLib) in the GStreamer code.

codecparsers: Add VP9 codec parser

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

glmemorypbo: fix wrapping data on GL 2.1

GL 2.1 only supports pbo upload.

The wrapped data pointer was only being set on the pbo memory and on the
glmemory so when a download was requested (in GL 2.1), glmemory was
allocating a new data pointer and thus not returning the wrapped data.

glmemorypbo: only create a pbo memory if the context actually supports it

e.g. GL <= 2.0 does not support pbo usage and GL 2.1 only supports pbo upload.

glmemory: small code reformat

makes the alloc_params selections fit on a single line.

tests/glmemory: output data pointer values on failure

Allows quicker inspection of what failed.

gleffects: fix shader compilation with legacy opengl

All the gleffects shaders can be run against a gles2 or a legacy opengl glsl
compiler but weren't being advertised as such.

Fixes gleffects under desktop opengl < 3.2.

glbasememory: Remove bogus NULL check

CID 1346534

applemedia: avfvideosrc: output at highest fps/resolution by default

Fixate to the highest possible resolution and fps. Otherwise by default we end
up fixating at 2fps and the lowest supported resolution, which is hardly what
someone who bought an overpriced smartphone wants.

gl: Add \0 terminators for the Apple sync extension

Otherwise GL initialization might check for extensions forever and never finishes.

glcontext: Add some debug output when the GL thread is actually running

dashdemux: removed unnecessary space trimming in gst_dash_demux_parse_http_head

sscanf has removed all spaces when it has constructed zone string. There is
no need to search for leading spaces.

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