curltlssink: swap check

Lets check for NULL first, as that is the default anyway.

h264parse: Return flushing if we get chained while being set to READY

Or not negotiated in the case we would be actually not negotiated

Currently we are getting assertions from
gst_pb_utils_add_codec_description_to_tag_list because of NULL
caps.

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

dashdemux: mark first buffer as discont after restarting a download task

Mark first buffer as discont after restarting a download task, so
downstream can reset its state to properly handle the new fragment.

Related issue: https://bugzilla.gnome.org/show_bug.cgi?id=736318

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

dashdemux: push a time segment when restarting streaming

This should inform demuxers/decoders downstream that the stream is
about to start from a new position and they should flush accordingly

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

video: use video lib conversion code instead of copy

mplex: break statement unnecessary

One more code flow issue where it never arrives to the break statement due to
the goto. So the break is unnecessary.
Plus two coding style fixes.

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

chopmydata: Transfer timestamps to output buffers

Allows using chopmydata as a simple repacketizer

androidmedia: break statements unnecessary

Code flow never arrives to the break statements due to the goto's. So the
breaks are unecessary. Second part to commit a3958ae65b

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

androidmedia: break statement unnecessary

Code flow never arrives to the break statements due to the goto's. So the
breaks are unecessary.

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

dshowsrcwrapper: avoid error when set_caps called twice

If set_caps is called in a running state, return immediately if the caps
haven't changed. If the pins are already connected, disconnect them.

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

glvideomixer: skip input frames with an alpha of 0

gl: download whenever we have sysmem capsfeatures

Otherwise we could pass on a RGBA formatted buffer and downstream would
misinterpret that as some other video format.

Fixes pipelines of the form

    gleffects ! tee ! xvimagesink

gsettings: Fix typo in description

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

mpegpsmux: refactor duplicate else branches

Unknown case should assert if reached instead of just assuming the max buffer
size.

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

vc1parse: initialize sent_codec_tag before using it

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

videofilters: Add support for more YUV formats

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

gltestsrc: add Mandelbrot fractal pattern.

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

gltestsrc: implement snow pattern with GLSL.

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

glupload: fallback to upload if contexts cannot share GL resources

Fixes pipelines of the form

    gleffects ! tee ! glimagesink

as tee does not pass on the allocation query.

glcontext: add gst_gl_context_can_share

Which determines whether two GstGLContext's can share sharable
OpenGL resources.

gl/tests: update for upload API changes

gl: Let gst_gl_context_get_thread() return a new reference to the GThread

Otherwise it might go away in the meantime and the caller has some random pointer.

gl: consecutive return, break statement

Signed-off-by: Anuj Jaiswal <anuj.jaiswal@samsung.com>
https://bugzilla.gnome.org/show_bug.cgi?id=736939

gl/debug: desktop GL does not have the KHR suffixes

examples/gl/gtk: use the sync bus callback to set the window handle

And output error messages.

dvbsrc: add missing FEC rates

Order of the enum values (new rates after _AUTO)
has been kept congruent with the one in the v4l
API for consistency.

gl: add since markers for new API

glmemory: add debug line for freeing textures

glupload: provide the output buffer that is rendered into

Allows callers to properly reference count the buffers used for
rendering.

Fixes a redraw race in glimagesink where the previous buffer
(the one used for redraw operations) is freed as soon as the next
buffer is uploaded.

1. glimagesink uploads in _prepare() to texture n
1.1 glupload holds buffer n
2. glimagesink _render()s texture n
3. glimagesink uploads texture n+1
3.1 glupload free previous buffer which deletes texture n
3.2 glupload holds buffer n+1
4. glwindow resize/expose
5. glimagesink redraws with texture n

The race is that the buffer n (the one used for redrawing) is freed as soon as
the buffer n+1 arrives.  There could be any amount of time and number of
redraws between this event and when buffer n+1 is actually rendered and thus
replaces buffer n as the redraw source.

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

glcontext: add API to retreive the thread that context is active in

gl: Removed unreachable break, unused variable

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

vtenc: Fix encoder to work on iOS

iOS has special stride requirements that we don't know yet, so copy
input buffers into buffers allocated by iOS for now.

Later we should check the stride and probably provide a buffer pool for these
buffers so upstream can directly write in there.

applemedia: We need to map the coremedia buffers in non-readonly mode

Our buffers claim to be writable so other elements expect to be able
to write there without losing their changes.

hlsdemux: lock client mutex before entering the retry_failover block

This is consistent with the case where the block execution is
triggered by the goto invoked after the current_variant update.

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

mssdemux: synchronize with the download loop thread to signal it to continue

If EOS or ERROR happens before the download loop thread has reached its
g_cond_wait() call, then the g_cond_signal doesn't have any effect and
the download loop thread stucks later.

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

hlsdemux: synchronize with the download loop thread to signal it to continue

If EOS or ERROR happens before the download loop thread has reached its
g_cond_wait() call, then the g_cond_signal doesn't have any effect and
the download loop thread stucks later.

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

dashdemux: synchronize with the download loop thread to signal it to continue

If EOS or ERROR happens before the download loop thread has reached its
g_cond_wait() call, then the g_cond_signal doesn't have any effect and
the download loop thread stucks later.

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

mssdemux: fix clearing of eos state in pads

The internal pad still keeps its EOS flag and event as it can be assigned
after the flush-start/stop pair is sent. The EOS is assigned from the streaming
thread so this is racy.

To be sure to clear it, it has to be done after setting the source to READY to
be sure that its streaming thread isn't running.

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

hlsdemux: fix clearing of eos state in pads

The internal pad still keeps its EOS flag and event as it can be assigned
after the flush-start/stop pair is sent. The EOS is assigned from the streaming
thread so this is racy.

To be sure to clear it, it has to be done after setting the source to READY to
be sure that its streaming thread isn't running.

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

dashdemux: fix clearing of eos state in pads

The internal pad still keeps its EOS flag and event as it can be assigned
after the flush-start/stop pair is sent. The EOS is assigned from the streaming
thread so this is racy.

To be sure to clear it, it has to be done after setting the source to READY to
be sure that its streaming thread isn't running.

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

vtenc: Let the encoder automatically choose a h264 level

We should negotiate these things via caps...

vtenc: Use 0 instead of G_MAXDOUBLE as the max keyframe interval

0 means no limit, which was meant here with G_MAXDOUBLE probably.

vtenc: Expected duration is supposed to be the duration of the stream, not a frame

Just don't set it for now, it isn't really needed.

vc1parser: add unit test for sequence-layer parsing

Check that a sequence-layer header is successfully parsed.

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

vc1parser: take care of endianness when parsing sequence-layer

sequence-layer is serialized in little-endian byte order except for
STRUCT_C which is serialized in big-endian byte order.

But since STRUCT_A and STRUCT_B fields are defined as unsigned int msb
first, we have to pass them as big-endian to their parsing function. So
we basically use temporary buffers to convert them in big-endian.

See SMPTE 421M Annex J and L.

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

vtenc: Properly handle keyframes

Especially set the SYNC_POINT flag on keyframes.

mpegdemux: removed an unwanted initialization and a variable

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

dshowsrcwrapper: Port to 1.0

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

camerabin2: removed redundant initialization

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

faad: logical disjunction consecutive return, break unnecessary

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

vc1parse: enable header-format conversion

In fact we support header-format conversion which is done in
update_caps() method.

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

vtdec: Fix compilation

vtdec: Handle 0/1 framerates correctly by not calculating their frame duration

atdec: Fix some compiler warnings with newer clang

vtenc: Prepare encoder after setup

This will allow encoding to happen faster on the first frame.

vtenc: Only drain the encoder in ::finish(), not on every frame

Otherwise quality and bitrate will be bad.

vtenc: Error out if encoding returned an error

Otherwise we will just continue consuming frames until all memory is filled
up and the app crashes.

vtenc: Check for errors from VTCompressionSessionCompleteFrames()

vtenc: Properly scale timestamps for the API and set invalid values

vtdec: Properly scale timestamps for the API and set invalid values

applemedia: Remove old code that is of no use anymore

vc1parse: forge sequence-layer from seq_hdr instead of seq_layer

If we don't have a seq_layer_buffer, we also don't have a valid
seq_layer because there are set together in
gst_vc1_parse_handle_seq_layer().

So when output header format is sequence-layer and when we don't have a
seq_layer_buffer, we forge one from seq_hdr.

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

applemedia: Unconditionally use VideoToolbox on iOS if available

Because we do weak linking now we can always compile it in and check at
runtime if it is actually available or not.

applemedia: Do weak linking with the VideoToolbox framework

It does not exist on older OSX and iOS but we still want to
be able to use it when it's available.

waylandsink: do not leak buffer pool in error case

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

vc1parse: fix sequence-layer/frame-layer endianness

Sequence-layer and frame-layer are serialized in little-endian byte
order except for STRUCT_C and framedata fields as described in SMPTE 421M Annex
L.

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

vtenc: Port to GstVideoEncoder base class

vtenc: Use correct instance struct size

vtdec: Fix compiler warnings

values of type 'OSStatus' should not be used as format arguments; add an explicit cast to 'int' instead [-Wformat]

vtenc: Port to the real VideoToolbox API instead of using our dlopen() wrapper

It's a public framework since a long time.

glfilter: do not leak pool in error cases

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

vdpdecoder: do not leak pool

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

gltestsrc: do not leak pool in error cases

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

openni2src: do not leak pool

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

glmixer: do not leak pool in error cases

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

vmncdec: modify wrong packetized mode logic

packetized mode is being set when framerate is being set
which is not correct. Changing the same by checking the
input segement format. If input segment is in TIME it is
Packetized, and if it is in BYTES it is not.

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

webpdec: Remove unused variable

webpdec: modify wrong packetized mode logic

packetized mode is being set when framerate is being set
which is not correct. Changing the same by checking the
input segement format. If input segment is in TIME it is
Packetized, and if it is in BYTES it is not.

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

templatematch: Compare the correct loop variables

vtdec: No need to set kVTVideoDecoderSpecification_EnableHardwareAcceleratedVideoDecoder on iOS

configure: And make the preprocessor check work properly

configure: Do the AM_CONDITIONAL() after finally setting the variable

applemedia: Don't include VideoToolbox on iOS < 8.0

It's private API and does not work without a jailbroken device.

hlsdemux: Also refetch the playlist after the first fragment failure

Previously we only refetched the playlist if downloading a fragment
has failed once. We should also do that if it failed a second or third time,
chances are that the playlist was updated now and contains new URIs.

dash: Actually retry 3 times as advertised instead of 2

hls: Actually retry 3 times as advertised instead of 2

vtdec: Fix compilation on iOS 8.0

vc1parse: fix malformed sequence layer header and STRUCT_C

This commit fix several issues with sequence layer header forging on
update_caps():
- 0x00000004 unsigned integer is before STRUCT_C.

- Set reserved bits of STRUCT_C to their values for simple/main
  profiles in sequence layer header format and ASF header format.

- Sequence layer shall be represented as a sequence of 32 bits unsigned
  integers and shall be serialized in little-endian byte order except
  for STRUCT_C which shall be serialized in big-endian byte-order.

See SMPTE 421M Annex L for more details about sequence layer format.

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

dvbsrc: Use proper variable for delivery system

We want to check the delivery system is ISDBT (and not the modulation
which we check later in that case).

CID #1238439

tsdemux: fix trivial coding style issue

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

tsdemux: fix overflow of packet_length field of PESHeader

packet_length is defined as a guint16 in the PESHeader structure. This
definition match the specification. But since we add 6 bytes to the
packet_length value (length of start_code + stream_id + packet_length),
we can overflow the guint16 when the value in the PES header is greater
than 65529.
So use a guint32 instead of a guint16 to avoid overflow.

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

mssdemux: improve position detection when a download restart

Query other src pads before falling back to the position of the last
known pushed segment (which can be far ahead in time).

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

dashdemux: improve position detection when a download restart

Query other src pads before falling back to the position of the last
known pushed segment (which can be far ahead in time).

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

mpegpsmux: fix memory leak when converting H.264 avc to byte-stream format

Fix memory leak for input stream with caps "video/x-h264,stream-format=avc".

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

aiffmux: allocate byte writer on stack

asfmux: allocate byte readers on the stack

tsdemux: clear mutex resources in dispose

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

Fix up one-element lists in template caps

avfvideosrc: fix bogus BGRA caps

Remove 0.10-style endianness/*_mask/bpp/depth fields.

build: don't AC_SUBST intermediate values