glcolorscale: use a shader on GLESv2

Otherwise you only see black frames

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

glimagesink: use new helper function to avoid duplicating default shader text

gleffects: use new helper functions to avoid duplicating the same vertex shader text

And also use the default fragment shader text for the identity effect

gl: add convenient functions to setup default vertex and fragment shaders

Most of our 2D filters use the same simple vertex shader.
Also define the default fragment shader as the identity.

Avoid duplicating the same vertex and fragment shader text.

hlsdemux: Only unref pending buffer if there is one

hlsdemux: Do not push last buffer after error

In decryption scenario, a buffer is always stored to be sent later
to wait for more data or EOS to be able to strip the final bytes
if requested. In case an error hapenned this buffer can be ignored
and not pushed downstream.

hlsdemux: flush the adapter in the end of a fragment

In case of error there might be some data left in the
adapter when EOS is received. Clear the adapter to be
able to restart again later if requested.

hlsdemux: handle more error cases

Handle some more error cases:

1) When the source element fails to go to ready
2) When decryption fails
3) When there is no source to handle a specific URI
4) When the URI is invalid

hlsdemux: handle errors from internal source

Set up a message handling function to catch errors from the internal
source and store the last return code to identify error situations
when returning from a fragment download.

Also moves the duration increase to after the download when we
know if it was successful or not

hlsdemux: Improve pad switching conditions

When using the internal source, hlsdemux doesn't know the caps of
the input before adding the pad, so remove the arguments that would
use that as it is always NULL.

And use an specific flag to signal when a pad switch is required.
Using the discont flag is a bad idea now because when a fragment
download fails it will lead to exposing a pad group without any
data, causing decodebin to abort.

hlsdemux: properly flush decryption status on seeks

Avoids mixing decryption of different fragments when seeking happens
and leading to broken stream output.

hlsdemux: Track fragments duration

When receving EOS from the internal src, increase the current positon
by the fragment duration to allow correct restoring of download position
if the bitrate changes

hlsdemux: properly stop tasks by stopping fragment download

Issue a signal to the fragment download cond to stop
the task earlier

hlsdemux: decrypt before typefinding

Make sure typefinding gets the decrypted content instead of
trying to typefind the encrypted data.

hlsdemux: fix decryption function return

Correctly return the decrypted buffer when it succeeds and
return NULL otherwise

hlsdemux: keep connection alive between downloads

Use the same properties as uridownloader to keep connections alive
between consecutive fragments downloads.

1) set keep-alive property to true
2) keep the element in READY instead of in NULL

hlsdemux: create sources from uri

Instead of using always an http source, create it dynamically from
uri when needed. If not needed just replace the URI on the current
element

hlsdemux: do not try to run typefind again if caps is the same

Always reset the do_typefind flag if hls did typefind because
trying it on non-zero offsets doesn't make sense and will cause
assertions

hlsdemux: enable download bitrate measure

Measure the download bitrate to be able to select
the best playlist.

As the buffers are directly pushed downstream and it
might block. The time is only measured from the download
until the pad push and it is started again after the push
returns.

hlsdemux: re-enable decryption after uridownloader removal

Now the decryption is done buffer by buffer instead of on the
whole fragment at once. As it expects multiples of 16 bytes a
GstAdapter was added to properly chunk the buffers.

Also the last buffer must be resized depending on the value of the
last byte of the fragment, so hlsdemux always keeps a pending buffer
as it doesn't know if it is the last one yet

hlsdemux: replace uridownloader with a GstElement

The GstElement is directly linked into a ghost pad and
its buffers are pushed as received downstream. This way the
buffers are small enough and not a whole fragment that usually
causes extra latency and makes buffering harder

gl: add missing G_BEGIN/END_DECLS in utils header

examples: gl: add missing entry to cflags to find gst/gl/gl.h

Otherwise build fails

tests: drop direct include

It is only allowed to include gdk/gdkx.h and that will bring all the x11 things.

sndfile: send a toc for the segments in the audio file

h264parse: unref/replace force_key_unit_event in gst_h264_parse_reset

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

gl/sdl example: Fix uninstalled build

gl: Reduce noisy Info level output to LOG

gl/examples/sdl: update for the latest gstgl changes

gl/examples: move to -bad

- fix all the compiler errors
- give them their own gl directory

egl: remove pc files

Forgot to do that in 09116bf10d515d50bdd1e9a20278be816cb762aa:
"egl/eglglessink: remove since EGLImage and iOS support have been added in glimagesink"

gl: append GL_LIB and GL_CFLAGS in pc files

gl: append EGL_LIB to GL_LIB

gl: use gst_gl_platform_from_string because we support gl shareList on all platforms

glfilter: only warn when other_context attribute is set

Fix false positive

gl: fix string literal warning

warning: format not a string literal, argument types not checked [-Wformat-nonliteral]

uridownloader: Unset referer if none was specified later on

hlsdemux: Set Referer in requests to the playlist URI

uridownloader: Add API to specify the referer of a request

codecparsers: h264: fix parsing of slice groups for map type = 2.

When parsing slice groups information for slice_group_map_type = 2, we
should only be reading up to num_slice_groups_minus1 groups since there
is always a "leftover" slice group and as many "foreground" slice groups
as needed.

This fixes parsing for SVCBMT-5 and SVCBMT-12 whereby the base layer would
have incorrectly been parsed to have up to 38 reference frames in list0,
which is not possible.

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

decklink: Don't fail if HW does not have all interfaces

This patch completes 1c1cc73a3b230454663971656515297f9ee8bd9f. These
changes where accidentally pushed as a separate patch with wrong commit
message.

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

Revert "y4mdec: check for VIDEO meta instead of CROP meta"

This reverts commit 68309bd21592a96dc8241702f17ded4e5f545f33.

Commit message and files changed do not match.

docs: remove outdated and pointless 'Last reviewed' lines from docs

They are very confusing for people, and more often than not
also just not very accurate. Seeing 'last reviewed: 2005' in
your docs is not very confidence-inspiring. Let's just remove
those comments.

codecparsers: h264: fix derivation of default scaling lists.

When useDefaultScalingMatrixFlag is computed to be 1 while parsing
scaling_list(), then the scaling list shall be inferred to be equal
to the default list (7.4.2.1.1.1). That default list is really one
of Default_4x4_{Intra,Inter} or Default_8x8_{Intra,Inter} and not
one from fall-back rule sets A or B.

This fixes parsing for FRExt1_Panasonic_D, FRExt2_Panasonic_C,
FRExt3_Panasonic_E and FRExt4_Panasonic_B.

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

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

curl*sink: report errors from curl when setting options

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

curl*sink: post error on bus in element, not transfer thread

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

curlhttpsink: allow for unconditionally disabling proxying

Previously if the proxy server hostname was the empty string
curlhttpsink would never even set the libcurl option. For libcurl
however, having a proxy server hostname be the empty string means that
proxying should be disabled even if environment variables might be set.
Now with the restriction lifted, doing this is allowed.

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

curl*sink: improve return value checks in test cases

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

curl*sink: fix typos

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

curl*sink: fix some gst-indent problems

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

gl: check the correct GstGLMemory in basic_test

mpegtsmux: Only require alignment and stream-format for h264, parsed is implicit

uridownloader: Set source state to NULL if downloading failed for whatever reason

gl/x11: Removed unused variable

gl/x11: Get rid of unneeded second display connection and do everything from the main loop

gl/x11: Block rendering until swapbuffers is called and don't go via X11 events as unneeded indirection

This should fix QoS problems, where basesink believed it was rendering with
20FPS but actually we were just queueing up X11 Expose events and only once
in a while something was rendered.

gl: fails glimagesink when shader compilation went wrong

Those shaders are fixed and very simple so it should not fail
but it's worth to handle a such case.

gl: call glClearColor before glClear

pkgconfig: fix gstreamer-gl uninstalled

Ensure to provide libgstgl also in the uninstalled setup.
Fixes build of gst-omx for RPI in uninstalled setup.

gl/rpi: do not scale to full screen

Having a 640x360 movie being scaled to 1900x1200 does not look nice

gl/rpi: change dispmanx attributes of existing element instead of creating a new one

I was lucky that:
e1 = vc_dispmanx_element_add ()
eglCreateWindowSurface (e1)
vc_dispmanx_element_remove (e1)
e2 = vc_dispmanx_element_add ()

resulted in having e2 equal to e1. And also having the egl surface
that does not allocate its buffers before the first draw.

gl/rpi: ensure the dispmanx element to be opaque

So that we ensure it does not get alpha composited with other
dispmanx elements like the desktop.

avfassetsrc: downgrade rank to SECONDARY in OS X

avfassetsrc: rename references to iOS

iosavassetsrc: rename element to avfassetsrc

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

iosavassetsrc: Port to 1.0

Also enables this element on OSX >= 10.7.

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

applemedia: add new source/decoder element for iOS assets

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

coremediabuffer: handle stride alignment

Handle stride alignment through the use of the video meta API. The
code is based on the corevideobuffer implementation.

If the video meta API is not supported and the underlying buffer
contains padding, the core media buffer is copied to a system memory
buffer.

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

avfvideosrc: check if downstream supports the video meta API

Check if downstream supports the video meta API, so we can use it later
with the core media buffers.

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

gl/cocoa: pass a NSView to set_window_handle instead of a NSWindow

Fix backwards compatibility

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

gl/cocoa: only draw once the window has been resized

It avoids to draw the first frame with an incorrect view port.

gl/cocoa: set the view to use for drawing by the context

It avoids to draw to an invalid buffer.
Withtout this the default frame buffer is undefined:
glBindFramebuffer (GL_FRAMEBUFFER, 0)

Visually you could see some white frames at the beginning
when lunching videotestsrc ! glimagesink

With OpenGL Profiler from XCode you could see some
GL_INVALID_FRAMEBUFFER_OPERATION for the first frames

gl: fix declaration-definition mismatch for _init_upload

gl/bufferpool: do not reset upload when set_config does not change the caps

With videotestsrc ! glimagesink it was reset 3 times

gl: use #ifdef instead of #if for __APPLE__

warning: "__APPLE__" is not defined [-Wundef]

with gcc version 4.7.2 20120731 (prerelease)
    (crosstool-NG linaro-1.13.1+bzr2458 - Linaro GCC 2012.08)

glmemory: User g_slice_new0 to ensure fully initilized structure

The pbo pointer not being initialized would trigger a use of unitilialized variable
in valgrind.

decklink: Fix support for HW without output

Devices suitable for decklinksrc may not have any output, hence querying
the input returns NULL. Add support for all cases where
input/output/config may be missing.

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

y4mdec: check for VIDEO meta instead of CROP meta

This element only need stride support, but checks for CROP_META, which
is a bit too much, even though this works as CROP imply having VIDEO
meta.

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

glimagesink: Use the pixel-aspect-ratio adjusted width/height when centering the frame in the display area

Makes sure we actually keep the display aspect ratio

glimagesink: By default keep the display aspect ratio

Also the default for the pixel-aspect-ratio should be 1/1, not 0/1.

mpegpsdemux: Flush program end code bytes

This should not harm regular files, since those are the last 4 bytes of
a normal file.

This allows to handle playback of concatenated mpeg-ps files. Seeking and
duration reporting is still wrong though.

gltestsrc: Unref context when creation failed and guard against that in fill()

glimagesink: Call gst_object_unref() on the GstGL GObjects again

While they're plain GObjects it does not hurt to call gst_object_unref()
on them and potentially allows to debug reference leaks a bit easier.

glimagesink: If creating a context or upload failed, destroy it

Fixes deadlocks in error cases when later code assumes the broken
context is actually usable.

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

mpegtspacketizer: Remove dead code and simplify if statement

CID 1204274

mxf: fix MP2 case

Testing mpegversion when mpegaudioversion was likely meant.

Similar tests in sys/androidmedia/gstamcaudiodec.c also test
mpegaudioversion with the same conditional code.

Coverity 206071

vc1parse: guard against dividing by zero

If framerate is unknown, we write the maximum framerate
allowed for this profile/level:

https://tools.ietf.org/html/draft-ietf-avt-rtp-vc1-06#section-6.1
http://wiki.multimedia.cx/index.php?title=VC-1#Setup_Data_.2F_Sequence_Layer

Coverity 1139694

mxfmux: guard against dividing by 0

Use a placeholder value in that case, it's better than crashing.

Coverity 1139697

gl: a couple spelling/grammar fixes

gl: guard against using a NULL window pointer

Coverity 1195145

gl: prevent division by 0 on unsupported texture type

Coverity 1199697

glcolorconvert: Fix bt709 conversion matrices.

Fix the sign on one entry in the bt.709 YUV->RGB conversion
matrix, and the corresponding inverse matrix. Fixes really
wrong colouring of some videos.

dvbbasebin: fix test for proper use count balancing

usecount is unsigned, so too many "unuse" will wrap the counter
around and the >= 0 check will always be fine.

It would be much simpler to just make the counter signed, but
moving the checks where the decrements happen allow a mistake
to be detected earlier, and thus easier to debug.

Coverity 1139791

mxfdemux: guard against NULL non source components

This component is dereferenced, and later code checking for
NULL in particular cases implies it can be NULL. This likely
does not fix the coverity warning as it was seeing another
path setting component to NULL explicitely, but this was
spotted by looking at:

Coverity 1139736

Which is actually OK from what I can see since the actual
dereference of the explicit NULL pointer will not happen
if the condition that led to the NULL pointer assignment
is met, since the assignment and defeference have mutually
exclusive tests.

rawparse: catch errors in caps manipulation

Coverity 1139622

vtenc: fix complier error using fixed lenght arrays

ivfparse: detect and propagate resolution changes.

Detect resolution changes on key frames, and propagate the resulting
caps to the src pad. Only the uncompressed data chunk is decoded, so
avoid using the new VP8 bitstream parsing library for now.

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

ivfparse: avoid possible division-by-zero when calculating PTS.

Avoid possible division-by-zero while deriving the presentation timestamp
of the buffer. The base class will take care of any interpolation needs.

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

ivfparse: enable build.

Drop `ivfparse' element from the non-ported set of plugins in configure.

ivfparse: port to baseparse.

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

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>