Revert "codecparsers: remove ignored increment of return"

This reverts commit 916b954315abc2f94348ec0be3e116c19b080b54.

Clearly something else was intended, and it also makes
more sense to add the extra bit. The resync marker is
N zero bits plus a 1 bit, and the pattern/mask needs to
be run on N+1 bits too.

(Even after the rever the code doesn't do that of course, so
it still needs to be fixed differently.)

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

opensles: Fix build with Android API level < 14

Headers were broken on older Android versions, apparently.

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

openslessink: Allow setting the stream type via a property

openslessrc: Implement recording presets

This allows us to signal what kind of audio we are expecting to record,
which should tell the system to apply filters (such as echo
cancellation, noise suppression, etc.) if required.

decklinkvideosink: Remove late frame-dropping workaround for basesink bug

This was fixed by https://bugzilla.gnome.org/show_bug.cgi?id=749258
in basesink, and is not necessary to duplicate here anymore.

glmixerbin: implement proper dynamic pad removal

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

glimagesink: Don't do pool caching

We now know that pool caching can cause renegotiation issues
when an element in the pipeline change from passthrough to not
passthrough. As it's not needed, don't cache existing pools.

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

configure.ac: Remove reference to cvs

tsdemux: Segment start should match first PTS

The segment should start at first PTS, and the vairable name lower_pts
state so correctly. Though we where using the first DTS instead. This
could lead to small desynchronization of video stream.

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

tsmux: Add negative DTS support

Use the saved DTS, make it signed and pass that to the stream muxer. This
preserves the running time sign. All usage of -1 as invalid TS are now
replaced with G_MININT64. Negative values will be seen as wrap-around
point, but the delta between PTS and DTS will remain correct. Demuxers
don't care about absolute values, they only cares about deltas.

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

tsmux: Remove uneeded cast and cast macro

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

mpegtsmux: Properly detect backward DTS

There was code to detect backward dts, but the marker min_dts
was never set. Setting it enable this feature that prevents
potential integer overflow when generating TS.

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

dashdemux: fixed segfault for missing default namespace

Added a check for a_node->ns before accessing a_node->ns->href in
gst_mpdparser_get_xml_node_namespace. This could happen if the xml
is missing the default namespace.

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

decklinkvideosink: Don't require the same framerate in the input as the mode's framerate

We only really care about the timestamps for the sink.

examples: playout: Add better help text

This should help people figure out how to use the example a bit better

examples: playout: Decrement the currently-playing counter correctly

Only do that when we're removing an item that was playing, otherwise we'll mess
things up while trying to edit the play queue (playlist).

playout: New example for seamless audio/video playback

An example app that takes video URIs as command line arguments and switches
between them seamlessly one after the other using compositor and audiomixer.
Both audio-video and video-only media files are valid inputs, but mixing files
of both types in a single invocation is cumbersome to support, and hence does
not work. The example attempts to keep the audio stream moving along perfectly,
and duplicates video frames where necessary to cover gaps in the video
timestamps using the 'ignore-eos' videoaggregator pad property.

Ensuring seamless (and mostly-glitch-free) switching is harder than it sounds,
and hence the example contains plenty of pad probes and running time
calculations to make things work.

The GPtrArray play_queue contains items that are being played back, have been
prepared for playback, and will be played back in the future. The queue itself
is mutable besides the first two items (playing and prepared). The item that has
been prepared should not be edited or removed since it has been prepared in
advance to be activated immediately on the current item's EOS.

The example also has support for switching to the next item in the queue
prematurely; see the --switch-after/-s flag to the application.

Note: the output video is hard-coded at 1280x720, and input video is scaled as
needed to fit this size. Set OUTPUT_VIDEO_WIDTH/HEIGHT to change this.

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

tests: Add test for the 'ignore-eos' compositor sink pad property

When the 'ignore-eos' property is set on a pad, compositor will keep resending
the last buffer on the pad till the pad is unlinked. We count the buffers
received on appsink, and if it's more than the buffers sent by videotestsrc, the
test passes.

videoaggregator: add "ignore-eos" property for input pads

When set, it causes videoaggregator to repeatedly aggregate the last buffer on
an EOS pad instead of skipping it and outputting silence. This is useful, for
instance, while playing back files seamless one after the other, to avoid
videoaggregator ever outputting silence (the checkerboard pattern).

It is to be noted that if all the pads on videoaggregator have this property set
on them, the mixer will never forward EOS downstream for obvious reasons. Hence,
at least one pad with 'ignore-eos' set to FALSE must send EOS to the mixer
before it will be forwarded downstream.

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

mpegts: return early if packetizing short section

Short sections have 3 bytes of common header, while other sections
have 8 bytes of common header. If packetizing common header of short
section, we should stop after the first 3 bytes.

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

debugutilsbad: Minor cleanup

Ran gst-indent on debugutilsbad.c, moved headers to noinst in Makefile.am

gtk: Do not try to activate a NULL GLContext

At that point in the code nothing guarantees it exists

gl/tests: remove reference to undefined buffer test

glmemory: separate pbo transfer from texture transfers

When supported, the potentially longer pbo upload/download can be
initiated before the texture upload/download, potentially increasing
throughput.

glmemory: implement on top of glbasebuffer

Provides convenient access to PBO usage.

Currently texture updates are coupled tightly to data transfers.

gl: new glbasebuffer GstMemory object

Provides generic handling of GL buffer objects accessible using
the GL bind points (GL_ARRAY_BUFFER, GL_PIXEL_*_BUFFER).

Implementation based off the current GstGLMemory.

gl/prototypes: add some buffer function prototypes

gtk: implement video aspect-ratio handling

For both the software and the GL sink's.

Doesn't deal with the pixel-aspect-ratio field at all yet.

gl/window/x11: only listen on the X Display when needed

5697b6b89b4b2a15c45bd47be940a17f4412ea11 causes us to possibly listen
on a toolkit provided Display connection.  We thus could eat their
precious winsys events.  Only listen if we need to
(!foreign_display or videooverlay).

gtk: fix a couple of typos

gtkglsink: reset the context/display in READY_TO_NULL

Fixes context propagation in pipelines with upstream GL elements.

dashdemux: fix error calculating large presentationTimeOffset values

If the presentationTimeOffset attribute of a DASH manifest contains
a value that is larger than 2^32, gstmpdparser incorrectly calculates
the stream's presentation time offset. This is due to two bugs:

1: Using gst_mpdparser_get_xml_prop_unsigned_integer rather than
   gst_mpdparser_get_xml_prop_unsigned_integer_64 to parse the
   attribute
2: gst_mpd_client_setup_representation multiplying the value by
   GST_SECOND and then dividing by timescale

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

gstgtk: No need to realize the widget

The widget already does that.

gstgtk: Don't leak the widget

g_object_get() returns a ref, gtk_container_add() only ref_sink().
That mean we still need to unref afterward. This leak was hiding
a reference bug previously present.

gstgtk: Allow doing gst-inspect-1.0 on these elements

This patch allow going gst-inspect-1.0 on these elements removing
ugly crash that was previously occurring. The method consist of
making the widget creation as lazy as possible. This way we don't
endup doing gtk_init() before the application. We also ref_sink()
the widget, so we don't crash if the parent widget is discarded,
and cleanly error out with GL if the widget has no parent window,
because calling gtk_widget_realized() can only be done if the widget
has been parented to a window).

gl: Add support for multiple views in upload, colour convert and download

Support multiple attached views on input/output buffers
by processing each one, not just the first.

h264parse: Add support for passing stereoscopic/multiview info

Pass any multiview info from the container or SEI info downstream

h264parse: Don't switch to passthrough on set_caps()

Wait until at least one keyframe has been parsed before
deciding to switch to passthrough mode, in case the
stream contains SEI messages that supplement the output
caps - for example by providing stereoscopic information

gl/window/x11: don't create our own X11 display

It's not needed anymore with most window operations occuring in the
GL thread.

gl/window/x11: handle_events() may be called before the window has been created

Fixes an XIO fatal error

gl: move basesink properties from glimagesinkbin to glsinkbin

mpegtsmux: Remove arbitrary constraint on prog-map program ids

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

gl: Use gst_object_ref_sink() for gl{filter,mixer,src}bin too

glsinkbin: Use gst_object_ref_sink() for consistency with the video-sink property on playbin

gtk: Do not try to initialize display if we have not have a GLContext yet

gtk: Add missing CFLAGS to example

Implement gtk sinks

two sinks are provided.  gtksink which is a cairo/software based renderer
and gtkglsink which utilises the GL support in gtk and gstreamer.

h263parse: Fix PSC matching

We were off by one byte in the matching

It should be (using 24 bit matching):
   * startcode  : 0000 0000 0000 0000 1000 00xx
   * mask (bin) : 1111 1111 1111 1111 1111 1100
   * mask (hex) :    f    f    f    f    f    c
   * match      :    0    0    0    0    8    0

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

vtenc: always enqueue frames, even on error

Even when we fail to encode frame, we should still enqueue it so
it could be passed into handle_frame (with output_buffer == NULL).
Otherwise, we risk GstVideoEncoder's queue of frames growing unbounded.

Note: We're slightly changing the renegotiation code to accommodate for
frames without output buffers, but this commit takes no ownership over
the way negotiation is being done.

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

d3dvideosink: Avoid frame rendering while the window is completely hidden

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

gl: OES_vertex_array_object is improperly implemented on PowerVR SGX 544MP

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

vtenc: free input buffer ASAP

VTCompressionSessionEncodeFrame retains the CVPixelBuffer during
encoding, and will release it as soon as it can (e.g. before it even
calls our callback). This means we can safely release input buffer
at this point, possibly allowing the system to reuse it sooner.

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

vtenc: Add missing gst_vtenc_frame_free() in error path

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

liveadder: Fix memory leaks iterating over pads

Rebased for latest master by Nicolas Huet <nicolas.huet@parrot.com>.

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

vtenc: add alignment=au to sink caps

VT compression callback gets a CMSampleBuffer per frame, so vtenc_h264
should set 'alignment=au' in sink caps to indicate this.

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

uvch264src: fix compiler warnings

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

avfassetsrc: fix huge memory leak

CMSampleBuffers were retained -> huge memory leak.

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

vtenc: no need for queue_length with try_pop

vtdec: don't copy meta from input to output

Copying arbitrary metas is going to cause problems and this should really be
handled by the base class. It overrides most other things already anyway,
including timestamp and duration. Those are just set here now so we can
insert the frame sorted into the queue.

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

cruft: add the obsolete tmpl dir to cruft-dirs

audiomixer: fix misleading documentation copied from adder

dash: Add $(GST_PLUGINS_BASE_LIBS) to LIBADD

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

Automatic update of common submodule

From d9a3353 to 6015d26

dvdspu: Map the pixel data once per render, not *ahem* twice per byte.

The naive port to 1.0 from years ago was silly, make it better.

Automatic update of common submodule

From d37af32 to d9a3353

tests: dashdemux: add unit tests for checking the parsing of MPD element

Create a dash test target and add unit tests that check the parsing of
attributes of the MPD element and the following child elements:
 - baseURL
 - program information
 - location
 - metrics

These tests check the compliance of parts of the MPD parser in the
dashdemux element against the DASH specification [1].

    [1] http://standards.iso.org/ittf/PubliclyAvailableStandards/c065274_ISO_IEC_23009-1_2014_Electronic_inserts.zip
        http://standards.iso.org/ittf/PubliclyAvailableStandards/c065274_ISO_IEC_23009-1_2014.zip

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

videoaggregator: simplify aggregate returning

Rework special handling with goto/labels to only have one case
and otherwise just return normally.

videoaggregator: refactor caps reconfigure to its own function

Makes the aggregation code shorter and easier to read

videoaggregator: fixing types in aggregate function

Correctly use boolean and GstFlowReturn types in the function.

videoaggregator: use macro to access aggregator src pad

Makes code a bit more readable

aggregator: add a convenience macro to get the source pad

Easier than casting or acessing the parent everywhere

tsdemux: correct fix for dead code

Rename template to caps to keep the original intention of the code after
commit b4c9aa1c

CID #1304674

Revert "tsdemux: remove dead code"

This reverts commit 0635acfec041b1c664bc0770839b1a576e3598b1.

tsdemux: remove dead code

After commit b4c9aa1c308f88bf4e1f69ab0156ed9f99815e8e template will always be
NULL. The if conditional will always be FALSE, so removing it.

CID #1304674

Automatic update of common submodule

From 21ba2e5 to d37af32

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 d676993 to c408583

Back to development

docs: only add the libs that we actually document right now

Keep the libs in alphabetical order unless a specific link order is required.
Also remove the -L directives as we specify the direct path to the .la files.

docs: remove duplicated entry from types file

zbar: split test

Lets not cram everything into a single test - this would render the test name
useless for quick diagnosis. Having separate tests for the optional feature is
also verifying the behaviour when the feature is off.

Release 1.5.1

po: Update translations

hlsdemux: Fix wrong gst-launch command in the description

Fix wrong gst-lauch command in the description.
This patch may help people to get right testing results using the script.

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

dfbvideosink: remove ignored assignments

Remove assignments to DFBResult res that are never read.

tsdemux: remove ignored assignment

Function goes to done before the value set in start_offset is ever used.

Fix a common typo: retreive -> retrieve

Seems to have been copy pasted around a few places

hlsdemux: drop TODO that doesn't need a solution

Connection speed is only checked at that point in hlsdemux so there
is no real need to refactor it.

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

Revert "hlsdemux: Simplify logic in process_manifest"

This reverts commit 4ca3a22b6b33ad8be4383063e76f79c4d346535d.

The connection-speed=0 is used as a special value in the property
of hlsdemux to mean 'automatic' selection, m3u8.c doesn't need
to know about that as it should be as simple as possible.

So this patch hides this automatic selection documented in hlsdemux
into m3u8 logic and I think the gets harder to understand the code.

It also makes the hlsdemux unit tests work again

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

mpdparser: Fix inverted logic introduced in last commit

simplevideomarkdetect: fix detect of videomark partially or fully outside video

In case of the videomark being partially or fully outside,
an error was bein thrown saying, mark width is more than video width.
And when the width, offset properties are set to maximum it resulted in crash.
Instead of throwing error, added logic to detect the mark
in case of partial visibility or dont show the mark when it is outside.

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

mpdparser: Calculate the number of segments we have when a segment template instead of a list is used

Otherwise we would just continue downloading new files forever until we get
404 for the first one, and then error out instead of going EOS.

simplevideomark: Add Error logs

When the pattern offset is outside the video, the print error message

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

Revert "mpdparser: Don't consider streams with a known media presentation duration as live"

This reverts commit 37011e519830e0786fa6e307ed26003a2a8774f1.

This change was actually completely unnecessary, the streams in question are
marked as static and are not considered live anyway.

mpdparser: Don't consider streams with a known media presentation duration as live

adaptivedemux: Error out if downloading a segment failed too often in live mode

It's true that we shouldn't consider errors fatal immediately, but if we
always ignore them we will loop infinitely on live streams with segments
that can't be downloaded at all.

srtp: Allow getting ssrc of more package types than RR and SR

This allows decrypting reduce size packages. See RFC 5506

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

tsdemux: refactor pad creation

Avoid repeating the same pad creation code everywhere

mpdparser: Don't crash in debug output if stream->segments is NULL