matroskamux: Fix constantly growing used uid list

Moves the used uid list to the class to avoid having it grow forever.

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

Automatic update of common submodule

From d48bed3 to 1a07da9

wavparse: Ignore Broadcast Wave Format (BWF) tags when searching for 'fmt' chunk

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

check: Use fakesink sync=True instead of an audio sink

Ensures the test can run on systems without alsa (or any audio output for
that matter), and will avoid people running build slaves wondering what
the hell was beeping during the night :)

ac3parse: custom get_sink_caps handling for private stream caps

... now that those are transformed rather than parsed, some transforming
of caps is required as well to make auto-plugging succeed.

v4l2: guard use of ENODATA with #ifdef

Not all systems with v4l have ENODATA defined, so check that we have it
before attempting to use it.

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

Revert "rtspsrc: Proxy rtpjitterbuffer do-retransmission property"

This reverts commit 9f7b1128b1f00a2b87a232ff890867549ab95ba5.

This should be handled automatically be rtspsrc if the AVPF profile
is used, and manual enabling of it can be done with the new-manager
signal.

rtspsrc: add signal to notify of new manager

So that you can configure and connect to signals on the rtpbin.

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

rtspsrc: Proxy rtpjitterbuffer do-retransmission property

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

rtpjitterbuffer: handle expected packet being an RTX packet

If the expected packet (do_next_seqnum is TRUE) is the one we requested
for retranmission earlier, do the logic to update the retransmission
statistics as well before setting up the timers for the next expected
packet.
Also reset the retransmission counter if the timer is reused for another
seqnum.

rtpbin: add a caps accumulator for the request-pt-map signal

Add an accumulator that stops the signal emission as soon as a caps has
been retrieved. Otherwise the default handler would continue emitting
the signal and possibly overwrite the result with NULL again.

rtxreceive: copy flags and timestamps from original buffer

rtpjitterbuffer: ignore invalid timestamps in rtt calculation

When the input buffer does not have a valid timestamp, don't try to
calculate the round-trip-time.

matroskaparse: better default caps when none set

Uses information gathered during EBML parsing to
forge a more suitable set of caps instead of blindly
assuming everything is video/x-matroska.

For consistency, stream type reset was added to
matroska-demux too.

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

tests: rtprtx::test_rtxreceive_data_reconstruction: remove useless code for triggering retransmission

There is no need anymore to push yet another buffer in rtxsend
in order to trigger the previously requested retransmissions
to actually happen.

tests: rtprtx::test_rtxreceive_data_reconstruction: fix race condition

Now with rtprtxsend pushing rtx buffers from a different thread,
this is necessary to ensure that the result of the test is deterministic.

This code makes use of GstCheck's global GMutex and GCond that are
being used inside GstCheck's sink pad chain() function in order
to synchronize with it.

tests: rtprtx::test_rtxsender_packet_retention: fix race condition

Now with rtprtxsend pushing rtx buffers from a different thread,
this is necessary to ensure that the result of the test is deterministic.

This code makes use of GstCheck's global GMutex and GCond that are
being used inside GstCheck's sink pad chain() function in order
to synchronize with it.

tests: rtprtx::test_push_forward_seq: fix race condition

Now with rtprtxsend pushing rtx buffers from a different thread,
this is necessary to ensure that the result of the test is deterministic.

This code makes use of GstCheck's global GMutex and GCond that are
being used inside GstCheck's sink pad chain() function in order
to synchronize with it.

tests: rtprtx::test_push_forward_seq: fix buffer refcounting

rtprtxsend: ensure that no rtx buffers are sent after EOS

To do that, enqueue the EOS event to be sent from the srcpad task
thread and flush the queue right afterwards, so that no more rtx
buffers can be sent, even if there are more requests coming in.

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

rtprtxsend: run a new GstTask on the src pad

The reason behind this is to minimize the retransmission delay.
Previously, when a NACK was received, rtprtxsend would put a
retransmission packet in a queue and it would send it from chain(),
i.e. only after a new buffer would arrive.

This unfortunately was causing big delays, in the order of 60-100 ms,
which can be critical for the receiver side.

By having a separate GstTask for pushing buffers out of rtxsend,
we can push buffers out right after receiving the event, without
waiting for chain() to get called.

shout2send: error out if no caps were received

Instead of assuming that input is ogg.

shout2send: accept audio/webm, audio/ogg and video/ogg as well

Those are advertised in the template caps, but the
setcaps handler didn't handle them. But then oggmux
and oggparse at least for now still always output
application/ogg anyway, so that wasn't a real problem.

rtpvp8pay: Don't leak input buffers

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

avimux: reset some more audio pad data when needed

avimux: write correct blockalign for vbr audio

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

rtpjitterbuffer: do not drop serialized events when latency is set

Serialized events are now queued in the jitter buffer, so we don't
want to drop them even latency is set.

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

avimux: don't make the buffer writable unless absolutely necessary

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

v4l2: set GST_BUFFER_FLAG_DELTA_UNIT when appropriate

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

v4l2bufferpool: don't ref the newly created allocator

Otherwise the allocator will never be deleted.

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

matroskademux: Don't skip all video frames until the first keyframe

Instead do it like all other demuxers and let parsers and decoders
handle that. The keyframe information inside the container might
be completely wrong like in the sample file of the bug report,
and if it is correct and we push no keyframes, then the parsers
and decoders will handle that properly anyway.

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

qtdemux: remove elst_offset variables

They are not used anymore

qtdemux: remember reverse playback when verifying the segment end

Check if the rate is positive or negative to correctly compare the current
position with the segment to make reverse playback work

qtdemux: do not ignore empty segments

Make sure empty segments are used and pushed with a gap event
to represent its data (or lack of it)

Each QtSegment is mapped into a GstSegment with the corresponding
media range. For empty QtSegments a gap event is pushed instead
of GstBuffers and it advances to the next QtSegment.

To make this work with seeks, need to keep track of the starting
'base' to make sure it remains consistently increasing when
pushing new segment events.
For example: if a seek makes qtdemux start from 5s, the first
segment will have a base=0. When the next segment is activated,
its base time will be QtSegment.time - qtdemux.segment_base so
that it doesn't include the first 5s that weren't played and
shouldn't be accounted on the running time

This purposedly will remove the fix made for
https://bugzilla.gnome.org/show_bug.cgi?id=700264, at this
point it was decided to respect the gaps, even if they cause
a delay on playback, because that's the way the file was crafted.

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

tests: Remove usage of the system clock from the rtprtx test

tests: Initial segment in rtpcollision test

examples/*-rtpaux: specify payload type association for the audio stream, so that rtx works also for audio

rtprtxsend: remove wrong check for payload type not having been set

1) pt can be lower than 96
2) there is no point in checking that because rtprtxsend will not
   even store buffers for payload types that it doesn't know about,
   so this case will never be reached

rtprtxsend: fix data locking when creating rtx packets

This patch moves the creation of rtx packets to be done early,
in the src_event() function, when they are requested. The purpose
is to run gst_rtp_rtx_buffer_new() with the object locked to
protect internal data, because if it is done at the pushing stage,
we would have to lock and unlock multiple times in a row while we
are pushing the rtx buffers.

Previously there was no locking at all, which was terribly wrong.

rtprtxsend: lock access to internal data in sink_event() function

rtprtxsend: remove unnecessary call to reset() from finalize()

...and use _free_full() on the pending buffers queue now that
reset() is not being called

rtprtxsend: remove unused parameter from the internal reset() method

rtprtxsend: Use g_slice_* for allocating internal structures

rtprtxreceive: remove stupid mutex unlock in the middle of chain()

rtprtxreceive: use GST_DEBUG_OBJECT / GST_WARNING_OBJECT instead of GST_DEBUG / g_warning

rtprtxreceive: fix integer format specifiers in GST_DEBUG

seqnum in this function is 32-bit, so G_GUINT16_FORMAT would
produce undefined output on big endian systems

rtprtxsend: change the rtx_pt_map directly in set_property() instead of delaying it for chain()

The same lock is held, so there is no point in complicating it...

rtprtxreceive: change the rtx_pt_map directly in set_property() instead of delaying it for chain()

The same lock is held, so there is no point in complicating it...

rtprtxreceive: simplify the code of finalize()

rtprtxreceive: use the GstObject lock instead of a new one

rtprtxsend: use the GstObject lock instead of a new one

v4l2: Add NV12_64Z32 support

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

osshelper: Don't leak fd when getting card name

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

Revert "aacparse: relax the detection of ADTS"

This was pushed by mistake along with the V4L2 fix.

This reverts commit 8eb4b032bef444397c4d211f2095c173ba114187.

rtpg726pay: don't leak encoding_name string

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

v4l2: fix build break using V4L2_CAP_VIDEO_M2M_MPLANE

This may not be defined. Since the previous version used
only the other define (V4L2_CAP_VIDEO_OUTPUT_MPLANE), fall
back on this only when not available.

aacparse: relax the detection of ADTS

According to ISO/IEC 13818-7, "channel_config" field in ADTS header
may have value of 0, as in the case of frame with PCE.
gst_aac_parse_detect_streams() returned FALSE for those frames
and discarded them.

v4l2bufferpool: check set_config return value in gst_v4l2_buffer_pool_new

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

v4l2object: Add parsed=1 field for encoded output

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

v4l2object: Don't leak empty caps

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

v4l2bufferpool: do not stop a stream not previously started

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

v4l2object: Don't enforce dimension field on encoded formats

Don't enforce having width, height and framerate in template caps for encoded
formats. These don't always need to be exposed and may break negotiation for
decoder and decoding sink. If needed, these field will be automatically added
when probed caps are known.

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

v4l2object: unref downstream pool

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

v4l2bufferpool: add gst_v4l2_buffer_pool_flush

STREAMOFF set all v4l2buffers to DEQUEUE state.

Then for CAPTURE we call QBUF on each buffer.
For OUTPUT the buffers are just push back in the GstBufferPool
base class 's queue.
But the loop actually looks like the same.

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

v4l2object: Add vp8 support

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

v4l2object: Don't force framerate field for OUTPUT

If there is nothing that seems to force a certain framerate on output device, it is
preferable to simply not set that feild. This allow negotiation with tsdemux in a
decoder for example.

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

v4l2object: _v4l2fourcc_to_structure() can be static

This function is not used anymore outside v4l2object.

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

v4l2object: Add MPEG1/2 support

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

v4l2object: Ask for a decent buffer size when dealing with encoded formats

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

v4l2bufferpool: On warn on size change if n_planes > 1

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

v4l2object: check if translated format is valid

Also add a FIXME in gst_v4l2_object_setup_format
to note that the whole function has to be improved
in order to support ENCODED formats.
It requires to have an encoder device which we do not
have right now.

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

v4l2object: Validate returned dimensions

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

v4l2object: Ensure max is not smaller then min in decide_allocation

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

v4l2object: Don't keep the max paramter when using our own pool

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

v4l2bufferpool: Respect the suggested min buffer

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

v4l2object: Allocate pool if needed in decide_allocation

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

v4l2object: Add V4L2_CID_MIN_BUFFERS_FOR_CAPTURE support

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

v4l2: Move decide allocation into v4l2object

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

v4l2object: Implement _setup_format()

This method allow setting up the object from the currently configured format on the
device. This is useful for M2M element where input data decides the format that will
be set on capture side.

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

v4l2object: Split out saving format from set_format()

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

v4l2object: set only one plane for encoded format

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

v4l2object: Move code block where it belongs

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

v4l2object: Don't check format specific information

The number of plane, and the stride does not represent a capability change. Same caps
can have different stride from the default GstVideoInfo and the number of planes will
never change for 1 format.

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

v4l2object: Move the extrapolation of stride at the right place

Now that we have a stride array, we should extrapolate only when
eeded (non multi-planar buffer).

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

v4l2object: Move back assertions where they should be

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

v4l2object: Move mplane logic into gst_v4l2_object_get_caps_info()

It makes the gst_v4l2_object_set_format() slightly simplier and will make that
logic reusable. Note that gst_v4l2_object_has_mplane() will always return the
same value for one device. There is no need to check against the caps as this
has already been done by _open.

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

v4l2object: Split _v4l2fourcc_to_video_format

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

v4l2bufferpool: Request buffers only once

VIDIOC_REQBUFS allocates buffer, it has no place inside set_config. Also, some driver do
no allow multiple calls to this ioctl.

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

v4l2object: Don't validate dimension for encoded format

We set the dimensions just in case but don't validate them
afterwards. For some codecs the dimensions are *not* in the
bitstream, IIRC VC1 in ASF mode for example.

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

v4l2object: Quirks for dev without initial format

Most M2M have undefined behaviour initially when VIDIOC_G_FMT is called.

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

v4l2object: Add gst_v4l2_object_open_shared()

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

v4l2object: Implement gst_v4l2_dup()

This will duplicated the FD from another object and copy over the probed result.

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

v4l2object: make IO_MODE enum public

This is to allow adding a second io-mode property on M2M device like decoder so
input and output can be controlled separatly.

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

v4l2: better handle quirks activation

This way we can activate deactivate those quirks all at once at one
place.

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

v4l2: Fix h264 caps

V4L2_PIX_FMT_H264 is documentated as byte-stream (with start code). The ensure proper
negotiation with element like h264parse.

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

v4l2object: Split caps in different categories

This is need to correctly expose capabilities on specialized devices
like decoders and encoders.

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

matroskademux: don't leak TOC chapter list

matroskamux: remove obsolete write-dummy-and-overwrite-on-eos code

The need for rewriting apparently is obsolete 0.10 leftover.
We now have caps for subtitles when we create the headers,
so we always write the correct data in the first place.

rtprtxsend: remove duplicate assignment

Coverity CID 1151680

matroskamux: write subtitle codec ID and data at start when known

This avoids issues with writing dummy data first, then having
to come back and write correct data later. Doing so prevents
the muxed stream from being actually streamable.

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