Release 1.5.2

Update .po files

po: Update translations

streamsynchronizer: Unblock EOS wait when track switching.

sink_event () will blocked on EOS event. which will cause can't
send event when switch EOS track to non-EOS one.

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

streamsynchronizer: Don't wait for sparse streams when doing stream switches

Their stream-start event might come a bit later, like just before the first
buffer... and queues might run full before that happens.

streamsynchronizer: Add some more debug output

streamsynchronizer: Reset group start time when flushing

We reset the group start time to the running time of the start of the other
streams that are not flushed. This fixes seeking in gapless mode after the
first track has played.

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

rtspconnection: Only drop everything after the ; of a session header in requests

For responses it is actually allowed and used to signal the timeout to the
client!

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

audioringbuffer: Fix alaw/mulaw channel positions

For alaw/mulaw we should also try to initialize the channel positions in the
ringbuffer's audio info. This allow pulsesink to directly use the channel
positions instead of using the default zero-initialized ones, which doesn't
work well.

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

tests: fix cpp directives

tests: add PPC64 abi struct sizes

playbin: Reset suburi also when receiving an error message from the sub uridecodebin

http://bugzilla.gnome.org/show_bug.cgi?id=751118

playbin: free group->suburi on failure

If suburidecodebin is failed to negotiate (e.g file does not exist)
then free internal suburi variable so that 'current-suburi' property
returns correct status.

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

oggdemux: set building_chain to NULL when deactivating chain

The chain is about to be invalidated so we shouldn't keep it around.
Prevent a double free crash when the demuxer is being finalized.

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

tools: gst-play: fix seeking issue

For positive seeking segment.stop value will be -1,
when we change rate to -1, then the stop value will be udpated
with the current position. And then again if we change rate to 1,
the segment.stop value does not get updated and remains as position
where we last changed rate to -1. Hence playback stops at that point.
In case of positive rates, call gst_element_new_seek with correct values

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

doc: Fix gsttrtphdrext section name

video: Add missing new symbol to win32 def file

Fixes make distcheck

Add gst_video_multiview_guess_half_aspect() to the docs

multiview: Add gst_video_multiview_guess_half_aspect()

Add a utility function that, given a video size and a
packed stereoscopic mode, attempts to guess if the video
is packed at half resolution per view or not, since
very few videos provide the information.

video-converter: make sure we draw enough border for YUY2 formats

Round width up to 2 so that we draw all border pixels for YUY2 formats

video-scaler: fix scaling of odd width for YUY2 formats

We need to scale groups of 4 bytes for YUY2 formats so round up to 4.
It's possible that there is no Y byte for the last pixel so make sure
we clamp correctly.

discoverer: Fix a wrong naming in the documentation

gst_discoverer_stream_get_missing_elements_installer_details does not
exist, one should use gst_discoverer_info_get_missing_elements_installer_details

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)