ges: Namespace NLE utils function into GES

Avoiding name clashes when built statically

nle: Avoid unsetting srcpad target after the srcpad is already freed

That leaded to segfaults

nle: Stop wrongly set operation segment base time

Inside the composition we actually do not need to have any notion
of what the timing outside the compositon as we already tweak the segment
base time outside the composition. This code was only there to work
around https://bugzilla.gnome.org/show_bug.cgi?id=753196

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

nlecomposition: don't leak internal hashtable

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

nleobject: don't leak srcpad when disposing

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

timeline: don't leak pad in private structure

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

ges-launcher: don't leak GError

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

ges-launcher: fix double free when argument is invalid

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

video-source: Use the priority being set to compute zorder

video:transition: Set mixer pad zorder

video-source: Make sure to set framepositionner zorder when creating it

And fix a computation bug where we would be having mixing order
reversed between layers.

And make sure that the positionner does not mix up Transition handling
of the zorder

video-source: Simply set framepositionner->zorder = self->priority

Summary:
Making the code simpler and handling the transition case
where elements are in the same layer (which was failing
/setting same zorders until now).

Reviewers: Mathieu_Du

Differential Revision: https://phabricator.freedesktop.org/D237

Use standard GST_PLUGIN_LDFLAGS for the nle plugin

Add the standard GST_PLUGIN_LDFLAGS to the configure.ac file.

Add support for static plugins builds

Release 1.5.90

Move NLE to a dedicated GstPlugin

Summary: Allowing external user to directly use it

Reviewers: Mathieu_Du

Differential Revision: https://phabricator.freedesktop.org/D231

nle: Enhance debug logging

ges: Do not leak and uselessly create errors

And avoid parenthesis in GstObject names

ges: Fix how we handle layer vs layer-priority in the structured interface

tests: Do not use gst-structured-interface in the tests

It breaks $ make distcheck

Update mailing list in doap file

timeline: Remove transitions that can no fit into an auto transition

When activating auto transition mode

ges; Minor debug enhancement

assets: Avoid deadlock when done initialising asset

Avoid to hold the CACHE lock when setting the GTasks return values.

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

nleobject: Concider objects as 'inactive' when they have a duration == 0

track-element: Handle the case where we have only one keyframe set when interpollating keyframes

asset: Port use of deprecated GSimpleAsyncResult to GTask

Automatic update of common submodule

From f74b2df to 9aed1d7

track-element: Fix splitting bindings and add unit tests

structured-interface: Better handle CLOCK_TIME type from GstStructures

element: Implement a paste method

Allowing user to copy paste clips very easily

timeline: Disable movement that lead to 2 transition at a position

We should never let 3 objects to overlap at a same position, for that
we introduce a "rollback" feature and whenever such an editing happens,
we rollback object position to whatever it was before the move.

smart-video-mixer: Always keep a ref on the mixer pad

timeline-element: Add a method to get the TrackType it interacts with

API:
  + ges_timeline_element_get_track_types

timeline: Never create transitions between rippled objects

In case of groups, we can have track elements that do not belong
directly to the moved_trackelements but will be moved as others. Never
create transition to all object that have a start > moving group start.

video-transition: Use a SmartMixer as mixer

So that the frame position metas are parsed and taken into account

track-element: Add method to remove control binding

API:
  ges_track_element_remove_control_binding

ges: Handle absolute GstDirectControlBindings

framepositionner: Make use of the new CompositorPad.width/height

So that the scaling is done in the compositor and this way we can cleanly interpolate its value

timeline: Never change output media time when trimming start

+ Fix testsuite

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

track-element: Return right value when editing

We used to always return TRUE which was wrong

+ Fix testsuite and remove randomness from the tests

ges: Do not add a final gap at the end of track while rendering

It is not correct to force a black frame at the end of the rendered
video and it also leads to rendering issue with vpX encoders.

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

clip: Use container priority offset when setting children prios

Instead of trying to compute it ourself which might lead to wrong
behaviour when moving between layer.

+ Make sure that when we reset clip children priority (to make space
  for effects,) we update the container knowledge of priority offsets

clip: Fix track element priority computation

We were computing the priority offset taking the global MIN_NLE_PRIO
(which is a constant == 2 to make space for the mixing elements) instead
of the layer 'track element' relative priority, leading to very big
offsets on layer with a prio > 0. In the end it leaded to effects having
the same priority as the sources which leads to an undefined behaviour
in NLE.

uri-asset: Bump the discoverer timeout to 1 minute

We should by default avoid false timeouts

track: Give usefull name to compositions

nle: Port tests to the "commit" action signals

Now that nle_object_commit symbol is hidden, we can't use it
in the tests.

ges: Unbreeak API after renaming of GNL to NLE

ges: Hide more symbols of headers that are not installed

nle: Remove unused function

nle/nlecomposition.c:2471:1: error: unused function '_parent_or_priority_changed' [-Werror,-Wunused-function]
_parent_or_priority_changed (NleObject * obj, GNode * oldnode,
^

nle: Hide away symbols, they're supposed to be internal

Back to development

Release 1.5.2

nlesource: remove outdated comment

Automatic update of common submodule

From 6015d26 to f74b2df

pipeline: Handle rendering with disabled tracks

Summary:
The user might want to render only some media type of the timeline,
for example he wants to only render the audio part of the timeline.

It was failing as we were not connecting the track but were still trying
to 'render' it.

Depends on D153

Reviewers: Mathieu_Du

Reviewed By: Mathieu_Du

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

ges: Raise an error when the discoverer returns != RESULT_OK

And do not try to run the pipeline when that happens

uri-asset: Add a way to control discoverer timeout through envvar

Making it possible to run ges-launch test under valgrind for example

ges-launch: don't print random position/duration values at startup

Automatic update of common submodule

From d9a3353 to 6015d26

Automatic update of common submodule

From d37af32 to d9a3353

Automatic update of common submodule

From 21ba2e5 to d37af32

Automatic update of common submodule

From c408583 to 21ba2e5

Automatic update of common submodule

From d676993 to c408583

timeline: Never snap end when rippling

http://phabricator.freedesktop.org/T74

timeline: Never create transition between elements inside the moving context

http://phabricator.freedesktop.org/T74

group: Disconnect from old layer notify::priority when a clip is moved to a NULL layer

This means we need to properly track the layer a clip was in. We now
keep track of the various signal IDs in a dedicated structure and
keep a ref on the layer an object is in.

http://phabricator.freedesktop.org/T88

tools: Exit the app as it is a simgle instance app

And force exiting GstValidate when wanted

ges: remove dead code

Summary:
No need to recheck if error exists since it has already been checked by the
conditional above.

Coverity CID #1302832

Reviewers: thiblahute

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

ges: Handle trimming auto transitions

Meaning trimming neighbors.

+ And add a test

timeline: Use a simple GList to track auto transitions

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