waylandsink/wlwindow: reuse code between the two constructors

waylandsink: increase debug messages

waylandsink: Use a boolean in combination with render_cond to comply with GCond's usage documentation

waylandsink: Implement expose() and handle resizing properly in non-PLAYING states

waylandsink: Use wl_scaler/wl_viewport to scale the surface in the compositor/hardware

waylandsink: Build bindings for the unstable wl_scaler spec

waylandsink: Set external surfaces and their child objects to use our own event queue

This fixes weird freezes because of frame_redraw_callback() not being
called from the main thread when it should with weston's toy toolkit.

It's also safer to know that frame_redraw_callback() will always be
called from our display thread... Otherwise it could be called after
the sink has been destroyed for example.

waylandsink: Wait for the frame_cb to redraw and drop frames meanwhile

We are not supposed to redraw until we receive a frame callback and this
is especially useful to avoid allocating too many buffers while the
window is not visible, because the compositor may not call wl_buffer.release
until the window becomes visible (ok, this is a wayland bug, but...).

waylandsink: Handle wl_buffer::release and don't reuse buffers that are not released

This is achieved by adding an extra reference on the buffers, which does
not allow them to return to the pool. When they are released, this reference
is dropped.

The rest complexity of this patch (hash table, mutex, flag, explicit release calls)
merely exists to allow a safe, guaranteed and deadlock-free destruction sequence.
See the added comment on gstwaylandsink.c for details.

waylandsink/waylandpool: remove useless munmap call

This data pointer is part of a bigger mmap'ed region,
it has not been returned from mmap itself.

waylandsink/waylandpool: call the start/stop methods of the parent class

start() makes sure that the minimum ammount of buffers requested is allocated.

stop() makes sure that buffers are actually destroyed and prevents
filling the file system when resizing the surface a lot, because the
wayland-shm-* files will stay on the file system as long as the wl_buffers
created out of them are alive.

waylandsink/waylandpool: unlink mmaped shm files so that they don't remain on the file system

waylandsink: implement the GstVideoOverlay & GstWaylandVideo interfaces

This is the initial implementation, without the GstVideoOverlay.expose()
method. It only implements using an external (sub)surface and resizing
it with GstWaylandVideo.

waylandsink: implement with stubs the GstWaylandVideo & GstVideoOverlay interfaces

wayland: Add new gst-wayland library containing a new GstWaylandVideo interface

This interface is needed to be able to embed waylandsink into
other wayland surfaces. Due to the special nature of wayland,
GstVideoOverlay is not enough for this job.

waylandsink: handle the list of supported formats properly

enum wl_shm_format is not a flags enum, as it may have been in the past,
so multiple formats cannot be stored in a bitfield. Use an array instead.

waylandsink/wlvideoformat: add mappings for many common formats

waylandsink: unref the buffer pool

waylandsink/waylandpool: ref the display instead of the sink to avoid cyclic references

The reference to the sink is not really needed anyway in waylandpool,
what matters basically is that the display is active as long as the
pool is active, so we really want to reference the display object
instead of the sink.

waylandsink: make the display property useful

Let the display property control the name of the display,
like in x(v)imagesink.

waylandsink: access sink->pool in a more atomic fashion

waylandsink: remove the useless wayland_lock

waylandsink: apply the same debug category to all the subobjects

waylandsink: cleanup header includes

waylandsink: split window-related code out to a new GstWlWindow class

GstWlWindow also has API ready to support subsurfaces.

waylandsink: remove callback and redraw_pending variables from the window structure

waylandsink/waylandpool: improve debug message

waylandsink/waylandpool: find the video format from the GstVideoInfo instead of accessing the sink

For the sake of isolation only. The format should be the same.

waylandsink/waylandpool: refactor code

* make use of GstBufferPool::start/stop functions to allocate/deallocate memory
* get rid of struct shm_pool and do all operations cleanly inside WaylandBufferPool
* store a GstVideoInfo during configuration instead of the width & height
  and use the stride from the video info instead of hardcoding its value

waylandsink: split video format related functions out to a separate file

waylandsink/waylandpool: move code around for better readability

waylandsink: move struct shm_pool and its related functions to waylandpool.c

And also make the instance of this struct to be owned by the buffer
pool instead of the element, for the sake of isolation

waylandsink: Use XDG_RUNTIME_DIR instead of /tmp for the shm file

waylandsink: remove unused variables

waylandsink: process display events in a separate thread

This also moves the display-related code into a new GstWlDisplay class,
which takes care of the new thread

waylandsink: tidy up the header files

pnmdec: use GstVideoDecoder Class

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

tsdemux: Don't overwrite original seek value

In accurate mode, we'll be using start to fill in the seek segment,
therefore don't overwrite it with the shifted seek position.

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

glvideomixer: silence incorrect number of arguments in format warning

gldownload: plug a memory leak

glvideomixer: wire up the alpha pad property

glvideomixer: support input frame scaling

glvideomixer: add positioning of input streams

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

glmixer: allow the subclass to choose the sink pad type

Allows custom properties to be placed on the sink pads by subclasses

mpegtspacketizer: live: handle pcr/dts discrepancies some more

When wrapover/reset occur, we end up with a small window of time where
the PTS/DTS will still be using the previous/next time-range.

In order not to return bogus values, return GST_CLOCK_TIME_NONE if the
PTS/DTS value to convert differs by more than 15s against the last seen
PCR

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

mpegtspacketizer: Fix wraparound detection issue

Using 32bit unsigned values for corrected pcr/offset meant that we
potentially ended up in bogus values

Furthermore, refpcr - refpcroffset could end up being negative, which
PCRTIME_TO_GSTTIME() can't handle (and returned a massive positive value)

docs: add gsteglimagememory

docs: add glframebuffer object

docs: misc adds for gl functions

docs: add gluploadmeta object

docs: update for some gl function name changes

glfilterapp: actually emit the client-draw signal

gl/examples: update for TEXTURE_2D change

tsdemux: implement proper seeking for h264 streams.

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

From a high level perspective, the new process for seeking h264
streams is as follows:

1) Rewind the stream until we find the first I-slice of a frame,
   and mark its offset in the stream.
2) Rewind the stream until we find SPS and PPS informations,
   to make sure the subsequent parser is up to date.
3) Accumulate optionnal SEI NAL units on the way.
4) Push the SPS, PPS and SEI units before the new keyframe.

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

dvbsrc: do not try to close negative file descriptors

Interestingly, Coverity implies that close takes an unsigned
argument, while my close(2) man page shows it taking a signed
argument. I guess it may be platforms specific.

Coverity 1214602

hls: fix build with GLib 2.32

Provide internal copy of g_list_copy_deep() until we
bump the GLib requirement.

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

aiffparse: bail out early when bytes per sample is not set

This should always be set for valid files when we get there,
and checking this avoids having ad hoc checks further down
in several places.

Coverity 1139698

codecparsers: h264: fix quantization matrix conversion routine names.

Fix routine names for zigzag/raster scan order conversion routines for
quantization matrices. This ought to use the gst_h264_quant_matrix_*()
naming convention instead of gst_h264_video_quant_matrix_*(), which
derived from the MPEG-2 function names.

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

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

codecparsers: add C++ guards for MPEG-4 and VP8 parsers.

Fix MPEG-4 and VP8 APIs to export their external symbols as pure C
symbols, i.e. un-mangled for C++.

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

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

glmixer: send the stream-start event

and the caps event after.

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

gl/examples: add GST_PLUGINS_BASE_CFLAGS

gl/examples: remove spurious include for doublecube example

gl/examples: add the srcdir and builddir includes

And remove references to $(GST_PLUGINS_GL_*)

glfilterapp: remove the reshape/draw properties

The reshape property was never used.
Replace the draw property with a signal.

Based on patch by Mathieu Duponchelle <mathieu.duponchelle@epitech.eu>

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

glmixer: lock the size of mix->frames to the number of pads

Fixes a segfault with decodebin ! glmixer where the request pads on
both sides were being requested after the state change to PAUSED.

Also fixes dynamically adding and removing pads while glmixer is
in a state >= PAUSED.

gl/examples: Port to using signals instead of properties

Based on patch by Mathieu Duponchelle <mathieu.duponchelle@epitech.eu>

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

glimagesink: replace pointer properties with signals

Based on patch by Mathieu Duponchelle <mathieu.duponchelle@epitech.eu>

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

mpegtspacketizer: Use first group for handling reference pcr

If _set_current_pcr_offset gets called after a flushing seek, we ended
up using the current group for delta calculation ... whereas we should
be using the first group to calculate shifts.

Also add an early exit if there are no changes to apply

tsdemux: Remove unused variable

The seeksegment was no longer used since the switch to calculating segments
when we see data.

mpegtspacketizer: Fix debug statement

mpegtspacketizer: Fix duration evaluation in push mode

When working in push mode, we need to be able to evaluate the duration
based on a single group of observations.

To do that we use the current group values

mpegtspacketizer: Handle early PTS conversion

When handling the PTS/DTS conversion in new groups, there's a possibility
that the PTS might be smaller than the first PCR value observed, due to
re-ordering.

When using the current group, only apply the wraparound correction when we
are certain it is one (i.e. differs by more than a second) and not when it's
just a small difference (like out-of-order PTS).

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

gdp: don't print already-freed string

CID 1221987

speed: Fix position query some more

Forgot to remove code in 16fd917632c0992349ca28cf9108e9562c493ad7

CID #1139677

mpegtsdemux: Fix sticky event handling

When we receive sticky events from upstream, always return TRUE.

Fixes the issue where we receive custom sticky events (such as "uri")
and no pads are created yet.

hlsdemux: Improve parsing quoted key URIs and apply it for I-frame-based stream URI

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

hlsdemux: Don't store the current key in the playlist

It's per fragment and applying to all following fragments
until the next key is specified.

hlsdemux: Fix compiler warnings

hlsdemux: Reload the variant playlist if refreshing a playlist or downloading a fragment fails

This can happen if the playlists have moved due to the variant playlist
now being redirected to another target. This currently only works as long
as the referenced playlists don't change in relation to the variant
playlist, and the new location is purely due to a new path triggered by a
new redirection target of the variant playlist, or a new redirection
target of the playlist itself.

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

schroenc: Use an arbitrary framerate if none if given

The schro encoder always requires a framerate, but some source material,
such as RTP doesn't have a fixed one, so just fake it.

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

dvbsrc: drop redundant assignment

dvbsrc: improve timeout handing at locking loop

New approach attempts to be more accurate by measuring
the elapsed time by iteration. Also:

* Use a 10 seconds default timeout and a half a second
  polling step. New values should better match the tuning
  process on real-life scenarios.
* Correct elapsed_time computation.
* Add _retry_ioctl() to avoid bailing out on temporary
  ioctl EINTR failures (no need to check for EAGAIN cause
  we are opening the frontend on blocking mode)
* Small corrections to fail condition handling

gl/wayland: create a dummy display instead of an EGLDisplay

Currently, GstGLWindowWaylandEGL holds the wayland display connection
If we create the EGLDisplay at the GstDisplay creation time, then
libEGL will internally open another connection to the wayland server.
These two display connections are unable to communicate resulting in
no window output/display and hangs inside libEGL.

Eventually we will move the wl_display from GstGLWindow to GstGLDisplay.

rtmp: proxy logging from librtmp

Helps with debugging various librtmp issues

rtmpsrc: Fix position querying

It's the position we're querying, not the duration :)

tsdemux: Remove extraneous extra parameter from check_pending_buffers function

mpegtspacketiser: Always initialise packet AFC flags and PCR fields

Also fix a slight typo in a comment about the smoothing algorithm

tsdemux: Reset pending_ts on a discont, to re-measure initial timestamps

Since all the other timestamp tracking now gets reset on a discont,
it makes sense to wait for a PCR and timestamp buffers like when
playback first starts

srtp: add support for rollover counters and replay protection window size

We add a new signal, get-rollover-counter, to the SRTP encoder. Given a
ssrc the signal will return the currently internal SRTP rollover counter
for the given stream.

For the SRTP decoder we have a new SRTP caps parameter "roc" that needs
to be set when a new SRTP stream is created for a given SSRC.

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

liveadder: handle gap buffer flag

Remove GAP flag unless both sides have gap data, ignore
incoming data if it is gap data.

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

glimagesink: improve state change debug message

glupload: always release the previous buffer

Fixes the case where _perform_with_buffer() is called without
intervening calls to _release_buffer() as is the case on start up
with glimagesink.

Also release the buffer when reseting the upload.

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

tests: Take account of memory alignment in shm test

gdp: Fail gracefully if event can't be parsed

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

shmsink: Allocate enough memory to do alignment

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

androidmedia: Actually print the stacktrace into the error string

androidmedia: Clear the exception before trying to call Java methods to convert it to a string

androidmedia: Fix crashes when loading the plugin in a standalone application on Android >= 4.4

Check if libnativehelper is loaded in the process and if
it has these awful wrappers for JNI_CreateJavaVM and
JNI_GetCreatedJavaVMs that crash the app if you don't
create a JniInvocation instance first. If it isn't we
just fail here and don't initialize anything.

See this code for reference:
https://android.googlesource.com/platform/libnativehelper/+/master/JniInvocation.cpp

speed: make position query able to convert bytes to time

(same as 744c58d71b21475d2edd5b9334e277cfa4c53260 but for the
position query)

It was only querying in time, but then trying to use dead bytes
to time conversion code.

Coverity 1139677

mpegdemux: avoid crashing in pathological case

(Identical to commit 612cdeec80da95b12f9fad928419c7de2be20d68 which
was for resindvd)

When we'd see an unknown stream type, then a SDDS stream.
Then we'd get to the end of the switch with a NULL temp stream
pointer, and dereference it.

Coverity 1139708