opusdepay: Set multistream=FALSE on the Opus caps

The RTP Opus mapping only allows mono/stereo, and not multistream Opus
streams.

opusheader: Do not include rate in caps if it is 0

As expressed in gst_opus_header_create_caps, value 0 means unset.
Setting rate value to 0 make negotiation with decoder fail.

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

androidmedia: Fix slice-height for Tegra 3 devices

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

androidmedia: Add support for COLOR_FormatYUV420Flexible

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

androidmedia: Add suport for Intel color formats

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

h264parse: Fix profile and level setting in caps

Don't use the apis in codec-utils to extract the profile and level
syntax elements since it is wrong if there are emulation prevention
bytes existing in the byte-stream data.

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

glcolorconvert: Improve passthrough check when no conversion is needed.

Make the passthrough check contingent on only the fields we
can modify being unchanged, and pre-compute it when caps
change instead of checking on each buffer. Makes the passthrough
more lenient if consumers are lax about making input and output
caps complete.

wrappercamerabinsrc: handle when source creation fail

Remember to set the source to NULL state as adding it to the pipeline
will set it to the READY state.

wrappercamerabinsrc: fix element linking order

Video source should be linked to videocrop and not to
videoconvert as it is done in the main linking path

tests: Add a check for the new compositor pad-is-obscured optimization

We verify that all the buffers on an obscured sinkpad are skipped by overriding
the map() function in the GstVideoMeta of the buffers to set a variable when
called. We also test that the buffers do get mapped when they're not obscured.

Blame^WCredit for the GstVideoMeta map() idea goes to Tim.

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

compositor: Only map the frame from a buffer if it will be used

It's a waste of resources to map it if it won't be converted
or used at all. Since we moved the frame mapping down, we need
to use the GST_VIDEO_INFO accessor macros now in the code above
that instead of the GST_VIDEO_FRAME accessor macros.

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

compositor: use accessor macros for consistency

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

compositor: Skip pads that are completely obscured by a higher zorder pad

For each frame, compare the frame boundaries, check if the format contains an
alpha channel, check opacity, and skip the frame if it's going to be completely
overwritten by a higher zorder frame. The check is O(n^2), but that doesn't
matter here because the number of sinkpads is small.

More can be done to avoid needless drawing, but this covers the majority of
cases. See TODOs. Ideally, a reverse painter's algorithm should be used for
optimal drawing, but memcpy during compositing is small compared to the CPU used
for frame conversion on each pad.

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

h265parse: Fix source caps to report cropped dimensions

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

h265parse: Fix the memory freeing of stored VPS nals

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

h265parse: Fix profile, tier and level setting in caps

Don't use the apis in codec-utils to extract the profile,tier and level
syntax elements since it is wrong if there are emulation prevention
bytes existing in the byte-stream data.

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

codecparser: h265: Fix nal unit size checking

The EOS and EOB nals have the size 2 which is the size of
nal unit header itself. The gst_h265_parser_identify_nalu()
is not required to scan start code again in this case.

In other cases, for a valid nalunit the minimum required size
is 3 bytes (2 byte header and at least 1 byte RBSP payload)

codecparser: h265: Calculate crop rectangle dimensions

codecparser: h265: Fix parsing multiple SEI messages in a single SEI Nal

An SEI Nal can have more than one SEI message.

Change in API: the gst_h265_parser_parse_sei()

srtpdec: Make sure to send caps events downstream before any segment events

Upstream might not give us a caps event (dtlssrtpdec) because it might be an
RTP/RTCP mixed stream, but we split the two streams anyway and should report
proper caps downstream if possible.

Fixes "sticky event misordering" warnings with dtlssrtpdec.

rtjpeg: remove unused quality property

gl: readd glupload/download onto element pads

Allows insertion of gl elements into non-gl pipelines without converter
(upload/download) elements.

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

Revert "glvideomixer: implement with glmixerbin"

This reverts commit 0fb56738a14391f248aa0be8756adeaf978baa0c.

Revert "glimagesink: implement as a bin"

This reverts commit 8a0017e21d5f9a8507f0593c6b24f723aa415258.

Revert "glimagesink: forward ALL the properties on the bin"

This reverts commit 4be45e5f30dc6121f2769323603447f591ca4a0a.

Revert "glimagesink: add pixel-aspect-ratio property on the bin"

This reverts commit 2ba6bb9b9325b63f58a9ff0b2c82fa28759dcabc.

ksvideosrc: don't timestamp buffers for muxed streams

ksvideosrc: fix header size for muxed streams

codecparser: h265: Fix the number of tile rows/columns parsing

The possible minimum value for num_tile_columns_minus1 and num_tile_rows_minus1
is zero (7.4.3.3).

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

mixmatrix: remove unused property enum items

These two properties have been there since the origin of the element but they
aren't used. Removing them.

adaptivedemux: Fix comparision logic

We can't answer the SEEKING query if we do *not* have the manifest

aiff: Re-add noinst_HEADERS instruction

Was removed in the previous android cleanup commit

GstDeviceProvider implementation for WIN Kernel Streaming plugin

gst_ks_device_provider_probe() is a no-braier, just runs ks_enumerate_devices()
and reports the results.

Monitoring is a bit more tricky. We have to create a dummy message-processing
window and register device change notifications for it.

As kernel streaming can (and should) be used for audio capture and audio
playback, this change also has certain placeholders for such.

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

glimagesink: Cache caps for passing to the client draw call

Don't convert the GstVideoInfo to caps on every draw call,
just cache the caps and pass them into the GstSample.

glupload: Make upload method static caps non-const.

Retrieving a GstStaticCaps does a one time internal
initialisation and caches it - they can't be stored as
const structures.

opus: fix includes and compilation against opus in non-standard prefix

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

opus: don't use deprecated gst_buffer_new_and_alloc

Use the helper function available in the base class instead.

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

de265dec: use g_get_num_processors() if available

And provide home-made fallback for older GLib versions,
so that we can later find these and remove them when
we bump the GLib requirement (which is certainly going
to happen before 2.0).

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

gst: remove some unnecessary glib version checks

We require 2.32, no need to check for anything older
than that.

mxf: fix descriptor leak

Free the existing descriptor array, if any, before replacing it.

Fix leaks with the
validate.file.playback.scrub_forward_seeking.test-mpeg2-mp3_mxf scenario.

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

tsdemux: fix taglist leak

If the stream which is about to be removed still has a ref on a tag list we
should drop it.

Fix a leak which was occasionally happening with the
validate.file.playback.change_state_intensive.tron_en_ge_aac_h264_ts scenario.

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

mpegtspacketizer: fix find_subtable() return value

find_subtable() returns a pointer, so return NULL and
not FALSE when nothing is found.

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

win32: update defs

glmemory: remove uneeded unref

The call to _gl_mem_alloc_data will unref and NULLify 'dest' for us.
We just need to return.

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

glimagesink: Use gst_pad_get_pad_template_caps in ::get_caps()

glupload: provide the sink template caps that could be used

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

examples: gst-camera: add zoom property to UI

Add a slider to modify the zoom in camerabin

examples: gst-camera: remove widgets deprecated in GTK3.0

VBox and HBox are now simply Box

examples: gst-camera: update to GTK3

When opening the .ui file it complains about being for GTK 2 and
automatically updates it to GTK3. Commit this version and update
the makefile

wrappercamerabinsrc: remove unused code

The structure is created and never used anymore. Remove it.

wrappercamerabinsrc: use digitalzoom element

Replace videocrop ! videoscale ! capsfilter with the digitalzoom
bin that has the same pipeline internally and already updates
the capsfilter automatically when caps change, removing this code
from wrappercamerabinsrc and making it cleaner.

digitalzoom: change videocrop parameters before passing caps event

Avoids one extra uneeded renegotiation if the elements are already
configured to their final property values when the caps event
goes through.

Also avoids hitting bug https://bugzilla.gnome.org/show_bug.cgi?id=748344

digitalzoom: also skip internal pipeline in upstream caps query

To avoid going through our capsfilter which would limit the choices.

digitalzoom: prevent assertion when caps is any

camerabin: digitalzoom: create a bin element for digital zooming

It contains videocrop ! videoscale ! capsfilter and implements digital
zooming.

At this moment, it is a private element of the camerabin plugin.

This will remove some code used in wrappercamerabinsrc to make
code clearer and digitalzoom can potentially be used by other
applications in the future, it has nothing camerabin specific.

egl: Use maximum bits per color instead of minimum

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

libde265: W32 thread count support

This code is imported from GLib g_get_num_processors(). This function
was added in 2.36 but we depend on 2.32.

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

libde265: Fix format string for gsize

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

spandsp: Don't use the private field 'missing_samples'

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

mpegdemux: remove unused property enum item

The property enum item has been there since the origin of the element but it
has never been used. Removing it.

glfiltercube: Don't initialize multiple shaders on renegotiation

Rename property enums from ARG_ to PROP_

Property enum items should be named PROP_ for consistency and readability.

glimagesink: Remove unused stop function

glimagesink: unref the pool in the correct place

Otherwise we could hold a pool to a context that is never going to be used.

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

gltransformation: Unref shader in ::stop()

gl: unref display/other-context in the correct place

Otherwise state changes from PLAYING->READY->PAUSED will cause there to
to be no display configured on the element.

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

codecparsers: h265: add helpers to convert quantization matrices

Add utility functions to convert quantization matrices from zig-zag scan
order into raster scan order and vice-versa

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

codecparser: h265: skip byte alignment bits while parsing slice header

Skip the byte alignment bits as per the logic of byte_alignment()
provided in hevc specification. This will fix the calculation of
slice header size.

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

codecparser: h265: Fix the NumDeltaPocs calculation

Always use the equation provided in spec (7-57) for finding
the NumDeltaPocs

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

codecparser: h265: Fix the NumPocTotalCurr calculation

The NumPocToalCurr should be calculated for all B and P slices.

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

codecparser: h265: Fix nal size calculation for EOS and EOB

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

glfilter: De-camelcase onInitFBO() vfunc

glfilter: Remove onStart/onStop vfuncs, and unused onReset()

onStart/onStop are just duplicates of the basetransform ones, onReset
was never called but was used everywhere when stop should've been used.

gltransformation: don't initialize multiple shaders on renegotiation

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

* delete shader if one exists
* set it to NULL after unrefing

glcolorconvert: Fix compiler warning

gstglcolorconvertelement.c:230:19: error: unused variable 'in_structure'
      [-Werror,-Wunused-variable]
    GstStructure *in_structure = gst_caps_get_structure (caps, 0);
                  ^

avfvideosrc: drop frames we get before we have a clock

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

glbasefilter: Unref other context in finalize, and display in READY->NULL

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

glcolorconvert: Keep colorimetry and chroma-site fields if passthrough

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

Remove obsolete Android build cruft

This is not needed any longer.

vtdec: fix build error with clang

This commit fixes a simple build error by clang with
by clang with [-Werror,-Wtautological-pointer-compare] options.

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

adaptivedemux: Only answer the duration query in non-live mode

audiovisualizer: fix the license from GPL to LGPL

This was a copy'n'paste buf in the initial commit done by myself.

audiovisualizer: fix the license from GPL to LGPL

This was a copy'n'paste buf in the initial commit done by myself.

examples: motioncells: remove GPL3 code we don't need

If 95% of the code of an example app consists of GObject
code, maybe that's defeating the point a little. So just
remove a lot of that and trim down the example to the
absolute minimum. Also removes the last remaining GPL3
licensed code in -bad.

examples: motioncells: simplify property setting

examples: motioncells: port to 1.x

examples: motioncells: relicense opencv example to LGPLv2

tests: camerabin: add tests for GstPhotography image capture

GstPhotography enables new paths in wrappercamerabinsrc that allows
the source to be notified about the capture caps and provide an
alternative caps if desired bypassing the negotiation (this doesn't
seem like a good idea these days). To make sure it keeps working
until we remove it from the API in favor of standard caps negotiation
features this test was added.

It adds 3 extra tests with a simple test source that will:
1) Test that capturing with ANY caps work
2) Test that capturing with a fixed caps work
3) Test that capturing with a fixed caps and having the source
   pick a different resolution from GstPhotography API works
   by having wrappercamerabinsrc crop the capture to the final
   requested dimensions

wrappercamerabinsrc: Rework cropping for zoom and dimension reduction

wrappercamerabinsrc has a videocrop element to be used for
zooming and for cropping when input caps is different when used
with the GstPhotography interface. The zooming part needs
the following elements:

capsfilter ! videocrop ! videoscale ! capsfilter

The capsfilters should always have the same caps to ensure the
zooming is done and preserves dimensions, unless when it is needed
to do more cropping due to input dimensions those caps
need to be modified accordingly to preserve the output dimensions.

This, however, makes it hard to get caps negotiation to work properly
as we need to have different caps in the capsfilters to account for
the extra cropping needed. It could be simple for fixed caps but it
gets tricky with unfixed ones.

To solve this, this patch splits the zooming and dimension reduction
cropping into 2 separate videocrop elements. The first one does
the dimension cropping, which is only needed when the GstPhotography
API is used and the source provides a caps that is different than
what is requested, while the second is dedicated to zoom crop only.

The first part of the pipeline goes from:

src ! videoconvert ! capsfilter ! videocrop ! videoscale ! capsfilter

to

src ! videocrop ! videoconvert ! capsfilter ! videocrop ! videoscale ! capsfilter

It might add an extra overhead in the image capture as the image might need
to be cropped twice but this can be solved by enabling videocrop to use
crop metas so only the later one does the real cropping.

It also makes the code a bit simpler.

wrappercamerabinsrc: remove obsolete comment

This is already handled in another place and doesn't make sense
in the function context anymore

wrappercamerabinsrc: error out if source fails to prepare for capture

Post an error when preparing the image capture through photography
interface fails

wrappercamerabinsrc: intersect instead of compare for equality

Intersect is enough to check if the requested caps are compatible
with what the source is going to provide. Equality will be too strict.

wrappercamerabinsrc: fix typo

camerabin: tests: remove unused macros

Those macros were with the wrong name (likely a copy n paste mistake)
and were unused.

tests: camerabin: remove obsolete check for 0.10 feature

remove unused enum items PROP_LAST

This were probably added to the enums due to cargo cult programming and are
unused.

glimagesink: balance change_state display ref/unref

the display was being unreffed on the incorrect state change causing
invalid state when changing from PLAYING/PAUSED->READY->PAUSED/PLAYING.

adaptivedemux: Don't claim to be live when answering the LATENCY query

Even for "live" streams we are not live in the GStreamer meaning of the word.
We don't produce buffers that are timestamped based on their "capture time"
and our clock, but just based on whatever timestamps the stream might contain.

Also even if we wanted to claim to be live, that wouldn't work well as we
would have to return GST_STATE_CHANGE_NO_PREROLL when going from READY to
PAUSED, which we can't. We first need data to know if we are "live" or not.

hlsdemux: Use the downloader of the base class instead of creating our own

The one of the base class is completely unused because we override all
the downloading here, so let's just use that one instead.

hlsdemux: Don't error out if we can't match variant playlists after updating

It's better to just select some random variant playlist instead of stopping,
chances are that it's still continuing to work and we might just have to
select a different variant again later.