hlsdemux: drop TODO that doesn't need a solution

Connection speed is only checked at that point in hlsdemux so there
is no real need to refactor it.

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

Revert "hlsdemux: Simplify logic in process_manifest"

This reverts commit 4ca3a22b6b33ad8be4383063e76f79c4d346535d.

The connection-speed=0 is used as a special value in the property
of hlsdemux to mean 'automatic' selection, m3u8.c doesn't need
to know about that as it should be as simple as possible.

So this patch hides this automatic selection documented in hlsdemux
into m3u8 logic and I think the gets harder to understand the code.

It also makes the hlsdemux unit tests work again

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

mpdparser: Fix inverted logic introduced in last commit

simplevideomarkdetect: fix detect of videomark partially or fully outside video

In case of the videomark being partially or fully outside,
an error was bein thrown saying, mark width is more than video width.
And when the width, offset properties are set to maximum it resulted in crash.
Instead of throwing error, added logic to detect the mark
in case of partial visibility or dont show the mark when it is outside.

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

mpdparser: Calculate the number of segments we have when a segment template instead of a list is used

Otherwise we would just continue downloading new files forever until we get
404 for the first one, and then error out instead of going EOS.

simplevideomark: Add Error logs

When the pattern offset is outside the video, the print error message

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

Revert "mpdparser: Don't consider streams with a known media presentation duration as live"

This reverts commit 37011e519830e0786fa6e307ed26003a2a8774f1.

This change was actually completely unnecessary, the streams in question are
marked as static and are not considered live anyway.

mpdparser: Don't consider streams with a known media presentation duration as live

adaptivedemux: Error out if downloading a segment failed too often in live mode

It's true that we shouldn't consider errors fatal immediately, but if we
always ignore them we will loop infinitely on live streams with segments
that can't be downloaded at all.

srtp: Allow getting ssrc of more package types than RR and SR

This allows decrypting reduce size packages. See RFC 5506

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

tsdemux: refactor pad creation

Avoid repeating the same pad creation code everywhere

mpdparser: Don't crash in debug output if stream->segments is NULL

ksvideosrc: fix logic and timestamp non-muxed streams again

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

libde265dec: fix up plugin name and decoder description

mpdparse: Clamp seek times to the availabilityStartTime

Otherwise we will seek to negative times, which are interpreted as unsigned
integers later.

mpdparser: Improve debug output a bit when advancing segments

adaptivedemux: Release the manifest mutex before waiting for a download

Otherwise we will hurt parallel downloads in the best case, and often cause
deadlocks.

opusdec: If channel/rate negotiation fails, fall back to stereo and 48kHz

opusparse: Set up default header with 48kHz sample rate instead of 0

opusdec: gst_structure_fixate_field_nearest_int() only works if the structure has this field

Just set the rate/channels directly if the caps don't have this field.

mpdparser: Use 64 bit integer format strings when passing a 64 bit integer

Otherwise we'll only get half of its bits printed on 32 bit architectures.

For this, promote the %d-style format strings to something that accepts
64 bit integers with G_GINT64_MODIFIER.

mpdparse: Validate format strings before using them

Using format strings from an untrusted source without validation is
calling for problems, and at least allows to remotely crash your application.
If not worse.

gl/dispmanx: Fix build

gl/example: fix build error when compiling cocoa-videooverlay

libtool: error: ignoring unknown tag OBJC
and
clang: error: argument unused during compilation: '-pthread'

gl/window/dispmanx: fix compiler warning

gl/window: use the default main loop implementation for all backends

fixes glimagsink being unable to display.

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

glwindow_cocoa: use parent default implementation

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

mpdparser: Add the startNumber as offset to the segment index when using a template

dtlsdec: Fix critical warning "got data flow before stream-start event"

Forward sticky events on requested src pad.

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

vc1parse: remove useless value assignments

In all these cases ret is set but overwritten before the return of the function

examples: gst-camerabin2-test: protect from division by zero

Highly unlikely to have 0 captures, but protect from crashes in the future by
doing none of the math if there is no data.

configure: lower curl requirement to 7.35.0

Build and unit tests were tested against 7.35 too now,
which is the version in Ubuntu 14.04.

mpegtsbase: do not access variable after unref

Data might not exist anymore

tests: audiointerleave: test not setting positions

Disable "channel-positions-from-input", but without actually giving
a position table, so every position should be NONE

tests: Fix indentation in audiointerleave test

zbar: don't ignore zbar_scan_image() errors

glsyncmeta: Set new sync point when copying

Set a new sync point when copying instead of
transferring the (non-refcounted) GL sync object
from the source meta.

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

glsyncmeta: Add some debug output

check: Use GST_CHECK_MAIN () macro everywhere

Makes source code smaller, and ensures we go through common initialization
path (like the one that sets up XML unit test output ...)

simplevideomark: fix display of videomark partially or fully outside video

In case of the videomark being partially or fully outside, an error was being
thrown saying the mark width is more than video width. And when the width,
offset properties are set to maximum it resulted in crash. Instead of throwing
an error, add logic to detect the mark in case of partial visibility or don't
show the mark when it is outside.

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

tsdemux: enable Chinese AVS video stream

Chinese broadcaster encapsulate AVS video codec into MPEG2-TS. They
use the stream_id 0x42 to identify AVS video streams. It should be noted
that this id is currently within the ISO reserved range, hence it's
utilisation is unofficial.

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

glmemory: provide compatibility definition for GLES2 for GL_RGBA8

gl: consolidate internal_rgba_format into glmemory

Expose some useful value format conversion functions available in
GstGLMemory.

audiointerleave: Always have "channels" be the actual pad count

Don't force it anywhere

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

audiointerleave: Use the channel count from the set caps

This is the same number that was used to allocate the buffer

aggregator: Document that the latency is in ns

amc: Only lower ranks of OMX.Exynos. audio codecs, the video codecs are actually working

amc: Give marginal rank to codecs that start with OMX.Exynos.

OMX.Exynos. codecs are existing on some devices like the
Galaxy S5 mini, and cause random crashes (of the device,
not the app!) and generally misbehave. That specific device
has other codecs that work with a different name, but let's
just give them marginal rank in case there are devices that
have no other codecs and these are actually the only working
ones

androidmedia: Give lower ranks to codecs not starting with OMX.

On some devices there are codecs that don't start with OMX., while
there are also some that do. And on some of these devices the ones
that don't start with OMX. just crash during initialization while
the others work. To make things even more complicated other devices
have codecs with the same name that work and no alternatives.
So just give a lower rank to these non-OMX codecs and hope that
there's an alternative with a higher rank.

Also stagefright gives codecs starting with OMX. a higher rank too and
considers other codecs that don't start with OMX. as software codecs.

tests: zbar: check for frame field on attach-frame=true

Also check for the sample to have a buffer and caps
associated with it.

Related to:

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

vtdec: Require width and height field for H264

This decoder does not work if width and height field are not set
in the sinkpad caps. Let's make this explicit by adding them to
the template caps.

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

glcontext_gpu_process: close the window when done

glwindow_gpu_process: avoid empty struct

glwindow: move main loop/context creation back to init/finalize

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

examples: Fix gl usage without wayland support

Not all platforms have wayland support. Handle that gracefully at
compile time

rtmpsink: Initialize GstMapInfo

Avoids doing a call to unmap with it uninitialized

CID #1302834

audiovisualizer: clean dereferences of private structure

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

vtenc: fix keyframe request race condition

It is incorrect to modify the frame properties after passing them, since
VTCompressionSessionEncodeFrame takes reference and we have no control
over when it's being used.

In fact, the code can be simplified. We just preallocate the frame
properties for keyframe requests, and pass NULL otherwise.

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

audiovisualizer: make private all variable subclasses don't need

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

zbar: add frame sample to barcode message

New attach-frame property enables barcode frame
dumping when set to true.

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

gl: Fix compiler warning

gstglwindow_wayland_egl.c:246:41: error: unused variable 'surface_listener' [-Werror,-Wunused-const-variable]
static const struct wl_surface_listener surface_listener = {
                                        ^

androidmedia: Conditionally use get_{input,output}_buffer() Android 21 APIs

Also properly set limit/position on byte buffer, some codecs prefer to have
correct values there.

glwindow: fix compiler error

gstglwindow.c:1118:544: error: 'return' with no value, in function returning non-void [-Werror]
   g_return_if_fail (GST_GL_IS_WINDOW (window));

gl/examples: update gtk examples for glupload

gl/examples: update gtk examples for wayland

gl/wayland: implement setting the render rectangle

Places our subsurface at the rectangle provided position

glwindow: handle gst_video_overlay_set_render_rectangle

gl/wayland: don't block the event loop after poll

Use the dispatch_pending set of functions which just run the currently
queued up events instead of potentially waiting for an event to occur.

tests/gl/gtk: implement setting a wayland display/surface

gl/wayland: implement basic video overlay support via subsurfaces

Currently does not position the subsurface relative to the parent surface at all

gl/wayland: allow a NULL wl_event_queue

perform operations on the default wl_display event queue in that case

gl/utils: implement wayland display GstContext

requried for sharing GL contexts or subsurface support

gl/wayland: run each window on a separate queue

Based on patch by Julien Isorce <julien.isorce@collabora.co.uk>

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

gl: Don't leak temp strings in _RGB_pixel_order()

Fix a memory leak of temporary strings when computing
swizzling of RGB formats.

glfilter: Add transform_internal_caps() vfunc

Add a vfunc that is called by glfilter before it sets
caps features and intersects with the peer caps, and
move removing the size from caps into its default
implementation. Allows sub-classes to do more
sophisticated management of the size fields in case they
don't support arbitrary resizing or have distinct
preferences.

glfilter: Don't remove size fields from caps

Instead of removing size fields, set them to the full range
like videoscale does, so the caps are clearly unfixed in
all cases.

videoaggregator: Add class property to disable caps scaling

Add preserve_update_caps_result boolean on the class to allow
sub-classes to disable videoaggregator removing sizes and framerate
from the update_caps() return result.

videoaggregator: Catch errors, and allow sub-class to return NULL from get_output_buffer()

A return value of GST_FLOW_OK with a NULL buffer from get_output_buffer()
means the sub-class doesn't want to produce an output buffer, so
skip it.

If gst_videoaggregator_do_aggregate() generates an error, make sure
to propagate it - don't just ignore and discard the error by
over-writing it with the gst_pad_push() result.

rtmpsink: Do not crash when receiving buffers after GST_FLOW_ERROR

If the RTMP URI is invalid, the rtmpsink will return GST_FLOW_ERROR.
If it still receives buffers after that, it shouldn't crash.

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

dashdemux: avoid assertion when splitting buffer

Pass size=-1 as is if that is the case instead of subtracting
the offset. Otherwise we have an invalid size passed for that
buffer.

dtlssrtpdec: Release requested pads

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

dtlsdec: Remove unnecessary ref/unref operations

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

dtlsdec: Fix release request pad function

Don't unref the pad reference we don't own and just remove the pad.

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

hlsdemux: Search more when advancing fragment

In live situations, it is not uncommon for the current fragment to end
up out of the (updated) play range (lowest/highest sequence). But the next
fragment to play *is* present in the play range.

When advancing, if we can't find the current GstM3U8MediaFile, don't abort
straight away. Instead, look if a GstM3U8MediaFile with the next sequence value
is present, and if so switch to it.

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

gl: add GstGLContextGPUProcess backend

It builds its GL vtable from a proc address provided
by the application.

gl: add and use gst_gl_internal_format_rgba

Previously when compiling GstGL with both GL and GLES2,
GL_RGBA8 was picked from GL/gl.h. But a clash may happen at
runtime when one is selecting GLES2.

gst_gl_internal_format_rgba allows to check at runtime
if it should use GL_RGBA or GL_RGBA8.

glwindow_x11: use parent default implementation

glcontext_glx: rename variable from window to context

glwindow: initialize debug in class_init

Useful if one uses gst_gl_window_x11_new directly
instead of gst_gl_window_new.

glcontext: initialize debug in class_init

Useful if one uses gst_gl_context_egl_new directly
instead of gst_gl_context_new.

glcontext: add gst_gl_context_set_display helper

glcontext: add missing _CAST after G_TYPE_CHECK_CLASS

glwindow: add missing _CAST after G_TYPE_CHECK_CLASS

glwindow: provides some default implementations to factorize with all backends

aggregator: Push EOS on error return.

Before shutting down the srcpad task due to a
downstream error, push an EOS to give downstream
a chance to shut down somewhat cleanly.

mpegtsmux: don't overwrite value

Value of res is reset to FALSE in each iteration of the while loop. We want to
conserve TRUE if any pad event succeeded until we arrive to done.

Also, buf is set to the value of *outbuf twice. Removing the first assignment
since the second one is outside of a conditional.

opencv: 'for' loop initial declaration

'for' loop initial declarations are not allowed in C89, moving the declarations
to before the 'for' loops.

gleffects: 'for' loop initial declaration

'for' loop initial declarations are not allowed in C89, moving the declarations
to before the 'for' loops.

resindvd: remove unused value

The data variable is incremented but never read again. Remove the unused value
assignment.

hlsdemux: remove more unused attributes

Those are not used or only read