Revert "videomixer: When all sinkpads are eos, update output segment stop and forward it"

This reverts commit 2d3910fc7901b5f29e16c0fdd4e9067a6d7f66fe.

It's not solving any problem and instead causes code to fall apart.

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

matroskademux: mark subtitle streams as sparse in stream-start event

And also mark the streams that should be selected by default if
marked so in the headers.

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

audiopanorama: add prebuilt files

audiopanorama: cleanup and expand the tests

Split out two more tests. Extract more common code into helpers. Add coverage for float.

audiopanorama: cleanup of transform()

Only map input if we are reading it. Cleanup the logging and the comments a bit.

audiopanorama: use orc to speedup processing

Use special variants for the case when we don't change the panorama (pan=0.0).
Simplify the processing functions by passing the panorama value directy instead
of the instance. Use orc for clearing buffers too.

videomixer: check last end_time after conversion to running segment

The last end_time was saved after conversion, so the comparison
had to be made after conversion for it to make sense.

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

videomixer: add mix->segment.start to output_end_time

When the segment start is not 0, this created a situation where
the output_end_time is inferior to output_start_time, and the duration
of the next buffer ended up underflowing.

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

matroskademux: Send stream headers after the segment event

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

qtdemux: Do allocation query after exposing all pads and no-more-pads

Also configure video streams as early as possible.

Related https://bugzilla.gnome.org/show_bug.cgi?id=701856
but not fixing that.

flvdemux: Don't forward CAPS events from upstream

Just use the default pad event handler.

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

pulsesink: Cache the getcaps/acceptcaps probe stream

getcaps is called frequently during stream setup, and creating a new
stream each time is very inefficient. There's some more room for
optimisation by caching the queried sink formats as well, but this needs
some more changes to listen for format changes on the sink (for when
supported formats change between probe stream creation and sink
querying).

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

pulsesink: Add a getcaps function

This allows us to have more fine-tuned caps in READY or above. However,
this is _really_ inefficient since we create a new stream and query sink
for every getcaps in READY, which on a simple gst-launch line happens
about 35 times. The next step is to cache getcaps results.

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

pulsesink: Take a lock on the ringbuffer in acceptcaps

This is needed as a concurrent state change could pull the context or
stream out from under our feet.

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

audiopanorama: move the enum to the header and use instead of gint

Move the enum for the processing method to the header so that we can use the
type for the instance struct.

level: rework the tests to cover other formats too

v4l2: make sure the element is not deleted before the pool

The pool accesses data from the v4l2object so it must exist at least
as long as the pool. Refcount the element which controls the object
live-time.

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

png: Link with libgstbase for GstByteReader and GstAdapter

wavenc: Link with libgstbase for GstByteWriter

wavparse: Push stream-start event in pull mode before anything else

pulsesink: Get rid of acceptcaps side-effects

The sink info callback should not have side-effects on the GstPulseSink
object since we are sometimes using with a dummy stream in acceptcaps.

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

Back to development

Release 1.1.1

Update .po files

wavenc: Fix taglist ref handling that made the unit test fail

Automatic update of common submodule

From 098c0d7 to 01a7a46

v4l2: iterate controls with V4L2_CTRL_FLAG_NEXT_CTRL if possible

In v2.6.18 control classes where added to the v4l2 API.
Iterating over CIDs starting with V4L2_CID_BASE will only find controls for
the first control class.
By iterating with V4L2_CTRL_FLAG_NEXT_CTRL all controls are found.

This is necessary to make controls from other control classes available in
the extra-controls property.

If V4L2_CTRL_FLAG_NEXT_CTRL is not defined at compile time or not supported
at runtime then the old mechanism for iterating is used.

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

udpsink: avoid leaking the host

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

v4l2: improve pixel aspect ratio handling

Instead of just assuming a aspect ratio of 1/1 use VIDIOC_CROPCAP to ask
the device.
This also add a pixel-aspect-ratio property to overwrite the value from the
driver and a force-aspect-ratio property to ignore it.

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

v4l2: Fix compilation with older kernels

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

v4l2: call VIDIOC_REQBUFS with count = 0 in pool_finalize

Without this the following sequence fails:

- set_caps()
  - object_stop() (does nothing)
  - set_format() -> VIDIOC_S_FMT
- set_config() -> VIDIOC_REQBUFS with count = N
- set_caps()
  - object_stop()
    - pool_finalize()
  - set_format() -> VIDIOC_S_FMT => EBUSY

Usually the pool is started after set_config(), in which case object_stop()
will result in a pool_stop and therefore VIDIOC_REQBUFS with count = 0 but
that is not guaranteed.
Also calling VIDIOC_REQBUFS with count = 0 in pool_finalize() if necessary
fixes this problem.

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

v4l2: rework sink buffer refcounting

This is a followup patch for #700781, which is not quite correct.
The buffer handling is quite complicated here.
The original code intended to the the following:

- gst_v4l2_buffer_pool_process() calls QBUF and adds the buffer to the
  local list.
- The sink calls gst_buffer_unref() which returns the buffer to the pool
  but not the 'free list'.
- Some time later DQBUF returns the buffer and
  gst_v4l2_buffer_pool_release_buffer() puts in on the 'free list'.

If the buffer must be copied then (parent_class)->acquire_buffer() is
called directly to keep the buffer in the pool.

This has two problems:
1. If gst_v4l2_buffer_pool_release_buffer() is called before the buffer is
   returned to the pool, then the buffer is put on the 'free list' twice.
   This can happen if a reference to the buffer is kept outside the sink,
   of if DQBUF returns the buffer, that was just queued with QBUF.
2. If buffers are copied, then all buffers are in the pool at all times. As
   a result gst_v4l2_buffer_pool_stop() and gst_v4l2_buffer_pool_dqbuf()
   can access pool->buffers at the same time, which can lead to memory
   corruption.

The patch for #700781 fixes those problems, but with the side effect that
there are always buffers outside the pool (because they are queued) and
the pool is never stopped.
This patch fixes this by releasing the reference to the buffer after
handling it (to avoid problem 2.) so it can be returned to the pool.
gst_v4l2_buffer_pool_release_buffer() is only called if the buffer is
already in the pool (to avoid problem 1.).

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

qtdemux: make sure taglist is writable before adding tags

Avoids assertions

qtdemux: effectively skip tracks that weren't listed on the 1st moov

Without this, stream is NULL and the code will try to access it, leading
to segfaults.

qtdemux: skip redundant check

!got_moov is already checked the line above

tests: cleanup level tests

Split out a few more tests to avoid checking the same stuff over and over again.

level: remove unused variables in instance struct

level: add a test for continous timestamps

A test that checks that msg[n].ts + msg[n].dur == msg[n+1].ts.

wavenc: add tags & toc support

Write tags as LIST INFO chunk. Format the toc as cue + LIST adtl chunk. Remove
old #ifdef'ed code.

Revert "rtph264pay: Restructuring to allow for adding optional caps"

This reverts commit 61666898cfe89a1b21d3e6850ab44f5b1633ed79.

This commit changes what the set_sps_pps() function does, not it doesn't
set caps anymore (and should have been renamed). The main problem is that
not all call sites are updated and thus leak the string.

Revert "rtph264pay/depay: Add frame dimensions a payloaded caps"

This reverts commit 3dca756a5dba55266256f239e3e12a3d058e185a.

The H264 RTP spec has no attributes for width and height.

Revert "rtph264pay/depay: Add optional framerate caps for use in SDP"

This reverts commit d8825e2a5c0bfb883ff88e2c9da499c800ebca0a.

There is no framerate attribute in the h264 RTP spec.

Revert "rtpjpegpay/depay: Replace framesize caps with width/height"

This reverts commit 0075d111b475ca27895ee9476154260b6902940b.

Extra application/x-rtp are SDP fields, which are strings.

Revert "rtpjpegpay/depay: Replace framerate caps field with fraction"

This reverts commit 9fd25a810b859e0ec205176578735100d83de4af.

We deal with sdp attributes in application/sdp, which are always strings.

rtspsrc: add extra TLS url protocols

We also support TLS protocols now.

videomixer: Add FIXME comment about the DURATION query from adder

Currently the code just takes with maximum upstream duration, which
is wrong. It should be the maximum upstream duration in running time.

videomixer: Set a reference to mix->current_caps as the QUERY_CAPS result.

level: misc cleanups

Fix some oudated comments. Sort out some confusion of interval_frames and num_frames.

v4l2: Only conditionally use V4L2_CTRL_TYPE_INTEGER_MENU, it's not available in older versions

v4l2: add a property for arbitrary v4l2 controls

This makes it possible to set any controls that can be set with
VIDIOC_S_CTRL.
The controls are set when the property is set (if the device is open)
and when the device is opened.

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

level: fix discontinuities in timestamps

gdkpixbufdec: Keep serialized events in order, and don't send SEGMENT before CAPS

rtspsrc: create and push stream-start in TCP mode

rtspsrc: remove some obsolete code

It is not needed to do a state change from the _play() function on
ourselves. The state change function already did that and we don't want to
interfere with that (or use hacks to avoid interference).

rtspsrc: set RTCP caps on the RTCP pads

rtpsession: send stream-start and segment events

Also send stream-start and segment event on the RTCP pad.
We don't need to send anything on the sync_src pad because we
already forwarded all incomming events.

rtspsrc: add signal to handle server requests

Add a signal to be notified of a server request. The signal handler can then
construct the response message for the server.

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

videomixer: Maintain z-order when new pad are added

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

vp8enc: Add property to manually specify the timebase of the encoder

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

videomixer: Always handle flush_stop_pending atomically

It is not protected with the COLLECT_PADS_STREAM_LOCK anymore

tests: videomixer: Add a testsuite for videomixer

This is mostly copy pasted from -base/tests/check/elements/adder.c

videomixer: Do not take COLLECT_PADS_STREAM_LOCK when unnecessary

Collectpad takes the lock itself when receiving serialized events
and we should not take it for not serialized ones

flxdec: Properly skip non-frame chunks

flxdec: Flush data from adapter after reading it

Otherwise we're going in an infinite loop, reading the same data
over and over again.

goom2k1: fix more duplicated symbols

rtpjpegpay/depay: Replace framerate caps field with fraction

The previous implementation had the formatting of SDP attributes happen
in each RTP payloader, now instead the constituent values are propagated
as caps fields. This allows for applications to do SDP offer/answer
based on caps negotiation.

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

rtpjpegpay/depay: Replace framesize caps with width/height

The previous implementation had the formatting of SDP attributes happen
in each RTP payloader, now instead the constituent values are propagated
as caps fields. This allows for applications to do SDP offer/answer
based on caps negotiation.

Keep parsing a-framerate, x-framerate and x-dimensions in rtpjpegdepay
to be backwards compatible with previous payloaders.

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

rtph264pay/depay: Add optional framerate caps for use in SDP

This allows for applications to format SDP attributes and still do SDP
offer/answer based on caps negotiation.

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

rtph264pay/depay: Add frame dimensions a payloaded caps

This allows for applications to format SDP attributes and still do SDP
offer/answer based on caps negotiation.

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

rtph264pay: Restructuring to allow for adding optional caps

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

(dyn|multi)udpsink: Add properties to specify the bind address and port

By default we use the any addresses and a random port for binding the socket.

(dyn|multi)udpsink: Bind socket before using it

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

(multi)udpsink: Add missing getters for socket-v6 and used-socket-v6 properties

videomixer: Don't hold stream-lock while pushing non-serialized events

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

videomixer: Don't hold object lock while sending events

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

deinterlace: The return value of gst_pad_set_caps() is not relevant anymore

Caps can fail to be set because the pad is not linked yet for example.

qtdemux: Add error if file has playready drm

videomixer: Send a reconfigure event upstream if sinkpad caps are not usable

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

v4l2: keep a reference to all queued buffers

Without this, a queued buffer may be required, filled and queued before it
is dequeued.
Calling gst_buffer_pool_acquire_buffer() ensures that the buffer is set up
correctly and gst_buffer_unref() calls buffer_release().

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

mulawdec: Handle NULL buffers in handle_frame

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

rtpjpegpay/depay: Add framesize caps for use in SDP

The format of the value adheres to RFC6064 and it is meant to be parsed
and included in the SDP sent by gst-rtsp-server to its clients.

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

rtpjpegpay: Add optional framerate caps for use in SDP

The format of the value adheres to RFC4566 and it is meant to be parsed
and included in the SDP sent by gst-rtsp-server to its clients.

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

videomixer: When all sinkpads are eos, update output segment stop and forward it

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

videomixer: Don't reset the output segment on flush stop

Only init it when getting from READY to PAUSED, and change it on seek events.

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

v4l2: Don't stop streaming when set_caps is called with unchanged caps

This can happen if other parts of the pipeline are reconfigured.
Stop streaming even for a short amount of time can be quite visible, so it
should be avoided if possible.

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

tests: Re-enable videomixer test

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

videomixer: Send caps event from the streaming thread

This way we avoid races in caps negotiation and we make sure
that the caps are sent after stream-start.

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

videomixer: Do not send flush_stop when receiving a seek

There is no reason to send a flush-stop when receiving a seek event.
In the case of a flushing seek, we could eventually want to, but in
the code path were we check if the seek is "flushing", we have the
following comment that makes sense:

"we can't send FLUSH_STOP here since upstream could start pushing data
after we unlock mix->collect.
We set flush_stop_pending to TRUE instead and send FLUSH_STOP after
forwarding the seek upstream or from gst_videomixer_collected,
whichever happens first."

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

videomixer2: Protect flush_stop_pending with the collectpad stream lock

And make sure to expect a flush-stop after a flush-start

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

rtpmp4apay: clear config buffer before using it

This is necessary because parts of the memory are only modified with "|="

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

qtdemux: Do not expect EOS after a segment event if upstream is mss

In case qtdemux is handling a mss stream, do not mark the stream to wait
for EOS after a segment. Even if it seems to be the last one according to
the current streams information.

MSS handling is different here because there is another demuxer driving
the pipeline

qtdemux: only set channels and rate if qtdemux knows it

Setting both of those to 0 is pointless and means that qtdemux
doesn't know the real value. Avoid setting it in this case.

qtdemux: set alac caps using info from codec buffer

The samplerate field in the STSD atom is not right for some ALAC files
(usually when audio is 96kHz/24bits), so the audio caps must be
extracted from the codec data.

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

avidemux: do not push discont buffers if they aren't discont

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

Automatic update of common submodule

From 5edcd85 to 098c0d7

videocrop: Add support for GRAY16_LE/GRAY16_BE

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

rgvolume: Send all events through the proxypads instead of just sending to the target

Otherwise the sticky events are missing on the proxypads.

rgvolume: Fix event handling in the unit test

rglimiter: Fix event handling in unit tests

rganalysis: Fix event handling in unit test