glbasememory: assert for a valid allocator instead of segfaulting

tsdemux: add support for LPCM with stream_type = 0x83

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

player: Add some 'const' to config getters

player: G-I annotation fixes

player: move 'position-update-interval' setting to the config struct

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

player: add API to change http user agent

Introducing a new 'config' API similar to GstBufferPoolConfig.

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

Update common submodule

For 'make update-exports' changes

qml: Don't forget to unref the actual sink element after setting it on glsinkbin

qml: Use glsinkbin instead of glupload directly

openh264: fix up for API changes in v1.6.0

Update for API changes in v1.6.0.

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

gl: wayland: improve debug message when connecting to display

And another copy'n'paste-o.

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

hlsdemux: Fix clearing of decryption state

Don't clear decryption state immediately after
initialising it in the start_fragment. Don't clear
the state of all streams when we want to only clear
the current stream.

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

tests: fix bus leak

gst_bus_remove_signal_watch() has to be called to release the ref
taken by gst_bus_add_signal_watch().

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

camerabinpreview: fix bus leak

We were never removing the watch introduced in
gst_camerabin_create_preview_pipeline() so its extra ref on the bus was
never released.

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

hlsdemux: make decryption key handling per-stream

Add demuxer instance-wide decryption key cache. The current and
last key url are per-stream, so make a shared cache. Move the
decryption handling into the stream object, and use the shared
cache for the keys.

hlsdemux: add hlsdemux-specific AdaptiveDemuxStream subclass

Prepare hlsdemux for more than one single stream. Currently hlsdemux
assumes there'll only ever be one stream and most of the stream-specific
state is actually in the hlsdemux structure. Add a stream subclass
instead and move some stream-specific members there instead.

adaptivedemux: Add more safeguards around state changes.

Make state changes of internal elements more reliable by locking
their state, and ensuring that they aren't blocked pushing data
downstream before trying to set their state.

Add a boolean to avoid starting tasks when the main
thread is busy trying to shut the element down.

adaptivedemux: Add debug code to change bitrate.

Add some disabled-by-default debugging code that forces
bitrate changes by modulating the measured bitrate every
few fragments

adaptivedemux: More locking/switching fixes

Try harder to make switching pads work better by
making sure concurrent downloads are finished before exposing
a new set of pads.

Release the manifest lock when signalling no-more-pads, as
that can call back into adaptivedemux again

adaptivedemux: Don't switch streams and expose pads while downloading.

If other stream fragments are still downloading but new streams
have been scheduled, don't expose them yet - wait until the last
one finishes. Otherwise, we can cancel a partially downloaded
auxilliary stream and cause a gap.

adaptivedemux: Drop the manifest lock during outbound actions

Drop the manifest lock when performing actions that might
call back into adaptivedemux and trigger deadlocks, such
as adding/removing pads or sending in-band events (EOS).

Unlock the manifest lock when changing the child bin state to
NULL, as it might call back to acquire the manifest lock when
shutting down pads.

Drop the manifest lock while pushing events.

new plugin: Android hardware sensor source

ahssrc is a new plugin that enables Gstreamer to read from the
android.hardware.Sensor Android sensors. These sensors are treated as
buffers and can be passed through and manipulated by the pipeline.

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

lv2: make it build with older lilv

Implementing the removal of presets is unfortunately not feasibla with
versions < 0.22.
Fixes #768595

webrtcdsp: Add delay-agnostic property

In this mode, we let WebRTC Audio Processing figure-out the delay. This
is useful when the latency reported by the stack cannot be trusted. Note
that in this mode, the leaking of echo during packet lost is much worst.
It is recommanded to use PLC (e.g. spanplc, or opus built-in plc).

In this mode, we don't do any synchronization. Instead, we simply process all
the available reverse stream data as it comes.

jpeg2000parse: Use correct enum type

GST_JPEG2000_SAMPLING_RGB instead of GST_JPEG2000_COLORSPACE_RGB

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

Automatic update of common submodule

From f363b32 to f49c55e

kmssink: Fix offsets handling

The calculation of the offset table was done base on a plane size
estimation. This does not always work. Instead, use memory offset the
same we as it's implement in GstVideoMeta map functions.

Fix various gboolean vs GstFlowReturn problems

Caught by building with MSVC which gave warnings for these

gl/build: fix configure when --enable-cocoa is passed on OS X

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

lv2: planning update

lv2: keep working with older versions

Provide a fallback impl. for the only new function we were using from 0.22.
Fixes #768595

camerabin: update docs section

configure.ac: Cleanup unused variable

We only check for a header, no need for overriding LDFLAGS.

gaudi: fix documentation warning in GstGaussianBlur

Fixing the following warning when generating documentation:
xml/element-gaussianblur.xml:72: element refsect2: validity error :
    ID GstGaussianBlur already defined
<refsect2 id="GstGaussianBlur" role="typedef">
                                             ^
Warning: multiple "IDs" for constraint linkend: GstGaussianBlur.
  DOC   Fixing cross-references

gaudi: fix documentation warning in GstChromium

Fixing the following warning when generating documentation:
xml/element-chromium.xml:74: element refsect2: validity error :
    ID GstChromium already defined
<refsect2 id="GstChromium" role="typedef">
                                         ^
Warning: multiple "IDs" for constraint linkend: GstChromium.
  DOC   Fixing cross-references

webrtcdsp: Relax C++11 check

The library does not require any STL C++11 specific headers.
This fixes our OSX build as we target a SDK version that does not
support C++11 STL.

webrtcdsp: Add Android GNUSTL support

webrtcdsp: Set USE_WEBRTCDSP when disabled

adaptivedemux: Also consider more seek flags as "snap" seeks

In the case of KEY_UNIT and TRICKMODE_KEY_UNITS seeks, we want to
"snap" to the closest fragment.

Without this, we end up pushing out a segment which does not match
the first fragment timestamp being pushed out, resulting in one or
more buffers being eventually dropped because they are out of segment.

webrtcdsp: Also allow webrtc-audio-processing 0.3

It seems to work and is the version available in Debian.

musepackdeck: unsigned can't be less than zero

Less-than-zero comparison of an unsigned value, guint64, is never true

CID 1363334

Back to development

Release 1.9.1

Update .po files

po: Update translations

webrtcdsp: Workaround GCC/MingW 4.7 limitations

It refuses to initialize a classes using brackets notation. This is
to allow building using our mingw version.

configure.ac: Add missing double quotes

That affectation was ignored. This worked on recent GCC as C++11 is
enabled by default now.

lv2: add support for saving presets

Plugns supporting the state interface can now save their presets under '.lv2'.

lv2: support for loading presets

Detect if plugins can do presets. Lazily read a list of presets and add support
for loading.

musepackdec: port to 1.0

shmsrc: Do not call gst_poll_remove_fd upon stop if reading was never started

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

dvb/parsechannels: improve parsing-success message

* Fix typo
* Give information on what was actually parsed

dvb/parsechannels: don't use default conf filename for messages

The file name can be user-defined.

qt: fix build some more when QPA is not available

Compiler would complain about include directory that didn't
exist because QPA_INCLUDE_PATH gets subst-ed regardless
(and if it didn't we'd have just an empty -I argument).

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

gstjpeg2000parse: no need for else after a goto

else clause is redundant when the end of the if statement is a jump.
If we haven't gone to the beach, we are in the false branch.

gstjpeg2000parse: improved parsing of jpc magic and j2c box

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

adaptivedemux: Add custom flow return for allowing subclasses to specify when a fragment is finished

If it is finished before upstream going EOS.

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

dashdemux: Implement SIDX tracking based on buffer offset

This simplifies the code but also removes a bug with tracking of the remaining
size for the initial subfragment: we were not considering the size between the
index and the start of the first moof here.

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

dashdemux: Properly keep track of current offset

GstAdapter does not guarantee to pass through all the offsets, we have to keep
track of it ourselves.

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

hlsdemux: Properly keep track of current offset

GstAdapter does not guarantee to pass through all the offsets, we have to keep
track of it ourselves.

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

hlsdemux: Clear pending data when needed

When switching fragments we don't want to keep any data around from the last
one, and also forget about all data when doing flushing seeks or selecting new
bitrates.

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

adaptivedemux: Get rid of internal stream adapter and let subclasses handle this directly

This allows subclasses to have more control and especially ensure that they
push data downstream with the correct offsets.

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

codecparsers: jpeg2000: don't allow 0 value for array access with i-1

codecparsers: jpeg2000: fix possible out-of-bounds array access

sizeof(array) != G_N_ELEMENTS(array).

CID 1362900

openh264enc: Set frame timestamps before sending to the encoder

openh264enc: Fix initial time-per-frame calculation

openh264enc: Remove meaningless drop bitrate handling

This doesn't even have a property.

openh264enc: Expose maximum bitrate setting

openh264enc: Actually hook up the rate-control property

openh264enc: Use a constant SPS/PPS ID no matter if openh264 older or newer than 1.4 is used

openh264enc: Make slice settings more explicit and don't set any number if not a fixed number of slices is selected

webrtcdsp: Rewrite echo data synchronization

The previous code would run out of sync if there was packet lost
or clock skews. When that happened, the echo cancellation feature would
completely stop working. As this is crucial for audio calls, this patch
re-implement synchronization completely.

Instead of letting it drift until next discont, we now synchronize
against the record data at every iteration. This way we simply never
let the stream drift for longer then 10ms period. We also shorter the
delay by using the latency up the probe (basically excluding the sink
latency. This is a decent delay to avoid starving in the probe queue.

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

webrtcdsp: We now fail if there is no echo probe

When echo cancel is enabled, we now fail the pipeline if there is
not echo probe. For this reason there is no need to check if probe
pointer is set anymore.

glviewconvert: fix transformation matrix usage

Calling glUniformMatrix before the shader is bound is invalid and
would result in errors like:

GL_INVALID_OPERATION in glUniformMatrix(program not linked)

Move glUniformMatrix() to after the gst_gl_shader_use() call.

kmssink: fallback to universal planes if no overlay plane is found

Without setting the DRM_CLIENT_CAP_UNIVERSAL_PLANES capability bit, only
overlay planes are made available for compatibility with legacy clients.

But if a CRTC doesn't have an overlay plane associated, then kmssink is
not able to find a plane for the CRTC and the pipeline will fail, i.e:

ERROR                kmssink gstkmssink.c:482:gst_kms_sink_start:<kmssink0> Could not find a plane for crtc

If no overlay planes were found for a given CRTC, fallback to universal
planes so DRM will also return primary planes that can be used instead.

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

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

kmssink: add a plane-id property

Without setting the DRM_CLIENT_CAP_UNIVERSAL_PLANES capability bit, only
overlay planes are made available for compatibility with legacy clients.

But if a CRTC doesn't have an overlay plane associated, then kmssink is
not able to find a plane for the CRTC and the pipeline will fail, i.e:

ERROR                kmssink gstkmssink.c:482:gst_kms_sink_start:<kmssink0> Could not find a plane for crtc

This patch adds a plane-id property to the kmssink element so a specific
plane can be used in case that a CRTC has only a primary plane associated.

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

openh264enc: Remove broken byte-stream to avc conversion and just output byte-stream as generated by the encoder

The byte-stream to avc conversion did not consider NAL sizes bigger than 2^16,
multiple layers, multiple NALs per layer, and various other things. This
caused corrupted streams in higher bitrates and other circumstances.

Let's just forward byte-stream as generated by the encoder and let h264parse
handle conversion to avc if needed. That way we only have to keep around one
version of the conversion and don't have to fix it in multiple places.

h264parse: Drop filler data

When skipping data, check if they are filler bytes. If so, drop the
data instead of skipping. We don't want to output filler bytes, but they
shouldn't cause a discontinuity.

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

dvbbasebin: Use a non-timer GstPoll

set_flushing() causes inconsistent states on timer polls and also signals a
g_critical() nowadays because of that.

glupload: fix raw data uploader with addition of GstVideoGLTextureType

Fixes regression from 989200820d43c78fb179d7984827800d83787390

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

glvideoconvert: RG/LA is available in OpenGL ES 3, so let that through

vtdec: use L/LA textures on GLES2 as well

Revert "Revert "gl/eagl: try getting a gles3 context""

This reverts commit eb142736318463e6e553c21707ba0d716f85a46f.

vtdec now successfully outputs textures that can be converted by
glcolorconvert in GLES3 contexts.

vtdec: add support for zerocopy decoding with GLES 3.x

It seems that Red/RG textures aren't supported anymore for zerocopy decoding
with GLES 3.x.  Instead use Luminance/Luminance Alpha textures which are.

glmemory: add the texture type to allocate to parameters

Rather than assuming something.  e.g. zerocopy on iOS with GLES3 requires
the use of Luminance/Luminance Alpha formats and does not work with
Red/RG textures.

glcolorconvert: don't assume Red/RG textures will be used

Take the used texture type from the memory instead.

Fixes conversion from multi-planar YUV formats with two components per plane
(NV12, NV21, YUY2, UYVY, GRAY16_*, etc) with Luminance Alpha input textures.

This is also needed for zerocopy decoding on iOS with GLES 3.x.

glcontext: correct precondition for get_gl_version

The intention was to assert if both maj and min were NULL (as there would be no
point calling the function).  Instead if either maj or min were NULL, the assert
would occur.

Fix that.

d3dvideosink: Replace some more gst_caps_to_string() with GST_PTR_FORMAT

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

d3dvideosink: leak on caps string

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

jpeg2000parse: fix typo in sampling check

Fixing small typo that changes the value of the return.

tests: Ignore kmssink for state change test

It can only be put to READY if the DRM stuff is available.

qmlglsink: Fix build error when don't have QPA installed.

Check header file existance and wrap the header file include
in the necessary #ifdef to avoid build error.

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

dvbbasebin: make sure we have an error in case of parsing failure

Drop a redundant comment and rellocate another one while at it.

dvb/parsechannels: implement channels.conf format autodetection

Simple autodetection avoids having to call the parsers
one after the other.

dvb/parsechannels: rename v5/zap parsers for congruency

Add function prototypes while at it.

dvb/parsechannels: fix modulation names

Some names were incorrect. Authoritative source for
the dvbv5 format taken from v4l-utils' lib/libdvbv5/dvb-v5.c

Aditionally, add the missing setter mapping for the
modulation param.

This change makes ATSC work.

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

dvbbasebin: also accept DVBv5 channels.conf format

Confirmed working with ISDB-T & ATSC conf files.

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

dvbsrc: fix ISDB-T property descriptions

Drop some stale trailing whitespace while at it.

dvbsrc: fix misplaced space on doc block

h264parse: When input alignment=AU, always drain each packet.

If the input alignment claims AU alignment, each received
buffer should contain a complete video frame, so never hold over parts
of buffers for later processing. Also reduces latency, as packets
are parsed/converted and output immediately instead of 1 buffer
later.

Fixes a problem where an (arguably disallowed) padding byte on the
end of a buffer is detected as an extra byte in the following
start code, and messes up the timestamping that should apply to
that start code.