composition: Avoid emitting COMMITED like crazy

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Add an enum to define why we update the stack

Making the code simpler to follow

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: We are now waiting for caps to restart our task

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Remove useless functions to add GSources and add debugging

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Remove DONE fixme

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

ghostpad: Do not try to be smarter than possible with seqnum

We can have several CAPS event comming at any time and thuse we will
need to rely on elements to handle their seqnum properly as we can not
do a safe guard at our level

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Keep our GSources in a list making their thread safe

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Concider the last action as DONE when we get a CAPS or SEGMENT

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: only forward our flush start / stops

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

ghostpad: remove useless debug

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: cleanup and enhance debug

Bye Bye STRAM START you were a brave little debug we will miss you.

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: do not flush downstream when updating pipeline ourselves.

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Set update to TRUE when updating the stack because of EOS

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Handle seeking current stack while PAUSED

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Add and enhance some debug

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Dot the newly created stacks

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

gnlghostpad: Add some more debugging and fix mistakes in seqnum handling

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Factor out a function to seek the current stack

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Set the seqnum only when receiving the actual seek event

Setting it before calling seek_handling is racy!

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Set the real_eos_seqnum in a dedicated method

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Reset the real_eos_seqnum when reseting the composition

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Release OBJECTS_LOCK when emiting the "commited"

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Use the private struct directly

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

gnloperation: Totally clear operations when removing emptying current_bin

Removing linked childre leads to weird behaviour, we want to make sure
that all elements are totally clean when they are out the current bin.

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

gnlghostpad: Do not forget to set output segment seqnum

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Fix the build

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

tests:composition: Fix some refcounts

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Relink nodes *after* setting the seqnum

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

gnlghostpad: Do not tolerate getting seeked when no target is set

This should not happen anymore

+ Do not set twice events seqnums

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

gnlghostpad: Add missing seqnum tweaking

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

tests: Use compositor instead of videomixer

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Handle the case were we get an EOS right after a segment

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Make basic seeking working

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

tests: Start fixing operation tests

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

tests: Fix composition tests

We can not expect a seek event anymore as we are seeking in READY the elements
themselves

+remove actual sinks

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Implement the logic to PAUSE the task while executing actions

We need to wait for the pipeline update to be actually finished before we can start another
action. That means that we pause the task until one buffer from the new stack is
outputed.

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Do not forget to set expandables state to NULL when disposing

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Factor out a function to commit only the values

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Get the current position before actually commiting values on commit

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Add a method that set the current bin to READY

We need to get the stream lock in some conditions, and thuse send
flush event in those cases.

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Remove useless code

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Clear the old stack when removing children that where used

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Remove useless toplevel_seek argiment from activate_new_stack

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Rework the seqnum logic to avoid races when setting the new stack seqnum

When we were seeking the same stack without a logic that gurantees that we actually
saw the seek with the new seqnum set, we could have ended up with an EOS set with
the right seqnum even if it was actually not the case.

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Add/Remove children in the same order as they were called

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Remove useless flush_start argument from the stop_task method

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Force setting children state to NULL rebfore unrefing them

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

compositition: Check last stack in the children thread

Avoiding to take the OBJECT_LOCK when recieving EOS. The computation is
based on the GstEvent.seqnum to make sure that the EOS we receive
corresponds to the right sequence.

In that patch we tweak seqnums so that they are correctly computed
avoiding to depend on all elements to do it properly as it might pretty
much not be the case!

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Actiually set current_bin state even when not debugging

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Set *all* children state when going back to READY and then NULL

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Fix toplevel seek event refcounting

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Teach the composition to seek same stack

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: The ref the user gave us is our, and we give another to the bin when needed

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Rename commit_pipeline_func to commit_func

We do not commit any pipeline, we commit the new state of the composition internals

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Set the state of the internal bin before removing its children

Otherwize when we set the composition state to READY or NULL we can end
up with children in PAUSED state outside of everything

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

tests: Minor cleanup

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

gnlobject: Allow commiting of object that are not inside a composition

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Empty current_bin on gnl_composition_reset

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Remove now useless external_gst_bin_add_remove field

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Remove now useless pad probes

Co-Authored by: Thibault Saunier <tsaunier@gnome.org>

composition: Add an internal bin where that contain used children

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

composition: Add objects to the pending IO list in a GSource

This way we make sure we do not manipulate our children from another
thread than the dedicated one.

composition: Start implementing seeking in a GSource

composition: switch to using an action signal to add and remove objects.

composition: chain up finalize before clering mcontext_lock.

composition: Initialize the first stack async

Finish fixing tests

tests/common: disconnect commited handler

composition: Actually commit in on our own thread

Avoiding races

composition: start and stop task in init and finalize

tests: unref message correctly

composition: home grown task

Don't be sync silly test

composition: simplify update_pipeline_func

composition: Reset the srcpad target when removing the toplevelentry

composition: Use a GstPad task to run the update pipeline thread

gnlcomposition: Factor out code to deactivate old stack and activate new one

composition: Avoid looping using gotos

gnlcomposition: Use the new _object_block_and_drop_data where appropriate

Renaming block_object_src_pad to _object_block_and_drop_data

gnlcomposition: factor out some functions

gnlcomposition: Factor out the condition of _parent_or_priority_changed

gnlcomposition: Add a function to block object source pad

gnl: Add the srcpad directly to GnlObject

Starting from now we will not claim that we support GnlObject that have
several source pads as this is
1- Not true at all;
2- the design of priorities in the GnlComposition tree does not allow that;
3- Not very useful in most of the cases and it complexifies quite a lot the code
   in the composition.

Conflicts:
configure.ac
tests/check/Makefile.am

Revert "gnlobject: Commit object in READY_TO_PAUSED"

This causes races when seeking, reverting for now even if we will
probably want to bring something like that back.

This reverts commit 3549e745a8f0de3977b83c60e9b447afaf55d8a0.

gnlsource: remove useless "controls_one" field.

Import GNL from 978332e7c4c3bba1949421d28b492540ab471450 'Release 1.4.0'

tests: timelineedition: Init GES once at the beginning.

Avoiding to forget to init in a particular test and failling stupidly

test: backgroundsource: Disable tests that rely on nlecomposition internals

We can't rely on that, in particular now that it does not actually
add its children all the time but only when it is needed (and that
it has an internal bin where actual things happen).

timeline: Add a method to retrieve a pad from a track

It allows user to easily get the proxied pad from a track.

API:
  + ges_timeline_get_pad_for_track

tools:launcher: Avoid commiting when we are not done loading the timeline

ges: Use the new GNL element message feature to notify async operations

xml-formatter: message-forward is not something that should be parsed.

timeline: handle async start

tests: implement our validate TestManager.

And make sure it installs alongside the other validate apps.

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

smart-video-mixer: unref pads_info hash table in dispose

Before gst_bin_dispose() runs and destroys elements.

smart-adder: fix crash in unit test

Unref pads_info hash table in dispose instead of
finalize, i.e. before gst_bin_dispose runs and
destroys pads_info->bin (to which the pads_info
does not hold a ref).

Automatic update of common submodule

From 84d06cd to 7bb2bce

track: fix indentation

timeline-element: don't leak name string

timeline: fix mutex and all_elements hash table leaks