Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Use a separate callback for pictures allocated in heap and pictures
allocated from video output buffers.

This is only used in error corner cases, and is always the same as
calling subpicture_Delete() directly.

All variants of the filter_t.video.buffer_del callback invoke
picture_Release() - possibly through recursion.

Most filters used picture_Release() directly already.

Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

This makes picture_pool_Get() reentrant.

This reduces the size of picture_gc_sys_t. The callbacks are always the
same for all pictures in a given pool.

We can offload the picture deletion to Destroy() in all cases.

The item-change callback might be called very often, and thus an
own code path is better, with only necessary updates.

Additionally, this fixes multiple resume playback dialogs occuring
at once when starting playback in certain situations.

refs #11981

Also handles CustomAttributes

Was downloading regardless of consumer

Fixes most of glitches for non live streams, especially on
quality change.

CTTS value can be negative on version 0, and match the not found
return value used in GetPTSDelta.
ex: Sintel Smooth Streaming sample