baseparse: restrict duration scanning to pull mode and avoid extra set_caps call

baseparse: update some documentation

Also add some more debug.

baseparse: allow increasing min_size for current frame parsing only

Also check that subclass actually either directs to skip bytes or
increases expected frame size to avoid going nowhere in bogus
indefinite looping.

baesparse: fix refactor regression in loop based parsing

baseparse: pass all available data to subclass rather than minimum

Also reduce some adapter calls and add a few debug statements.

baseparse: fix reverse playback handling

baseparse: minor typo and debug statement cleanup

baseparse: reduce locking

... which is either already mute and/or implicitly handled by STREAM_LOCK.

baseparse: avoid loop in frame locating interpolation

audioparsers: baseparse: Be careful to not lose the event ref

Don't unref the event if it hasn't been handled, because the caller
assumes it is still valid and might reuse it.

I ran into this problem when transcoding an AVI (with mp3 inside)
to gpp.

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

docs: minor baseparse docs/comment fixes

Remove copy'n'paste leftovers.

baseparse: increase keyframe awareness

... which is not particular relevant for audio parsing, but more so
in video cases.  In particular, auto-determine if dealing with video (caps).

baseparse: avoid unexpected stray metadata

baseparse: use proper _NONE output value when applicable

audioparsers: Remove dead assignments

audioparse: fix possible division-by-zero

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

baseparse: use correct offset when adding index entry

... bearing in mind that BUFFER_OFFSET is media specific and may not
reflect the basic offset after having been parsed.

baseparse: enhancements for timestamp marked framed formats

That is, as such formats allow subclass to extract position from frame,
it is possible to extract duration (if not otherwise provided)
from (near) last frame, and a seek can fairly accurately target the required
position.

Fixes #631389.

baseparse: refactor frame scanning peformed by _loop

baseparse: slightly optimize sending of pending newsegment events

baseparse: minor fixes and enhancements

Arrange for upstream as well as downstream flushing when seeking.
Also determine upstream size as well as seekability.  Adjust some comments
to reality and employ debug statement in proper order.

baseparse: use only upstream duration if it provides one

baseparse: reflow update_bitrate code

... which makes local variables represent real state better, and avoids
triggering unneeded updates/actions.

baseparse: add some debug statements

baseparse: perform bitrate handling and posting after newsegment sending

baseparse: immediately post subclass provided bitrate

Revert "baseparse: add skip property"

This reverts commit b5a3d60363d837a10f0533c141ec93d10b742312.

Reverting this for now, since no one really seems to remember why this
property exists or what it could possibly be good for. It seems to have
been in the original mp3parse since the beginning of time and was back-
ported from there.

audioparser: Let the format string agree with the parameters to fix compiler warning

baseparse: Fix debug output

We lose the reference to the buffer after gst_pad_push(), so the debug
print should happen before.

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

baseparse: support reverse playback

... in pull mode or upstream driven.

baseparse: remove done TODOs and update documentation

baseparse: use determined seekability in answering SEEKING query

baseparse: add skip property

baseparse: use _set_frame_props to configure frame lead_in and lead_out

... provided a corresponding decoder with sufficient leading and following
frames to carry out full decoding for a particular segment.

baseparse: use _set_duration to configure duration update interval

... as it logically belongs there as one or the other; either subclass
can provide a duration, or an estimate must be made (reguarly updated).

baseparse: localize use of provided fps information

baseparse: seek table and accurate seek support

baseparse: proper and more extended segment and seek handling

That is, loop pause handling, segment seek support, newsegment for gaps, etc

baseparse: add index support

baseparse: refactor state reset

baseparse: prevent indefinite resyncing

baseparse: specific EOS handling if no output so far

baseparse: adjust _set_frame_prop documentation and set default as claimed

baseparse: fix bitrate copy-and-paste and update heuristic

baseparse: post duration message if average bitrates is updated

baseparse: remove is_seekable vmethod and use a set_seek instead

Seekability, like duration, etc is unlikely to change (frequently), and
the default assumption covers most cases, so let subclass set when needed.
At the same time, allow subclass to indicate if it has seek-metadata (table)
available, and possibly have it provide an average bitrate.

baseparse: add another hook for subclass prior to pushing buffer

... and allow subclass to perform custom segment clipping, or to
emit tags or messages at this time.

baseparse: 0 converts to 0 by default

baseparse: refactor conversion using helper function and export default convert

baseparse: streamline query handling

baseparse: cleanup struct and remove unused member

baseparse: Allow chaining of subclass event handlers

This allows the child class to chain its event handler with
GstBaseParse, so that subclasses don't have to duplicate all the default
event handling logic.

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

baseparse: Don't use GST_FLOW_IS_FATAL()

Also don't post an error message for UNEXPECTED and do it
for NOT_LINKED.

baseparse: non-TIME seek event is simply not handled

baseparse: fix seek event ref handling

baseparse: prevent arithmetic overflows in pull mode buffer cache handling

baseparse: fix seek handling

Allow a few more seek event type combinations, and really use the result
of gst_segment_set_seek to perform the seek.  Also add some debug.

baseparse: Don't emit bitrate tags too early

We wait to parse a minimum number of frames (10, arbitrarily) before
emiting bitrate tags so that our early estimates are not wildly
inaccurate for streams that start with a silence. If the stream ends
before that, we just emit the tags anyway.

While it _would_ be nicer to be specify the threshold to start pushing
the tags in terms of duration, this would introduce more complexity than
this merits.

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

baseparse: Set the last stop to the buffer starttime if the duration is invalid

...instead of not setting it at all.

baseparse: Send NEWSEGMENT event with correct start and position

Instead of taking the last stop (which could be buffer endtime instead
of starttime) always take the buffer starttime.

Fixes bug #614016.

audioparsers: remove unused GstBaseParseClassPrivate structure

audioparsers: Add bitrate calculation to baseparse

This makes baseparse keep a running average of the stream bitrate, as
well as the minimum and maximum bitrates. Subclasses can override a
vfunc to make sure that per-frame overhead from the container is not
accounted for in the bitrate calculation.

We take care not to override the bitrate, minimum-bitrate, and
maximum-bitrate tags if they have been posted upstream. We also
rate-limit the emission of bitrate so that it is only triggered by a
change of >10 kbps.

audioparsers: rename baseparse GType name to avoid possible conflicts

audioparsers: documentation fixes

baseparse: adjust seek handling and newsegment sending

Perform sanity check on type of seek, and only perform one that is
appropriately supported.  Adjust downstream newsegment event
to first buffer timestamp that is sent downstream.

baseparse: minor refactor cleanup

Also add some debug logging.

baseparse: implement leftover draining in pull mode

baseparse: provide default conversion using bps if no fps available

Also store estimated duration as such, rather than pretending otherwise
(e.g. set by subclass).

baseparse: check for remaining data when draining in push mode

baseparse: fix pull mode cache size comparison

audioparse: fix a format string as reported on irc.

baseparse: custom bufferflag indicates not to count frame in stats

audioparsers: reference GstBaseParse now lives here

audioparsers: rename 'aacparse' plugin to generic 'audioparsers' plugin

baseparse: reset passthrough mode to default (disabled) on activation

baseparse: ensure buffer metadata is writable

baseparse: fix/enhance DISCONT marking

In particular, consider DISCONT == !sync, and allow subclass to query
sync state, as it may want to perform additional checks depending
on whether sync was achieved earlier on.
Also arrange for subclass to query whether leftover data is being drained.

baseparse: add timestamp handling, and default conversion

In particular, (optionally) provide baseparse with a notion of frames per second
(and therefore also frame duration) and have it track frame and byte counts.
This way, subclass can provide baseparse with fps and have it provide default
buffer time metadata and conversions, though subclass can still install
callbacks to handle such itself.

baseparse: documentation fixes

baseparse: use_fixed_caps for src pad

After all, stream is as-is, and there is little molding to downstream's
taste that can be done.  If subclass can and wants to do so, it can
still override as such.

aacparse: Fix compilation warnings

aacparse: fix warnings in macosx snow leopard

aacparse: forego (bogus) parsing of already parsed (raw) input

baseparse: prevent infinite loop when draining

baseparse: fix minor memory leak

aacparse: Add function for the baseparse subclass to push buffers downstream

Also handle the case gracefully where the subclass decides to drop
the first buffers and has no caps set yet. It's still required to
have valid caps set when the first buffer should be passed downstream.

baseparse: Fix seek event leaking

baseparse: propagate return value of GstBaseParse::set_sink_caps()

gst_base_parse_sink_setcaps() presumably should fail if the subclass
returns FALSE from its ::set_sink_caps() function.

baseparse: don't try to GST_LOG an already-freed caps string

The proper way to log caps is via GST_PTR_FORMAT anyway.

baseparse: fix debug category

baseparse: fix (regression in) newsegment handling

(aacparse, amrparse, flacparse).  Fixes #580133.

baseparse: Fix slightly broken buffer-in-segment check (aacparse, amrparse, flacparse)

baseparse: Fix push mode seeking (aacparse, amrparse)

Sending the flush-start event forward before taking the stream lock actually
works, in contrast to deadlocking in downstream preroll_wait (hunk 1).

After that we get the chain function being stuck in a busy loop. This is fixed
by updating the minimum frame size inside the synchronization loop because the
subclass asks for more data in this way (hunk 2).

Finally, this leads to a very probable crash because the subclass can find a
valid frame with a size greater than the currently available data in the
adapter. This makes the subsequent gst_adapter_take_buffer call return NULL,
which is not expected (hunk 3).

baseparse: Delay newsegment as long as possible.

If newsegment is sent (too) early, caps may not yet be fixed/set,
and downstream may not have been linked.

baseparse: revert last change and properly fix

Baseparse internaly breaks the semantics of a _chain function by calling it with
buffer==NULL. The reson I belived it was okay to remove it was that there is
also an unchecked access to buffer later in _chain. Actually that code is wrong,
as it most probably wants to set discont on the outgoing buffer.

baseparse: remove checks for buffer==NULL

Accordifn to docs for GstPadChainFunction buffer cannot be NULL. If we would
leave the check, we would also need more such check below.

Fix the return value of the default parse_frame function.

Fix the return value of the default parse_frame function in both
copies of GstBaseParse

gst/: Fix baseparse type name.

Original commit message from CVS:
* gst/aacparse/gstbaseparse.c:
* gst/amrparse/gstbaseparse.c:
Fix baseparse type name.

Add two new baseparse based parsers (aac and amr) from Bug #518857.

Original commit message from CVS:
* configure.ac:
* gst/aacparse/Makefile.am:
* gst/aacparse/gstaacparse.c:
* gst/aacparse/gstaacparse.h:
* gst/aacparse/gstbaseparse.c:
* gst/aacparse/gstbaseparse.h:
* gst/amrparse/Makefile.am:
* gst/amrparse/gstamrparse.c:
* gst/amrparse/gstamrparse.h:
* gst/amrparse/gstbaseparse.c:
* gst/amrparse/gstbaseparse.h:
Add two new baseparse based parsers (aac and amr) from Bug #518857.

queue[2]: Make src query MT-safe

It is possible that the element might be going down while the event arrives