ximagesink, xvimagesink: fix string leaks when setting class hint

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

video: Allow using bt2020 by name in colorimetry

As the lookup stops at the first element in the array with a NULL
name, bt2020 could not be used by name. Moving up this entry
fixes the issue.

ximagesink: set WM_CLASS of window

Set WM_CLASS of the ximagesink window so window managers can apply rules
based on xprop filtering.

xvimagesink: set WM_CLASS of window

Set WM_CLASS of the xvimagesink window so window managers can apply rules
based on xprop filtering.

Automatic update of common submodule

From 21ba2e5 to d37af32

libvisual: clean dereferences of private structures

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

libvisual: make private all variable subclasses don't need

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

Automatic update of common submodule

From c408583 to 21ba2e5

docs: remove variables that we define in the snippet from common

This is syncing our Makefile.am with upstream gtkdoc.

Automatic update of common submodule

From 241fcb7 to c408583

Back to development

Release 1.5.1

po: Update translations

rtpbasepayload: Always prefer downstream's ssrc suggestion if any

Otherwise ssrc changes via rtpsession's (deprecated!) internal-ssrc property
are not possible anymore. rtpsession was now patched to only suggest an ssrc
if it makes sense to do so.

In 2.0 we should get rid of all the properties that are also negotiated via
caps, the code and behaviour is too confusing otherwise.

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

rtcpbuffer: Improve documentation of new functions a bit

Also actually add them to the documentation.

rtcpbuffer: Update package validation to support reduced size rtcp packets

According to this section of the rfc.
https://tools.ietf.org/html/rfc5506#section-3.4.2
The validation should be updated to accept more types of RTCP
packages, with this mask change feedback packages will be also
accepted.

Change-Id: If5ead59e03c7c60bbe45a9b09f3ff680e7fa4868

audioresample: copy metadata that only has the "audio" tag.

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

audiofilter: copy metadata that only has the "audio" tag.

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

audioconvert: copy metadata that only has the "audio" tag.

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

discoverer: Serialize the top level DiscovererInfo

Which contains fields such as duration, uri and tags.

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

codec-utils: Add AAC channel configurations 11, 12 and 14 and levels 6 and 7

check: Use GST_CHECK_MAIN () macro everywhere

Makes source code smaller, and ensures we go through common initialization
path (like the one that sets up XML unit test output ...)

pbutils: add description for video/x-cavs caps

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

win32: Update def file for new encoding API

rtpbuffer: optimise payload mapping for buffers with one memory

Micro-optimisation: if the buffer consist of just one memory, we
know we have already mapped that memory to read the headers, so
no need to map it another time to get to the payload data, we
can just set up the payload data details right there and then
and avoid another map call in gst_rtp_buffer_get_payload().
Adds up when receiving RTP-payloaded raw video which can easily
be thousands of packets per frame.

rtpbasedepayload: provide chain_list function on sink pad

Implement a chain_list function, which avoids lots of locking
compared to the default fallback implementation in GstPad.
We may also want to do some more sophisticated timestamp
tracking here at some point, but for now leave it up to the
jitterbuffer and/or subclasses (in case buffers in the
buffer list have no timestamp set on them, there may only
be a timestamp for the whole list on the first buffer).
This provides the exact same behaviour as the default
fallback implementation.

encodebin: Add a way to enable/disabled a GstEncodingProfile

Summary:
So that the user can easily use the same encoding profile to render
with/without audio/video stream.

API:
  gst_encoding_profile_is_disabled
  gst_encoding_pofile_set_enabled

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

tools: gst-play: remove unnecessary variable

The second assignment of sret is never used. We can remove the first assignment
and use the value directly instead.

id3v2frames: Fix compiler warnings

id3v2frames.c:951:20: error: unused variable 'utf16enc' [-Werror,-Wunused-const-variable]
static const gchar utf16enc[] = "UTF-16";
                   ^
id3v2frames.c:952:20: error: unused variable 'utf16leenc' [-Werror,-Wunused-const-variable]
static const gchar utf16leenc[] = "UTF-16LE";
                   ^
id3v2frames.c:953:20: error: unused variable 'utf16beenc' [-Werror,-Wunused-const-variable]
static const gchar utf16beenc[] = "UTF-16BE";
                   ^

part-stereo-multiview-video: Add a section of open design questions

video-format: Fix minor docs typo

videotestsrc: Document the solid-color pattern

playback: Document GST_PLAY_FLAG_SOFT_COLORBALANCE

video: Make gst_buffer_get_video_meta() a real function, Return lowest id

Instead of returning the first video meta found on a buffer, return the
one with the lowest id (which is usually the same thing, except on
multi-view buffers)

discoverer: don't crash on unknown info types when deserializing

Handle unknown info types when deserializing instead of
dereferencing NULL pointers.

Coverity CID 1302394

sdp: prevent the sdp message parser from reading past the end of the buffer

Otherwise, a malformed SDP message could crash the application,
or even maliciously gather data from the memory located after
this buffer...

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

tests: add test for videorate caps renegotiation after a framerate has been calculated and added to caps

The original 0/1 framerate must still be allowed to be configured
on the upstream side of videorate, otherwise future caps renegotiation
is going to fail.

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

videorate: update the caps framerate only in the GST_PAD_SINK transform_caps direction

When a stream has a variable framerate, videorate calculates it and
forces it on the output caps. However, the code in _transform_caps()
currently also does that if the transform is going in the opposite
direction (GST_PAD_SRC), so during a renegotiation it tries to force
upstream to use the calculated framerate and it fails.

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

playsink: use queue to avoid lock in audiotee audio branches

This part of pipeline is:

tee name=t ! visualizationbin ! streamsynchronizer name=s
t. ! s.

streamsynchronizer might block and it could starve the visualization
branch of the pipeline when it is enabled.

The visualization bin has queues internally but the other branch
that links the audiotee directly to the synchronizer is vulnerable
to block. Adding a queue between "t. ! s." fixes deadlocks.

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

basetextoverlay: make deltax and deltay properties controllable

This will be more useful once we have absolute direct
control bindings.

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

oggdemux: fix chain leak

Don't leak the building_chain when destroying.

Fix leaks with the validate.http.playback.reverse_playback.vorbis_theora_1_ogg
scenario.

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

tag: id3v2: fix parsing of UTF-16 text on systems with crippled iconv

Use g_utf16_to_utf8() instead of the more generic g_convert(), so
that we can extract text in UTF-16 format even on embedded systems
with crippled iconv support.

This code path is exercised by the id3demux test_unsync_v23
check in gst-plugins-good.

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

Add new generated rtp enum files to .gitignore

tools: gst-play: keep configured playback rate and trick mode when seeking

Instead of resetting rate to 1.0

po: update for new translatable strings

tools: gst-play: mark more strings for translation

tools: gst-play: add keyboard shortcut help

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

tests: add back videoscale unit test

Has been removed in 835422b2 as part of porting
things over to the new videoscale API.

tools: gst-play: enable interative mode by default

And change --interactive option to --no-interactive.

rtp: Clean G-I files on make clean too

rtp: Add builddir to the include path for gobject-introspection

And also add missing headers/sources

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

win32: Update exports

rtp: Add GstRTPProfile enum

rtsp: Add FIXME 2.0 comment about GstRTSPTransport being an enum instead of flags

rtsp: Use glib-mkenums to generate GstRTSPProfile and GstRTSPLowerTrans GTypes

Revert "oggdemux: Prevent seeks when _SCHEDULING_FLAG_SEQUENTIAL is set"

This reverts commit 76647f2710d718e27f207b005956b7dba72c2d19.

Avoiding pull mode activation is a feature regression, and
demuxers should always use pull mode where that is possible,
e.g. if there's an upstream queue2 with a ring buffer or
a download buffer.

This patch made reverse playback no longer possible over http.

If the goal is to minimise seeks, then that can still be done
by making the demuxer behave differently in pull mode if
the SEQUENTIAL flag is set. If there are bugs, like the demuxer
needlessly scanning the entire file on start-up in pull mode,
then those should be fixed instead.

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

win32: update .def file for new API

rtsp: don't use soon-to-be-deprecated g_cancellable_reset()

From the API documentation: "Note that it is generally not
a good idea to reuse an existing cancellable for more
operations after it has been cancelled once, as this
function might tempt you to do. The recommended practice
is to drop the reference to a cancellable after cancelling
it, and let it die with the outstanding async operations.
You should create a fresh cancellable for further async
operations."

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

gio: don't use soon-to-be-deprecated g_cancellable_reset()

From the API documentation: "Note that it is generally not
a good idea to reuse an existing cancellable for more
operations after it has been cancelled once, as this
function might tempt you to do. The recommended practice
is to drop the reference to a cancellable after cancelling
it, and let it die with the outstanding async operations.
You should create a fresh cancellable for further async
operations."

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

tcp: don't use soon-to-be-deprecated g_cancellable_reset()

From the API documentation: "Note that it is generally not
a good idea to reuse an existing cancellable for more
operations after it has been cancelled once, as this
function might tempt you to do. The recommended practice
is to drop the reference to a cancellable after cancelling
it, and let it die with the outstanding async operations.
You should create a fresh cancellable for further async
operations."

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

gstdiscoverer: Add since annotation.

Forgot to add the since annotation to the
GstDiscovererSerializeFlags in the previous commit.

discoverer: Add serialization methods.

[API] gst_discoverer_info_to_variant
[API] gst_discoverer_info_from_variant
[API] GstDiscovererSerializeFlags

+ Serializes as a GVariant
+ Adds a test
+ Does not serialize potential GstToc (s)

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

rtpbasepayload: Try harder to reuse previously configured caps values and give more preference to anything set as properties

This affects the pt, ssrc, seqnum-offset and timestamp-offset properties. If
they were set from a property, or we configured caps before, we try to use
that value for them. Even if the first structure of the downstream caps
specifies a different value, we check if the value is supported by other
structures.
Only if all this fails, we use the values given by downstream in the first
structure, i.e. if no properties were set and these are the first caps we
negotiate or downstream does not support our values.

By doing this we ensure that we don't spuriously change ssrcs or other fields
in the middle of the stream (and also consider property values more). Ssrc
changes would currently happen after sending an RTX packet (thus creating a
new internal source inside the rtpsession), and then renegotiating the
payloader (which then gets the RTX ssrc from rtpsession).

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

docs: a random set of trivial fixes for the library docs

Warnings down to 35, unused symbols doen to 112.

docs: add fdmemory to docs

docs: a random set of trivial fixes for the library docs

All those where super straight forward from the warnings gtkdoc prints. It kind
of makes sense to apply them before the list of warnings is >100 and people
complain that gtkdoc is noisy.

mikey: fix a bunch of doc warnings

Rename header/source mismatch of parameters. Update the exposed API in
sections.txt.

Revert "doc: Workaround gtkdoc issue"

This reverts commit df7ef3c35d34352257a28307c07d4673f239452e.

This is fixed by the gtk-doc 1.23 release.

appsrc: optimise caps changing when previously-set caps have not taken effect yet

Only negotiate/change caps once when setting caps twice and
the first-set caps have not been used yet.

Based on patch by Eunhae Choi.

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

xvimagesink: fix pool leak

During set caps when config fails, the referenced newpool
is not unref ed.

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

playbin: check the flags before set again

check the previous flags of playsink to avoid the reconfigure of playsink repeatedly

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

doc: Workaround gtkdoc issue

With gtkdoc 1.22, the XML generator fails when a itemizedlist is
followed by a refsect2. Workaround the issue by wrapping the refsect2
into para.

playback: use the new gst_object api

Use gst_object_has_as_anchestor instead of the now deprecated _has_ancestor.

docs: fix up example pipeline

docs: update element example pipelines

- gst-launch -> gst-launch-1.0
- use autoaudiosink and audiovideosink more often
- review pipeline examples and descriptions

video: Update win32 exports for new libgstvideo API

videoconvert: Expose some properties from the videoconverter API

Expose chroma resampler, alpha mode, alpha value, chroma mode, matrix mode,
gamma mode and primaries mode from the videoconverter API.

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

video-converter: Change some implicit string enums to real enums

GST_VIDEO_CONVERTER_OPT_ALPHA_MODE, GST_VIDEO_CONVERTER_OPT_CHROMA_MODE,
GST_VIDEO_CONVERTER_OPT_MATRIX_MODE, GST_VIDEO_CONVERTER_OPT_GAMMA_MODE and
GST_VIDEO_CONVERTER_OPT_PRIMARIES_MODE were G_TYPE_STRING with only a few valid
options. Changed those to real enums.

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

audiodecoder: Also negotiate with downstream if needed before handling a GAP event

videodecoder: Also negotiate with downstream if needed before handling a GAP event

videodecoder: Try to be smarter when clipping buffers without duration/framerate to the segment

2 second frame duration is rather unlikely... but if we don't clip
away buffers that far before the segment we can cause the pipeline to
lockup. This can happen if audio is properly clipped, and thus the
audio sink does not preroll yet but the video sink prerolls because
we already outputted a buffer here... and then queues run full.

In the worst case we will clip one buffer too many here now if no
framerate is given, no buffer duration is given and the actual
framerate is less than 0.5fps.

Fixes seeking on HLS/DASH streams, when seeking into the middle of
fragments and having no framerate/buffer duration.

xvimagesink: fix navigation event leak when early returning

Create the event *after* the early return check so it's not leaked.

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

xvimagesink: fix navigation event leak when not handled

gst_navigation_message_new_event() is *not* consuming the event so we should
always drop our extra reference.

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

navigation: fix structure leak if subclass doesn't implement send_event()

The send_event() implementation is supposed to consume @structure.

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

streamsynchronizer: Don't override segment.base from upstream with 0

Upstream might want to use it to properly map timestamps to running/stream
times, if we just override it with 0 synchronization will be just wrong.

For this we remove some old 0.10 code related to segment accumulation, and
remove some more code that is useless now, and accumulate the group start time
(aka segment.base offset) manually now.

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

rtpbasedepayload: Add some debug output

video: add NV61 format support

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

docs: add new video API to docs

video: check colorimetry and chroma_site equality in gst_video_info_is_equal()

Add VideoInfo accessors for colorimetry and chroma_site and use them
when checking the equality of two GstVideoInfo

video-color: Add gst_video_colorimetry_is_equal()

Add a function for comparing the equality of 2 colorimetry
structures.

oggdemux: remove unused code

These lines have done nothing for about 10 years.

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

pbutils: Use more strict profile checking for hevc

Use the profile_idc value to set the profile string in caps.
Don't use compatibility flags for this purpose.

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

video-converter: Remove unused macro

Remove unused macro GET_TMP_LINE

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

gst-play: add some more key navigation mappings

And don't feed multi-character key descriptors to the
event handler, it won't be what it expects.

navigation: sprinkle some since markers and add new API to .def file

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

tools: Add mouse/keyboard handling from messages

Allows the user to control playback with the window in focus

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

xvimagesink: Post unhandled navigation events on the bus

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

video: Add a new "event" navigation message type

This will be useful for elements that wish to post unhandled navigation
events on the bus to give the application a chance to do something with
it

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

video-info: expose InterlaceMode conversion to/from string

Expose the methods used to convert a GstVideoInterlaceMode to and
from a string.

Rename property enums from ARG_ to PROP_

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