theoradec: Clean up handling of reset/flushing/start/stop

videoencoder: There's no point in resetting the encoder when the caps change

The subclass will be called with set_format() and there it can drain
if necessary and reset whatever is necessary. This is the same behaviour
as for the video decoder.

videoencoder: Reset internal state and segments on FLUSH_STOP

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

videoencoder: Refactor GstVideoEncoder::reset() handling a bit

Let gst_video_encoder_reset() call it as would be intuitive and
only call it indirectly from gst_video_encoder_drain(). Now it
actually makes sense.

videodecoder: Refactor GstVideoDecoder::reset() handling a bit

Let gst_video_decoder_reset() call it as would be intuitive and
only call it indirectly from gst_video_decoder_flush(). Now it
actually makes sense.

videodecoder: Take DTS as PTS for keyframes as a last resort if we can't calculate any PTS

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

playsink: Remove two unused variables

subtitleoverlay: Remove some more unneeded segment tracking

playsinkconvertbin: Remove obsolete segment tracking

This is now all handled automatically with sticky events.

playbin: Ensure that everything in a GstSourceGroup gets the same group-id

Including streams from external subtitle files. If not everything implements
the group-ids playbin invents its own.

playsink: Fix handling of colorbalance element if the sink does not implement it

playsink: Improve segment handling in the custom flushing for subtitle stream switches

This code needs a lot more work to be improved for 1.0.

subtitleoverlay: Segment events are sticky now and propagated automatically when needed

streamcombiner: Proxy all sink events downstream

Thanks to Mathieu Duponchelle for noticing this regression
introduced with the last change.

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

tagdemux: Add support for group-id in the stream-start event

ogg: Add support for group-id in the stream-start event

streamsynchronizer: Implement grouping of streams via the group-id

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

streamcombiner: Fix locking

We have to hold the streams-lock when iterating over all pads,
also the stream-lock of the pad is already locked when we receive
EOS.

Call gst_pad_event_default() for the correct default handling of
events.

encoding: fix EOS handling in streamsplitter / combiner.

This commit adds a streamcombinerpad with an is_eos field.
When streamcombiner receives an EOS on one of its pads, it
forwards it all its other pads are EOS.
This commit also removes the notion of "stream-switching-eos".

video-info: respect stride alignment

Increase the left padding so that we don't cause stride alignments later when we
apply the padding.

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

Revert "video: respect stride alignment when calculating planes offsets"

This reverts commit 28e1dadbfaa403679e69f8173d1aa2c7500fd556.

Incrementing the offset to make the plane aligned causes the image to be
incompatible with what Xv expects. Rather that forcing a memcpy in the
xvimagesink we would like to do adjust the left padding instead.

video: respect stride alignment when calculating planes offsets

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

id3: Use debug category and show FIXMEs

Allows spotting faster un-parsed tags

videometa: Add docs to the region of interest meta functions

subparse: use g_strdup() and friends

Fixes build issue on windows, but is also better seeing that
these string are going to get freed with g_free() and not free().

tagdemux: Put the modified time back in the time part of the segment

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

tests: fix videoscale test after video format addition

configure: remove obsolete libxml checks

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

subparse: don't leak parse context for sami and qttext

In gst_sub_parse_dispose() parser_type will be UNKNOWN,
so these deinit calls were never executed. And we should
clean up the parser state in the downwards state change
anyway.

tests: update sami parser testcases

Remove libxml dependency for sami parser
and add more testcases.

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

subparse: remove libxml dependency for sami parser and re-enable sami parser

To celebrate 2013.gnome.asia, updated sami parser for gstreamer 1.x. :D

Remove conditional block for check libxml usage and
implement a simple html markup parser for the sami
parser.

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

meta: fix ROI meta getter

playbin: Don't prefer decoders for which we found a matching sink

It doesn't make much sense.

video: Add support for NV24 color format

This is semi-planar 4:4:4 YUV.

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

playbin: Also consider possible converters for raw streams when selecting compatible sink/source combinations

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

videometa: Add to the docs and make function names more consistent with others

videometa: Add Region Of Interest meta

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

playbin: Fix sorting for decoder factories for which we didn't find a compatible sink

They might just need some converters for raw audio/video.

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

riff-media: Add 'png ' fourcc

On top of mpng, MPNG, PNG, there is also png it seems

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

allocators: dmabuf: allow testing allocator type

In decide_allocation function some element may when to test the proposed allocator.
For example like this:
if (gst_query_get_n_allocation_params (query) > 0) {
GstAllocator * allocator;
GstAllocationParams params;
gst_query_parse_nth_allocation_param (query, 0, &allocator, &params);
if (g_strcmp0(allocator->mem_type, GST_ALLOCATOR_DMABUF) == 0)
GST_DEBUG("got dmabuf allocator");
else
GST_DEBUG("got an other allocator");
}

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

oggdemux: Make bisecting fully accurate

When bisecting after an earliest time has been found, we need
to only consider the stream for which the earliest time was found.

Before, the following scenario could be and was encountered:

a) Find the earliest time for stream X
b) bisect and find a page which granuletime is indeed < target, but
   contains another stream.
c) decide to seek at the wrong offset, sometimes inferior to
   the real one, in which case the error was undected or
d) the offset was superior, and thus the actual target keyframe was
   not processed, and packets were skipped waiting
   for a granulepos.

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

Revert "oggdemux: fix seeking with negative rate with skeleton"

This reverts commit b41cd0428956f3ade9b428149e38be8e788556fe.

playbin: Don't print a warning when setting a sink to NULL

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

rtspconnection: Create a new write GSource after removing it

After removal, a GSource is destroyed and can never be attached
again to a main context. We need to create a new one instead.

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

playbin: Properly destroy and set to NULL sinks that don't work

playbin: Fix logic to detect if a stream-change is currently pending

Fixes duration reporting in gapless playback between files.

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

videometa: fix header formatting

riff: Provide correct media type for XSub

Xsub (fourcc DXSB) is a subpicture stream used for embeded
subtitles on divx files. This provides a correct media type
for them instead of just video/x-avi-unknown.

Back to development

Release 1.1.2

Update .po files

playbin: Only give sinks a new bus if they have no parent yet

Otherwise we will remove the bus that would proxy messages to playsink
and never set it again. If the sink is already in playsink, all failures
are fatal anyway as it's either a sink that worked before or one that
was set by the user.

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

playbin: Store a/v/t sinks locally too, not just in playsink

playsink: ref_sink() any sinks that are set on playsink

Otherwise the behaviour of the properties is inconsistent.

playbin: Fix assumptions in the unit test

Unused sinks are still set to READY now during autoplugging
to check their caps. Also playsink owns a ref to the sinks too.

streamsynchronizer: Non-TIME segment streams are not waiting automatically

This was leftover code from porting to 1.0 and fixes the playbin
unit test.

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

win32: add missing rtp buffer methods

playbin: Change sink ownership handling to be a bit more sane

playbin will now only activate the sinks in a single place and
will never change the states of any sinks that are owned by
playsink.

Also handle text-sinks the same way as audio/video sinks inside
playbin.

po: update POTFILES.in

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

colorbalance: Fix the typo in base_init().

adder: Do not send flush_start event with the stream lock taken

FLUSH_START is not serialized, so the lock should not be taken when
sending it.

tag: ignore malformed ID3v2 TDAT frames

Just skip them, don't cause criticals.

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

audioresample: make explicit that neon is disabled and why

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

audioresample: disable 16-bit integer NEON support

it seems to be broken (produces no audio), plus the performance gain
is small

Signed-off-by: Carlos Rafael Giani <dv@pseudoterminal.org>

playbin: If we had a previous autoplugged sink, try to reuse it

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

playsink: If we switch sinks, make sure that the old sink is set to NULL

playbin: Don't change the state of sinks that we passed to playsink already

playsink: Consider new audio/video sinks when reconfiguring

playbin: Improve debug output regarding sink selection

playbin: Post an error message if a stream combiner doesn't return a request pad.

playbin: Only intersect to check if a sink can handle raw caps

Doing a subset check requires fixed caps, which we might not have here.

https://bugs.webkit.org/show_bug.cgi?id=116042

pbutils: allow describing unfixed caps if they share the same media type

Caps description and missing plugin code does not really need caps to
be fixed, and indeed they may not be if giving encodebin unfixed caps
that correspond to an unknown encoder or muxer.

So we relax the check, and allow unfixed caps if all the structures
refer to the same media type.

videodecoder: Send all pending events with type < CAPS before sending caps

videoencoder: Send all pending events with type < CAPS before sending caps.

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

typefind: avoid too low mpeg/ts probability on small amount of data

With the current test, we get into problems when we try to typefind
a MPEG stream from a small amount of data, which can happen when
we get data pushed from a HTTP source. We thus make a second test
to give higher probability if all the potential headers were either
pack or pes headers (ie, no potential header was unrecognized).

This fixes an issue with a MPEG1/MP2 stream being properly discovered
as video/mpeg from a file, but as audio/mpeg from souphttpsrc.

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

video(enc|dec)oder: Don't return not-negotiated if flushing

If the pad is flushing after a failed negotiation, return
GST_FLOW_FLUSHING instead from finish_frame().

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

audioencoder: Don't return not-negotiated if flushing

If the pad is flushing after a failed negotiation, return
GST_FLOW_FLUSHING instead from finish_frame().

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

pbutils: descriptions: Allow smart codec tag handling

We already have internally the information on what type of stream (audio,
video, container, subtitle, ...) a certain caps is.
Instead of forcing callers to specify which CODEC_TAG category a certain
caps is, use that information to make a smart choice.

Does not break previous behaviour of gst_pb_utils_add_codec_description_to_tag_list
(if tag is specified it will be used, if caps is invalid it will be rejected,
...).

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

xmptag: Add a debug category

Instead of using the default category

videotestsrc: do not leak lines

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

rtpbasepayload: Do not leak the event when segment is delayed

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

rtsp: make read uncancelable when reading a message

When we start to read a message, we need to continue reading until the end of
the message or else we lose track and cause parse errors. Use a variable
may_cancel to avoid cancelation after we read the first byte until we have
the complete message.

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

audiodecoder: Don't return not-negotiated if flushing

If the pad is flushing after a failed negotiation, return GST_FLOW_FLUSHING.

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

ogg: The Daala headers are little endian, not big endian

ogg: Add Daala support

pbutils: Add VP9 description

videodecoder: Fix drop frame handling at startup

In the unlikely case that the decoder drops a frame before the first
input frame is outputted, use the input segment (since it wasn't
carried over to the output segment yet)

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

rtsp: dispatch when initial buffer has data

When we have data in the inital buffer, dispath the read function to read it
even if the socket has no data to read.

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

rtsp: manage writer child source better

Only add the write child source when we have something to write or else
we will dispatch forever without doing anything.

audioencoder: unref before memset

Unref allocator and input_caps in encoder context before memsetting the
context.

xmptag: More efficient GSList usage

Instead of constantly appending (which gets more and more expensive), just
prepend to the list (O(1)) and reverse the list before usage.

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

rtpbuffer: add gst_rtp_buffer_get_payload_bytes

The function gst_rtp_buffer_get_payload can not be used in Python
because it lacks necessary length parameter. This patch adds a new
function, gst_rtp_buffer_get_payload_bytes, to use from Python
bindings. The new function has the advisory "Rename to:" annotation
so it can replace the gst_rtp_buffer_get_payload whan creating
bindings.

The function gst_rtp_buffer_get_extension_bytes is also added. It wraps
gst_rtp_buffer_get_extension_data which doesn't work in Python due to
incomplete annotation and because it returns the length as number of
32-bit words.

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

audiobasesrc: add 2 missing gst_buffer_unmap () calls

There are 2 missing calls to gst_buffer_unmap () in the error handling in
create ().

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

playsink: Fix the block diagram of deinterlace bin.

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

playbin: Emit {audio,text,video}-changed signals when pads are removed

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

videoconvert: Fix leaking of the chroma resample helper objects

tests: add more unit test for playbin

Add unit test for autoplugging of video_decoder/video_sink combination
based on capsfeatures.

rtspconnection: Make sure to set a sensible default port for the GSocketConnection

Otherwise it will connect to port 0 if no port is given in the URI.

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

adder: Reject segments that have a different rate than the output segment

adder does no rate conversion.

playbin: When activating a fixed sink, proxy error messages too

If activating a fixed sink fails, everything will fail later anyway
and we can just error out early.