eglglessink: unref last buffer on DRAIN

Similar to 093574053fae243ec5fcfd6adae8185053b1e896
in gstbasesink

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

eglglessink: fix wrong doc block for create_window

Wrong since 1ad253b9 when prop name was changed from
can_create_window to create_window.

gl: don't fail configure if we do not understand the platform

gl: fix out-of-source builds pt2

21b897de2f762bee504cbc007fd108148ee5ae24 did this for x11 and wayland.
Do it for the other backends as well.

gl: only set CAPS_FEATURE_MEMORY_EGL_IMAGE on egl platform

configure.ac: fix OSX host check

Makes the check works with (i386|x86_64)-apple-darwin* hosts.

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

gl: let the user includes itself our egl headers if needed

Forgot to address this change.

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

gl: add EGLImage support

* picked from old libgstegl:
  - GstEGLImageMemory
  - GstEGLImageAllocator
  - last_buffer management from removed GstEGLImageBufferPool

* add-ons:
  - GstEGLImageMemory now old a reference on GstGLContext
    so that it can delete the EGLImage and its gltexture source
    while having the associated gl context being current.
  - add EGLImage support for GstVideoGLTextureUploadMeta which
    mainly call EGLImageTargetTexture2D
  - GstGLBufferPool now supports GstEGLImageAllocator
  - glimagesink / glfilters / etc.. now propose GstEGLImageAllocator
    to upstream

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

gl: deploy egl headers in gst/gl/egl instead of gst/gl

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

gl: remove commented and unsued code in x11 Makefile.am

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

pkgconfig: add gstreamer-gl

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

gl: fix crash if _build_extension_string is not called

On GLES2 then (gl->GetIntegerv && gl->GetStringi) is false
regression introduced by cc6df204e2f58fffda5cbe90f3450aeba95889c4

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

gl: Only enabled cocoa backend on osx (and not ios)

ios requires using EAGL

rtmpsink: Remove URL check for valid playpath.

The playpath is an optional component of the URL - don't require it.

srtpenc: unlock object lock before posting error

Posting an error with the object lock will deadlock. Rework the code
a bit to make things more simple.

codecparsers: cosmetics: drop TAB characters.

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

dvbsuboverlay: Clarify confusing debug message

The message is describing the number of bytes remaining for
parsing, not 'missing'.

tsdemux: Fix mpegts_packetizer_set_current_pcr_offset()

When adjusting the PCR offset of groups, adjust the correct group
entry from the list - not the current group repeatedly.

dvbsuboverlay: Remove warning for 2-bit and 8-bit subpictures

Both 2-bit and 8-bit mode now seem fine, so remove the warning
messages about them

dvbsuboverlay: Fix 8-bit subpicture rendering.

Fix 2 small flaws handling 8-bit subpictures that makes
my one test file work.

codecparsers: h264: fix slice_header() parsing for MVC.

The idr_pic_id syntax element depends on IdrPicFlag, which is a calculated
value that does not only depend on NAL unit type (IDR), but possibly also
on MVC non_idr_flag syntax element.

The computed idr_pic_flag is already stored in GstH264NalUnit structure.

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

Signed-off-by: Li Xiaowei <xiaowei.a.li@intel.com>
Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

codecparsers: h264: add gst_h264_parse_nalu_header() helper.

Add helper to parse the NALU header. Move bounds checking to there.

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

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

codecparsers: h264: add gst_h264_parse_sps_data() helper.

Split seq_parameter_set_data() parsing off gst_h264_parse_sps() so
that it could be re-used later on.

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

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

codecparsers: h264: complete set of NAL unit types.

Add missing NAL unit types. They are mostly related to alpha blending,
scalable video coding extensions (SVC, Annex.G), and multiview video
coding extensions (MVC, Annex.H).

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

codecparsers: h264: clean-up gst_h264_parser_parse_sei_message().

Fix build when GST_DISABLE_GST_DEBUG is not defined. Use a switch
statement to dispatch to the various SEI payload handlers.

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

codecparsers: h264: fix skipping of unsupported SEI messages.

The payloadSize does not account for emulation prevention bytes. So,
just use nal_reader_skip() for skipping payload_size bits. It should
be possible to further optimize this code since the NAL reader shall
be aligned to byte boundary already.

Kill the now unused nal_reader_skip_to_next_byte() function.

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

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

codecparsers: h264: fix SEI buffering_period() parsing.

Fix parsing of buffering_period() SEI messages. The number of bits
used to express {nal,vcl}_initial_cpb_removal_delay{,_offset} syntax
elements is not 5 but 1 + initial_cpb_removal_delay_length_minus1.

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

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

codecparsers: h264: fix rbsp_more_data() implementation.

Account for trailing zero bits when checking for rbsp_more_data().

In particular, fix an hypothetical stream whereby rbsp_more_data()
is called in the following conditions for PPS header: NalReader
reached position 20, 12 bits are remaining and trailing data at
current byte position is c8 00.

rbsp_more_data() used to return TRUE whereas it should obviously
return FALSE because x8 00 represents a valid rbsp_trailing_bits()
structure.

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

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

gl: fix out-of-source builds.

Always use the locally generated <gst/gl/gstglconfig.h> file.

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

gl/download: add support for the bt709 color matrix

gl/upload: add support for the bt709 color matrix

gl: remove explicit reference to X_CFLAGS

They are in GL_CFLAGS if required

gl/upload: remove the legacy GL 1.x upload path

uploading requires shaders and fbos available in GL2.

gl: fallback to glGetString if GL_NUM_EXTENSIONS fails

Need it on MacOSX 10.7.5

gl: and update the Makefile

gl: reorganize the extension headers by function rather than api

gl/win32: use the correct include and gst-indent

check: fix make check

The mpeg parser now flags the stream as mpeg2 if there is a picture
extension header, so remove it for mpeg1 data.

gl: use wglCreateContextAttribsARB to create share context

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

gl/mixer: fix download check for failure

gl: silence a critical if upstream does not provide us with meta params

gl: silence warnings building for RPI related to 'vcos_*'

Similar than 1190a79b199584cfc4dd62c474531c32cfbba425

gl: keep only one occurence of '#include <EGL/egl.h>'

To simply maintainance if we need to put specific includes around it.

dvbsuboverlay: Don't forward text gap events downstream

That gap is only for the subtitle stream and not for the video stream.

gl: don't default to GST_GL_HAVE_WINDOW_DISPMANX=1

Mistake introduced by 2a0f0399ae226089c2ba07b1b904741b856f37af

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

gl: fix the use of always-defined macros

After 2a0f0399ae226089c2ba07b1b904741b856f37af GST_GL_* macros are always
defined to 0 or 1. Don't use #ifdef ... or #if defined() on them.

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

gl: check for X unconditionally

Without this the HAVE_X macro is empty, not "no" or "yes", which is not
what the later code is counting on.

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

gl: set default value to 0 for all definitions that go to gstglconfig.h

To avoid warnings like "GST_GL_HAVE_OPENGL is not defined".
As we are using #if instead of #ifdef

gl: allow not building the gl library plugins

gl: add GModule libs for win32

gl: only link against gstegl if we are using egl

Assuming if we are building egl that the egl library is built

gl: add the option of disabling cocoa on OS X using --disable-cocoa

gl: add path of the gstegl library

Temporary until we merge the two libraries

Revert "gl: add dep on gstegl"

It was already in _LIBADD

This reverts commit b10a3530804335fc3b9494603ef78ce9417c3bc8.

gl: add dep on gstegl

Temporary until we merge the two libraries

gl/cocoa: Fix debug statements and platform

gl: set HAVE_EGL_RPI to yes if HAVE_GL and RPI

Which fixes build on Raspberry Pi since -gl has been moved to -bad

gl: Fix static build with objective-C

--tag=CC is needed for static build

make: whitespace cleanups

gl: Add EGLDisplay display subclass

gl/window: add a dummy window class

Effective for the case where we have a platform that does not
require a native window.  We require a mainloop to run the GL
commands which is currently operated by GstGLWindow.

gl: fix undeclared symbol when building without GST_DEBUG

gl: fix a double unref of the query when building without X

dvbsuboverlay: handle video/x-raw(ANY) if downstream supports the GstVideoOverlayCompositionMeta API

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

gl: silence a compiler warning about missing prototypes

gl/utils: add x11 GstContext handling

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

gl: define GL_NUM_EXTENSIONS if not defined

Move gst-plugins-gl to bad

gl: silence all the compiler warnings

move gl elements to ext subdirectory

[906/906] filter: warn about trying to share with multiple contexts at once

[905/906] Add GL context sharing support for non-gstgl elements

[904/906] mixer: ask for display handles on pad activation

[903/906] api: add GstGLPlatform to/from string

[902/906] context: implement glGetStringi handling for GL core contexts/GLES3

[901/906] glx: use the display handle from the global display

Intel drivers require the display handles be the same for context
sharing to occur.  Also solves some cases of use after free of the
display when integrating with gstreamer-vaapi.

See https://bugs.freedesktop.org/show_bug.cgi?id=41736 for the intel bug.

[900/906] tests/gstglcontext: reduce the number of frames displayed

Was causing timeouts on intel hardware due to vsync handling.

[899/906] egl: warn if we resort to display handles from the window

[898/906] glimagesink: add an other-context property

Allows applications to provide us with an OpenGL context with which
to share with.

[897/906] api: plugin a small memory leak

Found by adrien.schwartzentruber@gmail.com

[896/906] utils: use the vtable for deleting the texture

[895/906] context: call window_class->close last

We should destroy resources before closing the display connection

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

[894/906] gl: add missing G_BEGIN_DECLS/G_END_DECLS

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

[893/906] x11: close both of the display's we use

(the comment doesn't seem to apply anymore)

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

[892/906] silence a compiler warning on older gcc versions (4.6)

[891/906] context: add support for wrapping external contexts

[890/906] x11: add display subclass

GstGLDisplayX11 holds the display connection and name.  Each thread requires
it's own X11 Display connection (initialised from name) due to the fact that
we do not want to call XInitThreads().  Doing so would result in segfaults
when integrating with GUI toolkits Gtk, Qt, etc.

The Display connection is for OpenGL platforms where a constant display is
required in order to share contexts (egl).  In the case of a wrapped context
(added later), we do not have GstGLWindow to retreive the display from so a
'master' connection is used instead.

[889/906] display: add display type enum

[888/906] correct error handling in gstgldownload.c

[887/906] mixer: fixup a memory leak of the context in the GLTextureUploadMeta path

gst_structure_get returns a reference to the object and we asked
for another with gst_object_replace.

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

[886/906] fixup a memory leak of the context in the GLTextureUploadMeta path

gst_structure_get returns a reference to the object and we asked
for another with gst_object_replace.

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

[885/906] filter: free our data

plugs some memory leaks

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

[884/906] filter: return the pad template caps in transform_caps

We can transform from any input in our caps to any output.
With the following pipeline snippet:

  ... ! vaapidecode ! glcolorscale ! xvimagesink

GstVideoGLTextureUploadMeta was being used on both src and sink
pads causing linking to fail.  This allows the usage of the meta
on either pad without affecting whether the meta is chosen on the
other pad.

[883/906] api: provide from_string()

Also s/gst_gl_api_string/gst_gl_api_to_string/g

[882/906] port glcolorscale to GLES2

[881/906] upload: restore default values for unpack_length

fixes elements that upload a supplementary texture (glbumber,
gldifferencematte, etc)

[880/906] glimagesink: remove unused stored_buffer field

It has the potential to cause deadlocks.

[879/906] build: don't add X_CFLAGS to CFLAGS

It should already be included if needed inside GL_CFLAGS

[878/906] tests: update for glmem api change

[877/906] x11: Use correct format string for guintptr

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