fieldanalysis: exception when block width to 0

When block width property is set to 0, exception occurs.
This happens due to divide by zero errors in calculations.
block width property can never be 0. Hence adjusting the minimum value to 1.

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

glmixer: fix caps leak in gst_gl_mixer_pad_sink_getcaps()

Caps refcounting was all wrong in this function. Rewrote it and add some
comments to make it clearer.

Fix caps leaks with the
validate.file.glvideomixer.simple.play_15s.synchronized scenario.

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

Signed-off-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>

applemedia: avoid implicit color conversions

Rework the GL texture code a little to avoid implicit color conversion inside
AVF/VT on both iOS and OSX.

corevideotexturecache: free texture cache on iOS

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

avfvideosrc: check for failure to create buffer

gst_core_media_buffer_new and gst_core_video_texture_cache_get_gl_buffer
can fail for various platform reasons.

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

avfvideosrc: fix build failure with clang

Use fabs for floating point input.

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

mpegtsbase: Remove unused latency query code

The behavior changes based on the type of segment,
not on the liveness of the source.

tsdemux: Cleanup latency query handling

The minimum latency is always 0 or more. And we should
requery upstream as it may have changed.

androidmedia: Handle queue/release errors non-fatal

They can happen sometimes as a transient problem, e.g. if there
is a problem in the stream. Only handle them as fatal if many of
them happen in a row.

opencv: Change opencv plugin to use opencv2-style includes

If old opencv1-style legacy include directory is available,
this change becomes purely cosmetic (maybe will compile a bit faster).
It becomes an FTBFS fix when opencv1-style include directory is missing
(possibly because opencv package maintainer decided not to pack it).

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

opencv: Change configure test to check for OpenCV2 headers

* Checks for opencv2 headers only, not for legacy opencv1 headers
* Checks for every opencv2 header that the implementation needs,
  not just highgui_c.h

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

avfvideosrc: get connection and input clock early

mpegtspacketizer: Unify debug statements

Use 0x%04x for PIDs

tsdemux: Unify duration querying code

And properly use it in the SEEKING query.

Fixes seeking with gst-play

videoaggregator: fix caps leak when early returning

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

Signed-off-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>

glmixer: unref owned caps when finalizing the mixer

Fix a caps leak with the
validate.file.glvideomixer.simple.play_15s.synchronized scenario.

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

Signed-off-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>

glmixer: pass the proper free function to frames and buffers array

'array_buffers' contain borrowed GstBuffer and so shouldn't have a free
function. 'frames' is the one containing GstGLMixerFrameData and so should use
_free_glmixer_frame_data as free function.

Fix GstGLMixerFrameData leaks with the
validate.file.glvideomixer.simple.play_15s.synchronized scenario.

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

Signed-off-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>

gldownloadelement: Download *from* OpenGL, not into

openh264enc: Add complexity property for speed/quality tradeoff

glcontext: Make gst_gl_context_egl_activate fail if the old surface could not be destroyed

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

gluploadelement: Unref GstGLUpload object and caps in ::stop()

Fix leak of the GstGLUpload object.

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

gl: Remove some empty ::finalize() implementations

gldownloadelement: Fix element description

gldownloadelement: Remove unused ::finalize() implementation

glcolorconvertelement: Also unref caps in ::stop() already

They are not useful anymore afterwards, so keeping them until ::finalize()
might only cause someone to use them later and then fail.

glcolorconvertelement: fix GstGLColorConvert leak

convert->convert was never unreffed.

This can be reproduce with the
validate.file.glvideomixer.simple.play_15s.synchronized scenario.

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

apexsink: Fix buffer overflow, in case anyone ever ports it.

Fix a simple buffer overflow - 16 bytes isn't enough to hold
the string representation of a gulong on x86_64. I guess the
intent was to generate a 32 bit random key, so let's do that.

Only matters if anyone ever ports the sink to 1.x

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

mpegdemux: fix pad leak when not adding a new pad after no-more-pads

tsdemux: fix segment position tracking for the general case

Different streams can have different PTS/DTS bases, and some
streams may not even have DTS.

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

glcontext: Make gst_gl_context_egl_activate fail if the surface could not be created

Also add some error logging.

h264parse: Don't unref buffer that was unreffed just a few lines before already

Add adaptivedemux, badvideo and badbase to the link-list for the docscanner

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

avfvideosrc: remove unused variables

Complement commit 5496fd3e75e65a4e387715d0ba0740790aefa5f6 by removing
the involved variables.

applemedia: vtdec: fix GL negotiation

avfvideosrc: log timestamps

avfvideosrc: subtract time spent in AVF queues

The time spent in AVF queues was being incorrectly added to running time
rather than subtracted.

avfvideosrc: fix GL texture negotiation

GST_VIDEO_GL_TEXTURE_UPLOAD_META_API_TYPE should no longer be used.
Instead, just get the GL context.

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

dtlsdec: Fix typo

dtlssrtpdec: Add some more debug output

srtpdec: Add some more debug output

dashdemux: enable playback of content with text streams

There is a playback error when trying to play a content that
has 'application' mimeType. This commit prevents an exception from
setup text streams.

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

winks: Reset DTS to GST_CLOCK_TIME_NONE

Otherwise we might get a previous DTS set from a recycled buffer that we get
back from a buffer pool, which then confuses synchronization.

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

amc: Use new JNI utils almost everywhere

amc: Allow creating global and local references of objects

amc: Make GError argument order more consistent

amc: Add helper function for getting a direct buffer array

amc: Java longs are gint64s

amc: Improve JNI utilities and add some missing ones

We now fill GErrors for everything that could throw an exception, and method
calls now always return a gboolean and their value in an out-parameter to
distinguish failures from other values.

glcolorconvert: Fix last commit

gl: Remove iOS/EAGL workaround for R/RG textures

This should be unneeded now after https://bugzilla.gnome.org/show_bug.cgi?id=732507

glcolorconvert: Add correct check for RG/R textures in glcolorconvert too

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

winks: fix debug message parameter format

The first part of the GUID structure is a DWORD
which is defined as an unsigned long, so we need
to either cast or use %08lx.

pcapparse: Take buffer directly from the adapter

No need to make a copy into a new buffer, just take the data
from the adapter.

pcapparse: Produce GstBufferList

wrappercamerabinsrc: remove unused attribute and related pad probe

The variable was never set to true and can be removed along
with the probe in which it used to act

glbasefilter: avoid segfault when gl platform is not available

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

tests: fix type mismatch in varargs passing

A bitmask is 64 bits, but integer immediates are passed as int
in varargs, which happen to be 32 bit with high probability.

This triggered a valgrind jump-relies-on-uninitalized-value
report well away from the site, since it doesn't trigger on
stack accesses, and there must have been enough zeroes to stop
g_object_set at the right place.

tsdemux: handle seeks with no target (ie, keep current position)

Such seeks are used to change playback rate and we do not want
to alter the position in that case, so we bypass the flush/seek
logic, and set things up so a new segment is scheduled to be
regenerated.

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

tsparse: fix timestamps not updating after a PMT change

The PCRs stay locked onto the same PID as before the change,
but the relevant PID has no reason to be the same after it.

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

tsdemux: send new segment when a stream is added

This will happen when the PMT changes, replacing streams with
new ones. In that case, we need to accumulate the running time
from the previous chain in the segment base.

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

tsdemux: keep track of current position

This allows seeking to correctly set the base on the segment.

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

check: Don't run the state change test on the dtls elements

There isn't really any way to test the elements standalone.

videoaggregator: Remove broken timestamps-going-backwards check

This would've also triggered if for some reason the segment was updated
in such a way that PTS went backwards, but the running time increased. Like
what happens when non-flushing seeks are done.

We're doing a proper buffer-from-the-past check a few lines below based on the
running time, which is the only time we should care about here.

ksvideosrc: fix support for DV devices

ksvideosrc: only list capture devices

tests: Use AM_TESTS_ENVIRONMENT

Needed by the new automake test runner

opensles: Explicitly specify layout=interleaved in caps

This is fine to hard-code. Section 9.1.8 of the OpenSL ES 1.1
specification, it is expected that multi-channel audio is always
interleaved.

configure: Depend on openh264 >= 1.3.0

Required for the version check header, as agreed upon on IRC

avfvideosrc: fix setting of buffer offset end

Don't set offset end to random values off the stack.

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

open264: Fix GType/gsize usage

gstopenh264enc.cpp:108:7: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]

openh264: Detect 1.4 API change

As mentionned in release notes : Added new Sps/Pps strategies for real-time
video (replace the old setting variable 'bEnableSpsPpsIdAddition' with
'eSpsPpsIdStrategy')

glimagesink: Disconnect mouse/key event signal handlers from window when shutting down context

glimagesink: Guard against disconnecting invalid signal ids

mxfdemux: fix memory leak

Reset the internal segment before freeing it.

mxf_index_table_segment_parse() allocates data inside the segment
(like segment->delta_entries) which have to be freed using
mxf_index_table_segment_reset().

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

hlssink: write the playlist end marker on EOS

Catch EOS from the multifilesink and add the endlist marker to
the playlist when it happens

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

aggregator: document gap handling behavior

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

aggregator: drop stale white space at warning

aggregator: fix typo in test suite

aggregator: add gap event handling unit test

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

aggregator: implement gap handling

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

Automatic update of common submodule

From bc76a8b to c8fb372

opus: Fix incorrect fall-through condition in property getter

shmsink: add an shm-area-name property

The shm-area-property tells the name of the shm area used by the element. This
is useful for cases where shmsink is not able to clean up (calling
shm_unlink()), e.g. if it is in a sandbox.

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

aggregator: Unify downstream flow return and flushing

Also means that having a non-OK downstream flow return
wakes up the chain functions.

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

aggregator: Flushing is always in pad lock, no need to atomics

The usage of atomics was always doubtful as it was used to release a
GCond

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

aggregator: Reset pending_eos on pad flush

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

aggregator: Unify code to set a pad flushing

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

libgstgl: fix rendering on iOS

Stop assuming that the handle has been set by the time ->create_context is
called. After bc7a7259f357b0065dd94e0668b5a895d83fa53a set_window_handle always
happens after ->create_context in fact.

See also https://bugzilla.gnome.org/show_bug.cgi?id=745090

aggregator: Query latency on first incoming buffer.

And keep on querying upstream until we get a reply.

Also, the _get_latency_unlocked() method required being calld
with a private lock, so removed the _unlocked() variant from the API.
And it now returns GST_CLOCK_TIME_NONE when the element is not live as
we think that 0 upstream latency is possible.

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

audiomixer: Allow downstream caps with a non-default channel-mask

Instead of failing, take the downstream channel mask if the channel
count is 1.

glvideomixer: Don't use context if not present

Avoids assertions at runtime

introspection: Don't use g-ir-scanner cache at compile time

It pollutes user directories and we don't need to cache it

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

directsoundsrc: Implement volume and mute

Using the MixerAPI as IDirectSoundCaptureBuffer doesn't implement volume
control.

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

gleffects: port all effects to GLES2.0

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

hlssink: implement a chain_list to request key unit for segments

upstream might send buffer lists instead of buffers and hlssink's
probe won't get called and a new segment won't be created when needed.

This patch fixes it by adding a chain_list function to the sink pad
that will just pass through the whole bufferlist if no segment needs
to be requested at the moment or convert the list into buffers to
check the proper timestamp to request the next key-unit that will
start the segment.

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

mxfdemux: resurrect some flow return handling

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

mpegdemux: resurrect some flow return handling

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

adaptivedemux: do not try to advance fragment if there is none

It might return OK from subclasses and it could cause a bitrate
renegotiation. For DASH and MSS that is ok as they won't expose
new pads as part of this but it can cause issues for HLS as
it will expose new pads, leading to pads that will only have EOS
that cause decodebin to fail

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

rtpopuspay: Forward stereo preferences from caps upstream

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

rtpopuspay: Set the number of channels to 2 as per RFC draft

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