adaptivedemux: tests: added test for fragment download error

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

adaptivedemux: tests: made adaptive demux test thread safe

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

adaptivedemux: tests: added query test

Testing if adaptive demux responds to queries.

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

dashdemux: tests: added test for multiple BaseURL entries

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

dashdemux: tests: added unit test for 6d1eda9391d3143b5fc633edfd94b89dbba209ab

Commit 6d1eda9391d3143b5fc633edfd94b89dbba209ab fixed the period start
time scaling. This patch updates the unit tests to cover that scenario.

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

adaptivedemux: tests: added download error test

Tests adaptive demux behaviour when the uri is wrong and download does
not happen.

adaptivedemux: stop task on multiple download errors

On multiple download errors, we stop the download task and change the src
element state to NULL.

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

adaptivedemux: fix a missing lock in write to stream->last_ret

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

adaptivedemux: return error if src element cannot start

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

adaptivedemux: fixed waiting for fragment updates

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

adaptivedemux: fix header redownload in case of errors

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

adaptivedemux: release manifest lock before sending flush stop event

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

adaptivedemux: reset download_finished before starting the src element

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

adaptivedemux: release manifest lock before changing element state

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

vtdec: fix compilation error

Undefined symbols for architecture x86_64:
  "__gl_target_to_gst", referenced from:
      __do_get_gl_buffer in libgstapplemedia_la-corevideotexturecache.o

gl/tests: update for glmemory api changes

glcolorconvert: remove unused ret variable

glslstage: remove typedefs from the header

They are already defined in the forward decleration header and defining them
more than once will give an error with OSX's clang about typedef redefinition
being a C11 feature.

glcolorconvert: follow the correct texture target names

glupload: explicitly configure the texture target on the gl buffer pool

If we don't, then the pool could end up allocating 2D textures when the
caps explictly state they should be rectangle textures.

glbufferpool: add bufferpool options for the various texture targets

This was chosen over relying solely on the caps as glupload needs to propose an
allocation and set the texture target based on the output caps.  Setting the
caps in the config is currently pointless as they are overwritten in a lot of
element's decide_allocation functions.

This provides a mechanism for the buffer pool to be configured for a certain
texture target when none has been configured.

glcolorconvert: add support for converting texture targets

Solved with a simple shader templating mechanism and string replacements
of the necessary sampler types/texture accesses and texture coordinate
mangling for rectangular and external-oes textures.

glmemory: add support for rectangle textures

Add the various tokens/strings for the differnet texture types (2D, rect, oes)

Changes the GLmemory api to include the GstGLTextureTarget in all relevant
functions.

Update the relevant caps/templates for 2D only textures.

glcolorconvert: order the caps template so that RGBA is first

glcolorconvert ! glcolorconvert would sometimes negotiated RGB
(the first list entry) otherwise.

glbufferpool: move typedef's into the forward decleration file

Otherwise, for example, clang will warn about typedef redefinitions
being a C11 feature.

tests: fixed fake soup http src plugin

The soup http src changed the way it interprets the seek segment stop value.
Previously it was inclusive, now it is not (see commit
21c6da6764c0cd015e9f3c5eecba36e297187deb,
bug https://bugzilla.gnome.org/show_bug.cgi?id=748316)

Updated fake soup http src to also consider segment stop not inclusive.

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

adaptivedemux: tests: added unit test for adaptivedemux

Created a unit test for dashdemux. It relies on a fake SOUP HTTP src plugin
that will feed data to dashdemux. The test controls the data to be
generated and checks the correct data was received for each expected
stream.

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

tests: added fake http src plugin

Added a fake http src plugin capable of generating data buffers for a
request of a http url. Can be used in tests to simulate http accesses.

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

dashdemux: provide a default suggestedPresentationDelay

If MPD@suggestedPresentationDelay is not present in the manifest,
dashdemux selects the fragment closest to the most recently generated
fragment. This causes a playback issue because this choice does not allow
the DASH client to build up any buffer of downloaded fragments without
pausing playback. This is because by definition new fragments appear on
the server in real-time (e.g. if segment duration is 4 seconds, a new
fragment will appear on the server every 4 seconds). If the starting
playback position was n*segmentDuration seconds behind "now", the DASH
client could download up to 'n' fragments faster than realtime before it
reached the point where it needed to wait for fragments to appear on the
server.

The MPD@suggestedPresentationDelay attribute allows a content publisher
to provide a suggested starting position that is behind the current
"live" position.

If the MPD@suggestedPresentationDelay attribute is not present, provide
a suitable default value as a property of the dashdemux element. To
allow the default presentation delay to be specified either using
fragments or seconds, the property is a string that contains a number
and a unit (e.g. "10 seconds", "4 fragments", "2500ms").

ivfparse: Add vp9 support

Differentiate the vp8/vp9 bitstream based on fourcc.

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

dashdemux: improve detection of stream language

Improved the detection of stream's language if the AdaptationSet
contains more than 1 ContentComponent

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

dash_mpd: restrict segment template format strings to %0[0-9]*d as per spec

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

dashdemux: segment template parsing: added support for %d

Added support for %d in template identifier.
Added testcases for %d, %3d, %0-4d identifier formats.

dashdemux: corrected parsing of segment templates

Corrected the parsing of a segment template string.
Added unit tests to test the segment template parsing.
All reported problems are now correctly handled.

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

dahsdemux: task is signaled to stop without stop condition being set

The gst_adaptive_demux_stream_free function is trying to stop the stream's
download task. For this, it signals the task. But it fails to also set the
stream->download_finished = TRUE, so the task will go back to sleep and
only exit when the download is finished.

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

dashdemux: fixed crash when segment timeline list is greater than segment url list

When building the media segment list using a SegmentList node, the
gst_mpd_client_setup_representation function will iterate through the
list of S nodes and will expect to find a matching SegmentUrl node. If
one does not exist, the code made an illegal memory access.

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

dashdemux: unit test for missing profiles

mpdparser: do not dereference profiles is absent

These are used to apply restrictions on what the MPD file may
use, so no profile means no restrictions.

Besides, nothing actually uses the profiles (yet) anyway.

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

adaptivedemux: fixed multithread support

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

dashdemux: parser rejects negative values for mediaPresentationDuration

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

dashdemux: corrected parsing of negative values into unsigned data

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

codecparser: h264: initialize parsing structures

Initialize to 0 these parse structures before filling them: GstH264SEIMessage,
GstH264NalUnit, GstH264PPS, GstH264SPS and GstH264SliceHdr.

When calling the functions which fill those structures, they may fail, leaving
unitialized those structures. This situation may lead to future problems, such
as a segmentation fault when freeing, for example.

This patch initializes to zero these structures, before filling them.

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

codecparser: h265: initialize parsing structures

Initialize to 0 these parse structures before filling them: GstH265SEIMessage,
GstH265NalUnit, GstH265VPS, GstH265PPS, GstH265SPS and GstH265SliceHdr.

When calling the functions which fill those structures, they may fail, leaving
unitialized those structures. This situation may lead to future problems, such
as a segmentation fault when freeing, for example.

This patch initializes to zero these structures, before filling them.

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

liveadder: Make latency property be a uint in millisecs

This restores roughly the same behaviour as the old liveadder element.
Except that the latency now also includes the output-buffer-duration.

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

mpdparser: forbid negative values for duration

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

dashdemux: parser rejects XMLs with negative period duration

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

mpdparser: check segment lists have either duration or timeline

And add error checking along the way.

Add duration where appropriate so unit tests still pass.

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

mpdparser: validate representation set identifier

It must have no whitespace, and must comply with RFC 1738 when
used to build a URL.

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

glslstage: initialize output params to 0 before calling gl functions

The client side API of the Chromium's GPU Process has asserts
in debug mode that check that output params are initialized to 0.

mxfmux: Remove some dead code that could never be called

Coverity CIDs #1328818, #1328819, #1328820.

opengl: build/provide glstereomix/split on gles only systems

geometrictransform: rename gemetric math functions to have their symbols namespaced

Otherwise those symbols can conflict with external libraries when
linking everything statically for mobile targets.

Use the gst_gm_ prefix, short for gst geometric math.

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

waylandsink: don't advertise the local wl_display context

This wl_display proxy is temporary only until waylandsink goes NULL,
at which point the connection to the display is disposed. Unfortunately,
if this is advertised as a GstContext, playbin will cache it and re-feed
it to the sink when it goes PLAYING again, but the wl_display pointer
will at that point be invalid and cause a crash.

Another solution to the problem would be to also cache the GstWlDisplay
object inside the GstContext, which would automatically ref-count
the display connection, but I see no reason in doing that at the moment,
as there are no known users of this GstContext outside waylandsink.
It's probably better to avoid chasing hidden refcounts.

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

glcolorconvert: allow outputting multiple textures for gles3 contexts

the USING_GLES2 includes all GLES3 contexts as well which does support
drawing to multiple buffers.  Instead make or decision solely based on
whether glDrawBuffers is available or not.

glviewconvert: expose the element on gles2 platforms

We can do everything with gles3 however gles2 restricts us not allowing
separated or frame-by-frame multiview modes due to multiple draw buffers.

tests: hlsdemux_m3u: add test for master playlist with missing variant list uri

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

hlsdemux: don't crash or leak memory on broken master playlist

If a (master) playlist contains a variant list entry without a
URI then during parsing of the next variant list entry we are
a) leaking the entry we're currently parsing (new_list), and
b) free'ing the pointer to the previous list entry (list) without
   updating the pointer.

Hence when then adding the URI for the latest parsed entry, incorrect
information is stored, as the information is used from 'list' which
is not valid memory anymore, also leading to crashes.

Fix this by correctly storing the new variant list entry pointer
as needed.

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

hlsdemux: change dispose function to finalize

hlsdemux: remove helper function only used once

hlsdemux: remove deprecated 'fragments-cache' property

It's been non-functional for two cycles now, let's
get rid of it.

hlsdemux: m3u8: move file lookup by sequence directly into code

Nicer to read, two lines of code less, and also the callback
function should've been a GCompareFunc that returns a gint
and not a boolean (it did work correctly, was just confusing).

hlsdemux: m3u8: clean-up: remove unused argument in function call

hlsdemux: m3u8: clean-up: reindent header for better readability

And remove duplicate define.

mxfmux: Remove empty set/get_property()

We can add them again if we ever add properties.

mxfmux: Port to GstAggregator

mxfmux: Set GC essence element UL version to 0x01

ffmpeg otherwise rejects it and the spec is not 100% clear about that.

srtp: Fix critical warning trying to retrieve SSRC from a non fb message

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

mxf: Reorder CFLAGS and LIBS

aggregator: Add create_new_pad() vfunc to allow subclasses to override the default behaviour

Not all aggregator subclasses will have a single pad template called sink_%u
and might do something special depending on what the application requests.

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

mxfmux: Set KAG size to 1

We're not aligning our output in any way, and 0 is invalid.

mxfmux: Don't let the header partition's prev_partition point forward to the footer

mxftypes: Fix generation of operational pattern UL

mxfdemux-structure: Sync fakesink state with the bin to make the application work at all

mxfmpeg: h264 is always byte-stream inside MXF

mxf: Add unit tests for H264/PCM

mxfmux: Don't leak SEEK events when dropping them

mxfvc3: The wrapping is the 15th byte of the essence container UL, not the 16th

In other mappings it is the 16th though.

mxfdemux: If seeking to the remaining parts of the file fails on EOS, consider the stream done

Without this we would run this while loop forever, always seeking again for
the same stream.

mxf: Properly enable unit tests again

mxf: Re-enable mxfmux element

mxfjpeg2000: Fix caps

There is no fields field anymore.

mxfmux: Don't fail SEGMENT events

mxfmux: Push stream-start and caps event before segment event

And don't push the caps event during instance initialization already.

mxfaes-bwf: Format is unsigned if it's 1 byte per channel

... not 1 byte per block.

opencv: handdetect: no need to get a writable buffer

No need to get a writable buffer inside cv_trans_func of opencv video
filter. GstBaseTransform takes care of this.

Automatic update of common submodule

From b99800a to b319909

Use new GST_ENABLE_EXTRA_CHECKS #define

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

Automatic update of common submodule

From 9aed1d7 to b99800a

glshaderelement: free the vertex and fragment strings on finalize

Fixes a memory leak when using the vertex/fragment properties.

androidmedia: fix build errors

printf %lld instead of G_GINT64_FORMAT.
Double ret variable declaration.

glshaderelement: remove unneeded reference to <gst/gl/gstglshadervariables.h>

gl: fix usage of string.h and stdio.h functions without including the headers

e.g:
gstglcontext_egl.c:613:7: error: implicit declaration of function 'strcmp'
[-Werror=implicit-function-declaration]
   if (strcmp (G_MODULE_SUFFIX, "so") == 0)

glimagesink: create a context in NULL_READY

So that it's possible for decoders et al. to request the OpenGL context
in their READY_PAUSED transition with decodebin/playbin.

amcviddec: use gstcontext to retreive the OpenGL context

androidmedia: Only allow GL output if the decoder has unknown color formats

If GST_AMC_IGNORE_UNKNOWN_COLOR_FORMATS is set to yes, non-GL output
is still allowed.

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

androidmedia: Add support for GL output in amcvideodec

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

androidmedia: Do not flush codec if it is not started

androidmedia: Allow object to be NULL in gst_amc_jni_*unref functions

androidmedia: Fix debug statement (%d for a gsize argument)

androidmedia: Improve color format debug messages