video-info: implement GstVideoInfo as boxed type

GstVideoInfo usually is created on the stack, but boxed type can be useful
for bindings.

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

rtcpbuffer: Fix validation of packets with padding

The padding (if any) is included in the length of the last packet, see
RFC 3550.

Section 6.4.1:
   padding (P): 1 bit
      If the padding bit is set, this individual RTCP packet contains
      some additional padding octets at the end which are not part of
      the control information but are included in the length field. The
      last octet of the padding is a count of how many padding octets
      should be ignored, including itself (it will be a multiple of
      four).

Section A.2:
   *  The padding bit (P) should be zero for the first packet of a
      compound RTCP packet because padding should only be applied, if it
      is needed, to the last packet.

   *  The length fields of the individual RTCP packets must add up to
      the overall length of the compound RTCP packet as received.

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

videodecoder: Fix setting default pixel-aspect-ratio

It's needed to check if pixel-aspect-ratio exists before fixating.
It does not exist if input caps is not set yet and allowed caps
does not contain pixel-aspect-ratio (e.g. when using GST_VIDEO_CAPS_MAKE)

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

Automatic update of common submodule

From f74b2df to 9aed1d7

docs: order and canonicalize the -sections.txt file

Have all sections in alphabetical order. Also make the macro order consistent.
This is a preparation for generating the file. Remove GET_CLASS macro for
some elements, since it is not used and the header is not installed.

cdparanoiasrc: remove unused defines

videoscale: fix debug categories

Use a local category for the default category and fix the import for the
performance category.

basetextoverlay: Fix bug with unused upstream_has_meta

The intention was to skip the allocation query if upstream has decided
to use the overlay meta feature in the caps. We can safely assume that
upstream have done that query already before making this decision. This
is an optimization since doing allocation queries is relatively
expensive.

CID #1308943

Revert "basetextoverlay: remove dead code"

This reverts commit e863e5f8a98ceec0ec0bd24274bbae8795e0ab75.

basetextoverlay: remove dead code

upstream_has_meta is set to FALSE and never changed. The two checks for if
upstream_has_meta will never go to the true branch. Removing the boolean
and the true branches of these checks.

CID #1308943

audioencoder: Don't try to get buffers from an empty adapter

{audio,video}{en,de}oder: Also copy POOL metas and make sure to copy over metas when creating subbuffers

POOL meta just means that this specific instance of the meta is related to a
pool, a copy should be made when reasonable and the flag should just not be
set in the copy.

audiodecoder: Add transform_meta() vfunc with default implementation

The default implementation copies all metadata without tags, and metadata
with only the audio tag. Same behaviour as in GstAudioFilter.

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

audioencoder: Add transform_meta() vfunc with default implementation

The default implementation copies all metadata without tags, and metadata
with only the audio tag. Same behaviour as in GstAudioFilter.

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

videodecoder: Add transform_meta() vfunc with default implementation

The default implementation copies all metadata without tags, and metadata
with only the video tag. Same behaviour as in GstVideoFilter.

This currently does not work if the ::parse() vfunc is implemented as all
metas are getting lost inside GstAdapter.

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

videoencoder: Add transform_meta() vfunc with default implementation

The default implementation copies all metadata without tags, and metadata
with only the video tag. Same behaviour as in GstVideoFilter.

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

rtpbaseaudiopayload: Don't copy memory if not needed, just append payload to the RTP buffer

playbin: remove unnecessary break

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

videoscaler: remove check for below zero for unsigned value

CLAMP checks both if value is '< 0' and '> max'. Value will never be a negative
number since it is a division of an unsigned integer (i). Removing that check
and only checking if it is bigger than max and setting it appropriately.

CID #1308950

audioresample: Also copy metas if their API has no tags attached to it

This is the default basetransform behaviour, being more strict than that
is not really useful.

audioconvert: Also copy metas if their API has no tags attached to it

This is the default basetransform behaviour, being more strict than that
is not really useful.

audiofilter: Also copy metas if their API has no tags attached to it

This is the default basetransform behaviour, being more strict than that
is not really useful.

videofilter: Also copy metas if their API has no tags attached to it

This is the default basetransform behaviour, being more strict than that
is not really useful.

Back to development

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