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.

videoconvert: Keep colorimetry and chroma-site fields if passthrough

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

audio: Change the remaining "samples" in the ::delay() vfunc docs to "frames"

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

tests: tcp: remove SOCK_CLOEXEC which causes build problems on OS/X

It's not needed here.

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

audio: The delay vfunc returns the number of frames, not samples

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

Remove obsolete Android build cruft

This is not needed any longer.

tests: typefindfunctions: add test for UTF-16 MSS manifest typefinding

typefinding: don't read more data than needed in MSS typefinder

typefinding: detect MSS manifests without using g_convert()

Embedded systems often have limited charset conversion
functionality, so don't rely on g_convert() (i.e. iconv)
for UTF-16 to UTF-8 conversions, we can easily enough do
that ourselves by converting to native endianness and
then using GLib's helper functions.

audiovisualizer: fix the license from GPL to LGPL

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

xmptag: fix invalid reads in GST_DEBUG statement

Don't try to print a string that is not NUL-terminated. This
log line does not really seem useful so let's just drop it.

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

remove unused enum items PROP_LAST

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

audiodecoder: Add sink and src query virtual method

API: GstAudioDecoderClass::src_query()
API: GstAudioDecoderClass::sink_query()

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

tests: define GST_CHECK_TEST_ENVIRONMENT_BEACON

Make sure the test environment is set up.

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

configure: bump automake requirement to 1.14 and autoconf to 2.69

This is only required for builds from git, people can still
build tarballs if they only have older autotools.

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

Update .gitignore

video-converter: n_lines member should be a guint not a boolean

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

oggdemux: fix event leaks

gst_event_replace() takes its own reference on the event so we should drop
ours after creating and storing an event using it.

This fix leaks which can be reproduced using the
validate.http.media_check.vorbis_theora_1_ogg scenario.

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

Remove INSTALL file

autotools automatically generate this, and when using different versions
for autogen.sh there will always be changes to a file tracked by git.

Remove LICENSE_readme

It's completely outdated and just confusing, better if people are
forced to look at the actual code in question than trusting this file.

video-scaler: fix YUY2 scaling some more

Take into account the different steps between Y and UV when calculating
the line size for vertical resampling or else we might not resample
enough pixels and leave bad lines.

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

video-scaler: scale enough pixels in YUY2 (and friends) mode

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

tests: rtpbasedepayload: fix crash in test when passing varargs

Need to pass 64 bits where 64 bits are expected.

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

video-converter: Remove unused variables

Remove unused variables n_taps, max_taps in setup_scale()

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

video: add missing part of documentation text

discoverer: fix GstToc leak when parsing toc messages

gst_message_parse_toc() returns a reffed GstToc which is owned by the
GstDiscovererInfo. But we have to make sure we unref its previous value before
setting the new one.

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

win32: Update defs for new API

allocators: make GstFdAllocator non-abstract

Make the GstFdAllocator non-abstract because it is perfectly possible
to make memory from a generic fd. Mark the memory as simply "fd".

audioconvert: fix mixed usage of gint and gint32 in int matrix

This is a fixup for b2db18cda2e4e7951655cb2a34108a8523b6eca9
audioconvert: avoid float calculations when mixing integer-formatted channels

The int matrix was using gint and gint32 synonymously, which can theoretically
cause problems if gint and gint32 are actually different types.

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

gio: fix gvfs plugin dependencies

Try harder to look for gvfs backend changes in the right
place, to make sure the plugin gets reloaded when backends
are removed or installed. We watch the gvfs mounts directory
because the files there contain absolute paths to the
backend executables, and those may not be in the usual gio
path.

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

examples: disconnect scale callback in scrubby

When the position slider's button is released, disconnect the "value_changed"
callback to avoid triggering false seek callbacks.

examples: keep scrubby command consistent

scrubby has two options, wav and playbin. Wav takes a file location so make
the playbin option take a file location as well instead of an uri. This also
means the usage help string will be correct for the playbin option.

examples: no need to set intermediate states

examples: wavparse doesn't need dynamic linking

In scrubby, there is no need to link wavparse with the sink dynamically.
The pad is available when the element is generated.

Change video and audio sinks to the automatically detected sinks.

videodecoder: Break instead of return if default negotiation on GAP events fails

Otherwise we're going to leak the event.

app, videorate: fix CFLAGS and LIBADD order

Make sure local headers are included before installed -base.

examples: remove reference to 0.10 in playrec

examples: remove deprecated function in gtk-videooverlay

gtk_widget_set_double_buffered () has been deprecated since GTK 3.14.
Also, widgets are realized automatically and gtk_wiget_realize () is only
meant to be used in widget implementations.

basetextoverlay: fix buffer leak in chain function

If we don't consume the buffer by passing its reference to
overlay->text_buffer then we need to unref it.

Fix a leak with validate.file.playback.fast_forward.test5_mkv
when running inside Valgrind.

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

appsrc: docs grammar fixes

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

examples: add example description to giosrc-mounting

Also, use GST_MESSAGE_TYPE instead of accessing the GstMessage structure

audiobasesink: fix ring buffer leak on open failure

audiobasesrc: fix ring buffer leak on open failure

examples: reuse variables in encoding example

audiodecoder: Don't post error messages while holding the stream lock

audiodecoder: Don't get and parse the current srcpad caps

We only get here if we don't have any srcpad caps, and we're going
to override the GstAudioInfo a few lines below anyway without ever
using it if for whatever reason we get caps here.

videodecoder: Try to invent default caps instead of setting none at all when getting a GAP event before CAPS

Otherwise we would forward the GAP event without ever providing any caps,
which then would make decodebin expose a srcpad without any caps set. That's
confusing for applications and can lead to all kinds of interesting bugs.

Instead do the same as already is done in GstAudioDecoder, and try to invent
caps based on the sinkpad caps and the caps allowed by downstream and the
srcpad template caps.

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

decodebin: Also log the pointer value of sticky events in debug output

Makes it easier to follow them in the debug logs.

examples: remove unused return value in addstream

Removing unused return value of pause_play_stream ().
Fixing code style to satisfy the git hook.

examples: avoid sprinkle running endlessly

Quit sprinkle when there are no more frequencies to remove.
Also rename for readability the check for linking elements.

tests: Use AM_TESTS_ENVIRONMENT

Needed by the new automake test runner

rtp: rtcpbuffer: fix typo in enum

and in docs. Spotted by Rob Swain.

tests: remove unused filename string from appsink-src2

tests: check file exists before running appsink-src

tests: add missing license headers for example apps

{audio,video}decoder: Forward SEGMENT_DONE events immediately and drain decoders

Otherwise we're going to wait with draining until the next data comes, which
is a bit suboptimal and might take a long time... or maybe never happens.

tests: appsrc: clean up block_deadlock test and make it work in valgrind

Remove all the bus watch and main loop code from the block_deadlock
test, it's not needed: neither pipeline will ever post an EOS or ERROR
message on the bus, and we're the only ones posting an error, from a
timeout. Might just as well just sleep for a bit and then do whatever
we want to do.

Don't gratuitiously set tcase timeout, just use whatever is the
default (or set via the environment).

Make individual pipeline runs shorter.

Check for valgrind and only do a handful iterations when running
in valgrind, not 100 (each iteration takes about 4s on a core i7).

Make videotestsrc output smaller buffers than the default resolution,
we don't care about the buffer contents here anyway.

Fixes test timeouts when run in valgrind.

tests: multisocketsink: fix flaky unit test

On slower systems, or under high system load (e.g. check-valgrind),
the sending_buffers_with_9_gstmemories test would sometimes fail,
because the read call only returns 32 bytes instead of the full
36 bytes expected. This is because multisocketsink might end up
doing a partial write of 32 bytes first, and then write the
missing 4 bytes later, but since we don't wait for all of data
to be written, there's a short window where our read call in the
unit test might then only receive the 32 bytes written so far,
which makes it deeply unhappy.

Instead, make sure we loop to read all bytes.

tcpserversink: don't error out if clients send us something, just ignore it

We don't expect clients to send us any data, but if they do, just
ignore it. Web browsers might send us an HTTP request for example,
but some will still be happy if we just send them data without
a proper HTTP response.

There was a bug in the reading code path. We only have a small
read buffer and would provoke an EWOULDBLOCK trying to read
because we don't bail out of the loop early enough.

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

tests: basetime: fix timeouts when running under valgrind

This test sets a rather short timeout, increase this when
we run under valgrind. Also add a short sleep to the
fakesrc ! fakesink pipeline to avoid thrashing the CPU,
which would often not stop the main loop when it should.

Also fix wrong (0.10) return value from pad probe callback.

videorate: downgrade left-over ERROR debug message

videorate: fix a couple of memory leaks

tests: videorate: fix leak in unit test

doc: Add gst_video_encoder_get_allocator() to doc

tag: exiftag: don't try to convert utf-8 to latin1 if string is ASCII already

Bypass g_convert/iconv if there's nothing to convert. That way,
conversion won't fail on systems where iconv doesn't support
converting utf-8 to latin1 and there's nothing to convert.

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

Automatic update of common submodule

From bc76a8b to c8fb372

oggdemux: fix wrong duration on partial streams with a skeleton index

When a stream has a skeleton index, the stream time is taken from that
index. However, when part of the stream is captured, the index is
invalid as its offsets are now wrong. To avoid this, we ignore the index
when the last offset points beyond the end of the stream (when its
byte length is known).

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

textoverlay: fix disappearing text with high deltax

When deltax is large enough to cause the text to push past the
width of the frame, it would disappear due to a bug in setting
the layout width.

While there, fix a log printing an incorrect width to set.

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

oggmux: fix deadlock when not pulling a buffer from collectpads

oggmux keeps a cached buffer per pad, and pulls buffers from
collectpads to this cached buffer for all pads before processing
the best pad. In some cases, the move from collectpads buffer
to cached buffer is delayed till next call. However, when there
is only one pad, this can't be delayed till next call as there
will be a deadlock since collectpads has no other pad to push to.

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

decodebin2: fix deadlock on chain shutdown

When shutting down the chain, we can get a deadlock when removing
a pad, if that chain was being busy streaming but blocked (eg, while
waiting for a queue to have free space).

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

examples: add license header to scrubby