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>

tests: add standalone program for VP8 parser.

Add standalone test application that demonstrates how to use the new
VP8 bitstream parsing library, while also allowing simple debugging/
tracing of IVF files.

[clean-ups, updated to new parser API]
Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

tests: add test for VP8 bitstream parsing library.

[updated to new parser API]
Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

codecparsers: vp8: rename dboolhuff symbols.

Rename VP8 dboolhuff symbols so that to avoid clashes with libvpx when
static linking.

codecparsers: vp8: add GStreamer native utilities.

Import libvpx 1.3.0 range decoder files (dboolhuff.[ch]) to implement
the VP8 utilities native interface. Likewise, copy and use the default
libvpx generated entropy probabilities tables.

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

codecparsers: add VP8 bitstream parser.

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

[refactored, among other fixes]
Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

mpegtspacketizer: Improve ts_to_offset code

* Search in current pending values first. For CBR streams we can very
  easily end up having just one initial observations and then nothing
  else (since the bitrate doesn't change).
* Use one group whether we are in that group *OR* if there is only
  one group.
* If the group to use isn't closed (points are being accumulated in the
  PCROffsetCurrent), use the latest data available for calculation
* If in the unlikelyness that all of this *still* didn't produce more
  than one data point, just return the initial offset

tsdemux: Fix scaling macros

While the calculation done in these macros will work with 64bit
integers, they will fail if working with 32bit integers.

Force the scaling up to solve that.

This amazingly didn't introduce major issues up to now, but resulted
in bogus values in debug logs.

mpegtsbase: Don't hard-flush the packetizer on discont buffers

Doing a hard flush on the packetizer will drop all observations, which
will eventually break push-based seeking (with BYTES segment) since
we won't know where to seek to anymore (new data would always be
considered as the beginning of the stream).

mxfdemux: Fix copy/paste error

We want to check whether the rate is different. We check changes in
numerator *and* denominator.

CID #1139631
CID #1139642

mxf: Handle fraction parsing failure

And properly cleanup/reset the segment before returning on errors

CID #206012

mxf: Remove useless check

a guint will always smaller or equal to the maximum value it can
contain

CID #206049

mxf: avoid dereferencing NULL mapping data pointer

Also unref buffers on error, as it seems to be done in one, but
not all, error paths.

The NULL pointer part is Coverity 206112

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

dvb: Fix if() check

introduced by my previous commit :(

configure: Detect EGL header presence

Fixes the build for Android, and I suspect for other systems too...

dvb/camsession: Don't do operations on unexisting sessions

CID #206114

dvb: Handle socket() failure

CID #206178

dvbsrc: Handle failure to get statistics from the device

Check the ioctl return value and don't output bogus values.

CID #206016

dvbsrc: Don't close a fd that wasn't open

We are handling the failure cases in this block where the open failed

CID #206179

dvbsrc: Don't try to access NULL string

And refactor the pid setting code to be in its own function for clarity

CID #206374

mpegts: add dvb ca identifier descriptor

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

mpegts: add scrambling descriptor

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

mpegts: add frequency list descriptor

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

mpegts: add data broadcast id descriptor

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

mpegts: add private data specifier descriptor

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

dvb: Minor code style fix

mpegts: add parential rating descriptor

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

mpegts: add linkage descriptor

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

mpegts: for loop initial declarations are only allowed in C99 mode

Fixes compiler warnings.

egl: Add libgstegl and eglgles plugin to the CRUFT directories

egl/eglglessink: remove since EGLImage and iOS support have been added in glimagesink

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

gl: allow to include GLES/gl.h

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

wayland: Remove dead assignment

We set it just after variable declaration

CID #1197709