nlecomposition: Do not fail when removing/adding child without commiting

Summary:
We use to end up removing the nleobject when the following case happened:

 * add an object
 * remove that object
 * re add the object
 * commit the composition

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D193

timeline: Minor documentation addition

ges: Enhance xges format versioning

Summary:
Handle the fact that some new features can be added and that means
generated files will not be fully understandable by older versions of
the formatter.

Make sure that we set the format version to 0.2 when we serialize the
GstEncodingProfile.enabled property.

Add some tests around that.

+ Fix a minor bug in the test-utils
+ Add a meta on the projects to tell in what format version a project
  has been serialized/parsed back

API:
  GES_META_FORMAT_VERSION

Depends on D178

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D184

ges: If last added clip is not in a layer, get the first layer

Summary:
In case we just removed it from its layer, make sure to
just use the first layer when none specified.

Depends on D177

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D178

ges: Fix some error settings

Summary: Depends on D176

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D177

ges:structured-interface: Use GET_AND_CHECK in more places

Summary:
Giving more details about the issue to the user

Depends on D151

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D176

xml-formatter: De/serialize whether encoding profiles are enabled or not

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D151

Remove obsolete Android build cruft

This is not needed any longer.

Update .gitignore

tools: Fix string leak

Only allocate the return string when we know we are going to return
it.

Coverity CID #1292292

Remove INSTALL file

autotools automatically generate this, and when using different versions
for autogen.sh there will always be changes to a file tracked by git.

ges:xml-formatter: Call g_markup_parse_context_end_parse

Summary:
Otherwise the parser context will never know that is all the XML it
will receive and fail out if the XML document is not valid (in that
case if it does not end)

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

Reviewers: Mathieu_Du

Reviewed By: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D38

ges: Add debug output when get_element returns NULL

tools:launch: Print out the timeline description as an INFO not an ERROR

tools:launch: clean user facing message on wrong timeline description

Summary:
Before:

  $ ../gst-editing-services/tools/ges-launch-1.0 -p
  0:00:00.028629728  8155      0x17e1b60 ERROR                default ges-launcher.c:214:_create_timeline: serialized timeline is   -p

  ** (lt-ges-launch-1.0:8155): ERROR **: Could not create timeline, error: Could not find a suitable formatter
  [1]    8155 trace trap (core dumped)  ../gst-editing-services/tools/ges-launch-1.0 -p

  $

After:

  $ GST_DEBUG=0 ges-launch-1.0 -p

  ERROR: Could not create timeline, error: Could not find a suitable formatter

  $

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D95

nleobject: It is wrong to update object->stop in set_property.

Summary: It must only be done when the object is commited.
We can do that in constructed though, as the changes will
anyway be commited when the object is added to a composition.

Also update the tests, as we set properties spearately then
check the stop, we can commit the source at its creation without
removing meaning from the tests.

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D84

timeline, track: Emit commited at the correct moment.

Summary:
+ [API] GESTrack::commited signal.
+ [API] ges_track_commit_sync

We were emitting commited when timeline_commit was called, which
wasn't very helpful. This commit makes it so we emit commited once
all the compositions have actually been commited.

We also add a synchronous commit method to spare the user
the need to connect to the signal and wait, and update the
documentation.

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D83

layer: call timeline_element_set_timeline in layer_set_timeline.

Summary:
Otherwise if there was still a reference to the layer when it
is removed from the timeline, it fails when the last reference
is released, because timeline_element_set_timeline calls
timeline_remove_element, which tries to remove the element from
an already disposed hashtable.

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D82

tests: Use AM_TESTS_ENVIRONMENT

Needed by the new automake test runner

ges-launch: Add support for +test-clip

Summary: With the pattern as a mandatory argument.

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D68

ges-project: Surface a meaningful error when no suitable formatter.

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D67

ges-launch: Add a save-only option.

Summary: + Allows to serialize the timeline without playing it back.

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D66

Automatic update of common submodule

From bc76a8b to c8fb372

timeline-element: Disconnect child properties handlers.

Summary:
+ And freeze notifies while doing so.

We had a race with GstController which isn't MT safe, we can
fix it by propertly disconnecting signals, and making sure
no notifies are emitted while doing so.

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D64

track: Set any caps features on tmpcaps.

Summary: Before checking if we have a specific constructor for a track type.

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D63

pipeline: no reason to disconnect a pad that is NULL anyway.

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D62

ges-launch: Better document options.

+ Sort them by topic
+ remove --sample-paths and --sample-paths-recurse.

http://phabricator.freedesktop.org/D58

ges-launch: port to GApplication

Summary: + Extract some utility functions.

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D55

build: no reason to introspect nodist sources.

Summary: g-ir-scanner was erroring like crazy on the generated sources.

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D57

ges-pitivi-formatter: Don't flood stdout with alarming conclusions.

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D56

completions: port to new base gst script

completions: Fix completions after the first command.

Revert "ges-launch: no need for a tmp string pointer"

This reverts commit 44a0924c1f6b07f0c91ee8bd03d3ae5d97da92d5.

There indeed is a need for a tmp string pointer.

ges-launch: no need for a tmp string pointer

ges-launch: free string before going out of scope

CID #1291632

build: fix make distcheck.

ges: Do not clear potentially NULL errors

And avoid dereferencing NULL errors

ges-launch: Prefix clip, transition and effect instruction with a +

Slightly changing the CLI so that we have indicators of the timeline
commands adding new objects.

ges: Accept path as URI in the create clip structured interface

ges-launch: Remove useless options, rename some short options.

bash-completion: Add support for new ges-launch commands.

parse.l: Modify command arguments.

+ --clip uri=file:// becomes clip file:// for example.

ges: command-line-formatter: Properly error out on invalid arguments

ges: Factor out a GESCommandLineFormatter class

This formatter will allow any user to deserialize a timeline using
the new ges-launch command line interface

structured-interface: Be clever when no layer priority specified.

And add the new element to the same layer as the last clip that
was added, insted of adding to the last layer of the timeline
(and with the current code, actually adding a new layer each time)

ges-structured-interface: fix build

ges: Automatically put clips at the end of layer if no start specified

In the 'structured' interface we should add it at the end of the layer

And make use of the new ges_timeline_get_layer API

structure-interface: rename layer-priority to layer.

+ And add a short name for it in ges-launch.

ges: Add layer up to the wanted layer priority in the structure interface

Making the thing easier to use

ges-launch: parse property names longer than 1 char.

+ And finish the previous structure when encountering a setter.

ges-launch: Update lexer / parser to handle set-*

+ cleanup of the lexer

tools: Implement a new CLI interface for the timeline creation

ges: Handle setting child property on container directly in the structured based interface

launcher: Add support to --set-property in the parser

ges-launch: Implement a new parser for the commandline.

Summary: + flex-based lexing and manual simplistic parsing.

Test Plan: Use that stuff to make awesome things, see if it breaks.

ges: Add an internal GstStructure based interface

To be use by GstValidate action and ges-launch

Reviewers: Mathieu_Du, thiblahute

Differential Revision: http://phabricator.freedesktop.org/D42

ges: Fix build for older GLib

The return type of g_hash_table_insert changed from void to boolean

container: implement children property handling

ges: Move the notion of children properties to GESTimelineElement

Summary:
Deprecate the old GESTrackElement children property handling API.

New APIs:
  * ges_timeline_element_list_children_properties
  * ges_timeline_element_lookup_child
  * ges_timeline_element_get_child_property_by_pspec
  * ges_timeline_element_get_child_property_valist
  * ges_timeline_element_get_child_properties
  * ges_timeline_element_set_child_property_valist
  * ges_timeline_element_set_child_property_by_pspec
  * ges_timeline_element_set_child_properties
  * ges_timeline_element_set_child_property
  * ges_timeline_element_get_child_property
  * ges_timeline_element_add_child_property
  * ges_timeline_element_remove_child_property

Deprecated APIs:
  * ges_track_element_list_children_properties
  * ges_track_element_lookup_child
  * ges_track_element_get_child_property_by_pspec
  * ges_track_element_get_child_property_valist
  * ges_track_element_get_child_properties
  * ges_track_element_set_child_property_valist
  * ges_track_element_set_child_property_by_pspec
  * ges_track_element_set_child_properties
  * ges_track_element_set_child_property
  * ges_track_element_get_child_property
  * ges_track_element_add_child_property

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D40

timeline: Add API to find a layer with a specific priority in a timeline

Summary:
API:
  * ges_timeline_get_layer

Test Plan: Nan

Reviewers: mathieu.duponchelle

Revert "ges: Move the notion of children properties to GESTimelineElement"

I got some trouble with

  arc land

and I wanted to push the 3 commit coming after this revert as 3
different commits but they ended up being all squash into one single
commit, which is clearly not cool for later bisecting and blaming.
Reverting that commit and re pushing those 3 commits as they were
supposed to be.

This reverts commit 9fe15ef4354dc1d878dbdec80908ac8541bc6131.

ges: Move the notion of children properties to GESTimelineElement

Summary:
Deprecate the old GESTrackElement children property handling API.

New APIs:
  * ges_timeline_element_list_children_properties
  * ges_timeline_element_lookup_child
  * ges_timeline_element_get_child_property_by_pspec
  * ges_timeline_element_get_child_property_valist
  * ges_timeline_element_get_child_properties
  * ges_timeline_element_set_child_property_valist
  * ges_timeline_element_set_child_property_by_pspec
  * ges_timeline_element_set_child_properties
  * ges_timeline_element_set_child_property
  * ges_timeline_element_get_child_property
  * ges_timeline_element_add_child_property
  * ges_timeline_element_remove_child_property

Deprecated APIs:
  * ges_track_element_list_children_properties
  * ges_track_element_lookup_child
  * ges_track_element_get_child_property_by_pspec
  * ges_track_element_get_child_property_valist
  * ges_track_element_get_child_properties
  * ges_track_element_set_child_property_valist
  * ges_track_element_set_child_property_by_pspec
  * ges_track_element_set_child_properties
  * ges_track_element_set_child_property
  * ges_track_element_get_child_property
  * ges_track_element_add_child_property

Reviewers: Mathieu_Du

Reviewed By: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D40

ges: Remove all reference to already dead GESSimpleLayer

project: remove unnecessary dereference

g_clear_error() already dereferences the error pointer, no need to manually
check and do it.

CID #1257630

examples: check argument is valid

ges-base-xml-formatter: fix setting of child properties

Make sure all child properties get set. GstStructureForeachFunc
takes a gboolean return value that decides whether to
continue or not.

ges-meta-container: fix ges_meta_container_foreach()

Really call function on all metadata inside the container
instead of stopping randomly. GstStructureForeachFunc
takes a gboolean return value.

ges-base-xml-formatter: fix property setting

GstStructureForeachFunc has a gboolean return value,
and the foreach function will stop unless we return
TRUE here. This meant it was potluck whether all
properties in the structure got set or not.

Fixes setting of text overlay clip text property
in particular.

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

build: fix make distcheck.

And install bash-completions in the supplied prefix.

ges-launch: enable auto-completion.

Summary: + And be a little smart about it.

Test Plan: New feature, working, not testing bash completion

Reviewers: tsaunier

Differential Revision: http://internal.opencreed.com:8888/D25

ges: Always set ANY capsfeatures on tracks caps property

Summary:
We should not restrict the CapsFeatures on the track caps.
If someone want to do such a restriction he should add it to
the restriction caps directly

Test Plan: Run testsuite

Reviewers: mathieu.duponchelle

ges-project: no need to commit an empty timeline.

Summary: Can lead to deadlocks if the user commits at the same time.

Test Plan: Ran make check, it worked

Reviewers: tsaunier

ges: initialize timeline_duration value

If priv->timeline is False the function does not set any value for
timeline_duration before using it in gap_new (). Initialize the value to aviod
unexpected behaviour.

CID #1268405

configure: Bump our Gst related dependencies to 1.5.0.1

nlecomposition: Properly protect the children task

Cleanly handle removing the last object in a composition

The strategy here is to seek at the new end of the composition. And in
GES we always add a 1ns long gap at the end of the tracks so that all
track have the exact same duration, and we have black frames when the
timeline is empty

validate: Do not wrongly set clip duration for UriClips

That was making no sense at all....

validate: Properly expose the commit action as ASYNC

layer: Remove child from children list before emitting "child-removed"

ges: remove useless gpointer variable

gpointer useless is indeed useless since we can use GST_DEBUG_REGISTER_FUNCPTR
to avoid having to store the return of the GST_DEBUG_FUNCPTR registration.

CID #1265771

ges: merge MIN() and MAX() into CLAMP()

Merge the usage of MIN() and MAX() into one CLAMP() function.

CID #1265770

ges: Make sure the GESTextOverlayClip is register on init

So it can be used when de serializing projects containing it.

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

Automatic update of common submodule

From f2c6b95 to bc76a8b

Automatic update of common submodule

From ef1ffdc to f2c6b95

tools: Avoid trying to remove a signal handler that has already been removed

uri-asset: Check that the newly computed URI exist

No the one we know failed!

validate: Add an action type to load a project from its content

Revert "track: [API]: ges_track_update_restriction_caps."

This reverts commit e9544ce1d67da6990f0a1cae75774063ec37be9d.

This commit should never have landed we decided we do not want to do
that.

timeline: Add a method to easily check if a timeline is empty

API:
    + ges_timeline_is_empty

ges: Recompute Group priority when one of its clip.layer change priority

And add a unit test for that case where it was previously failing

validate: Start using the new testsuite based API from GstValidate

track: [API]: ges_track_update_restriction_caps.

+ And specify default restriction caps for audio and video tracks.
+ Add ges_track_set_restriction_caps to the sections, it was missing.

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

tests: Remove integration tests, GstValidate is the way forward!

validate: Handle long tests in the TestManager

+ Minor bug fixes

nlecomposition: Minor debug enhancements

pipeline: Expose playsink::video-filter and playsink::audio-filter

That can be used to add filters at the very end of the pipeline,
and one can think of adding a watchdog element in there for
example.

pipeline: Dot not check for chain->tee twice

validate: Avoid depending on not stable APIs

And cleanup includes

validate: Move to new action type registration API

validate: Add missing action execution printing