gi: Use INTROSPECTION_INIT for --add-init-section

This new define was added to common. The new init section fixed
compilation warning found in the init line that was spread across
all files.

Automatic update of common submodule

From 6015d26 to f74b2df

tools: gst-play: error out instead of crashing if there's no playbin element

gi: Skip Scaler, Chroma, Conveter, Dither constructor

Please box these types before removing the skip mark.

gi: Add (transfer none) for various video meta

These method chains gst_buffer_add_meta() which is also transfer
none.

gi: Fix warnings in GstRtsp

* The custom GSource is not boxed (skip for now)
* The comment block has wrong name for _read_socket()

gi: Don't produce gir and typlib for GstRiff

The API does not follow the type naming convention. Re-enable
only if one take the time to box and rename (see (rename-to SYMBOL)
annotation) all types.

gi: Skip fft constructor for now

These types have never been boxed, hence cannot be used
safely in interpreted languages. This fixes warnings.

gi: Fix warnings in libgstaudio

* Duplicate section
* Miss-named parameter
* Missing transfer none annotation for meta

basetextoverlay: add "draw-shadow" and "draw-outline" properties

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

videoencoder: fix gtk-doc chunk for new function

basetextoverlay: Rewrite negotiation method

This cleanup the negotiation function by properly splitting the probe
and the decisions. This allow handling correctly pipeline where upstream
caps have special memory type. An example pipeline is:

  gltestsrc ! textoverlay text=bla ! fakesink

The upstream caps will be memory:GLMemory, which isn't supported by the
blitter.

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

xvimagesink: Don't share internal pool

Sharing the internal pool results in situation where the pool may have
two upstream owners. This creates a race upon deactivation. Instead,
always offer a new pool, and keep the internal pool internal in case
we absolutely need it.

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

ximagesink: Don't share internal pool

Sharing the internal pool results in situation where the pool may have
two upstream owners. This create a race upon deactivation. Instead,
always offer a new pool, and keep the internal pool internal in case
we absolutely need it.

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

videoencoder: Add gst_video_encoder_set_min_pts()

For streams with reordered frames this can be used to ensure that there
is enough time to accomodate first DTS, which may be less than first PTS

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

Update .def file for new API

videodecoder: Copy multiview-mode, flags and view count from ref info

When copying info from the reference input state, duplicate
all the fields of the video info. The sub-class will have the
chance to override them later.

video-scaler: enforce same taps when combining scalers

video-scaler: make sure to clamp to max width

When estimating the area that should first be vertically scaled, make
sure we clamp to the max input size or else we get invalid reads.

video-scaler: Enforce same taps on Y and UV scalers for merged formats

Make sure we have the same number of taps for the Y and UV scalers so
that the scalers can be merged correctly.

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

rtsp: Add a FIXME 2.0 for gst_rtsp_connection_create_from_socket()

There's a couple of redundant arguments from the pre-GIO days.

typefinding: check for full UTF-8 BOM in MSS typefinder

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

typefindfunctions: UTF-8 MSS Manifest detection support

Check if the first bytes of data contain an UTF-8 BOM.

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

playbin: Check in autoplug_continue against the subtitle factory caps correctly

6a2f017bfa9cb73c6db65eea0b84b1d5b56febb7 changed it to check the subtitle
factory caps if there is a text-sink but we fail to get its sinkpad. What
actually should be done here is to use the factory caps if there is no
text-sink at all.

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

playbin: Fix some warnings with clang around multiview enums

There is the GstVideoMultiviewMode enum and the
GstVideoMultiviewFramePacking, which is a subset of the
multiview modes, with the same values as the corresponding
types from the full enum. Do some casts and use the right
times to avoid implicitly using/passing GstVideoMultiviewFramePacking
when a GstVideoMultiviewMode is needed.

tests: Fix video libs test for multiview GstVideoInfo change

The GstVideoInfo struct was changed late in integrating the
multiview changes, and I forgot to run and fix the unit test.

playbin: Implement multiview frame-packing overrides

Add GstVideoMultiviewFramePacking enum, and the
video-multiview-mode and video-multiview-flags
properties on playbin.

Use a pad probe to replace the multiview information in
video caps sent out from uridecodebin.

This is a part implementation only - for full
correctness, it should also modify caps in caps events,
accept-caps and allocation queries.

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

video: Add multiview/stereo support

Add flags and enums to support multiview signalling in
GstVideoInfo and GstVideoFrame, and the caps serialisation and
deserialisation.

videoencoder: Copy multiview settings from reference input state

Add gst_video_multiview_* support API and GstVideoMultiviewMeta meta

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

rtpbaseaudiopayload: Use GST_BUFFER_PTS() instead of GST_BUFFER_TIMESTAMP()

playsink: fix the channel of color balance element

When traversing the color balance element channel list to find the one that
matches with the playsink proxy, the assignation was set to iterator of the
playsink proxy, not the balance element. Thus, the mapping to the values of
the balance element channel was wrong.

This patch fixes the assignation of the color balance element channel, so the
mapping to the channel of the color balance element is fixed.

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

playsink: cannot enable text flag while playing

when text playbin is not enabled in the beginning, then
video_srcpad_stream_synchronizer gets linked to videochain->sinkpad
and when we try to enable text bin during play, since it is already linked to videochain,
text chain does not get linked properly. Hence unlinking the same
before linking to text chain

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

win32: Update defs file

GstRTSPConnection: Add GTlsInteraction support

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

tools: gst-play: don't print 64 whitespaces next to the time indication

Printing 64 whitespaces to erase the "Paused" message (after \r) would make
it wrap to the next line on shorter terminals. Instead we only print the
amount of spaces needed. Also mark the "Paused" string for translation
while we're at it.

cruft: add the obsolete tmpl dir to cruft-dirs

Update win32 exports

alsa: report recoverable device failures to base class

This gives custom slave methods in the base class a chance to
resynchronize themselves

Signed-off-by: Carlos Rafael Giani <dv@pseudoterminal.org>
https://bugzilla.gnome.org/show_bug.cgi?id=708362

audiobasesink: added custom clock slaving method

This new clock slaving method allows for installing a callback that is
invoked during playback. Inside this callback, a custom slaving
mechanism can be used (for example, a control loop adjusting a PLL or an
asynchronous resampler). Upon request, it can skew the playout pointer
just like the "skew" method. This is useful if the clocks drifted apart
too much, and a quick reset is necessary.

Signed-off-by: Carlos Rafael Giani <dv@pseudoterminal.org>
https://bugzilla.gnome.org/show_bug.cgi?id=708362

Automatic update of common submodule

From d9a3353 to 6015d26

tools: gst-play: add shortcuts to switch audio/subtitle/video tracks

playback: Skip 'ANY' capsfeature while finding the count of common capsfeatures

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

playback: Add gstplaybackutils.{h,c} to deploy the common subroutines

Bring some of the helper functions in gstplaybin2.c to new files
gstplaybackutils.{h,c} which can be utilized by other files
in gst/playback too.

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

Automatic update of common submodule

From d37af32 to d9a3353

tools: gst-play: sort directory entries

When adding a directory to the playlist, the order would be whatever
g_dir_read_name returned. Sorting these using natural sort order.

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

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