nle: Drop tags getting out of the composition

Those tag are meaningless in for the new stream created by the composition

First step toward fixing T3070

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

timeline: Properly compute the end of groups when checking snapping

Computation was not taking into account the fact that the start of
the element being moved could be at the middle of a group and not
necessarily at the start!

Fixes T7544

Reviewed-by: Alex Băluț <alexandru.balut@gmail.com>
Differential Revision: https://phabricator.freedesktop.org/D1282

ges: Handle moving groups with effects inside

We were only concidering that we should let the group handle moving
transitions when changing transitions but in fact as soon as a
transition is happenning between two clips that are in a same group
the group properly handles moving the transition, so let the
group do its job.

Fixes T7543

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

ges: Deprecate GESTimelineElement::priority writability

GESLayer is now responsible for setting clips priorites. Also
GESClip top effects priorities are now set by the
ges_clip_set_top_effect_index method, the user should never call
ges_timeline_element_set_priority as it will anyway be overriden
by GES itself.

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

layer: Handle operation priorities

All operations should have higher priorites and sources should be
on top of those. We now first set the operations priorities in
a first pass and then stack sources on top of those.

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

ges: transition: Make crossfade fade out at the same time as it fade in

Until now fade out was just fading in the new clip, but this is not
correct and crossfade should at the same time fade out while fading
in.

Fixes https://phabricator.freedesktop.org/T3451

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

layer: Make sure to resync priorities on commit

In case effects have been added priorites might become wrong,
but until the timeline is not commited, it does not matter.

Make sure all priorities are correct before commiting compositions

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

Finally move clip priority handling to GESLayer.

Fix all tests as we now have 1 priority inside the layer
dedicated to transitions (basically no source clip will
ever have a priority of 0 inside a layer).

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

clip: Make top effect priority inside the clip priority range

And simplify the way we start computing children priority
making min_priority already relative to the clip itself.

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

tests_: timelineedition: Fix test now that we ripple from start and not from end

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

nlecomposition: Don't try to seek on an empty stack

We would seek on a NULL pad then, which gives ugly assertions.

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

nleurisource: Always provide a srcpad

By putting uridecodebin into a bin with a ghostpad. Without this,
nlesource tries to get a srcpad too early (before uridecodebin added
one) and everything fails miserably.

This has to be fixed properly in nlesource at some point, by properly
handling dynamically added pads. Currently they can only work if they
are added in states <= READY, which is not the usual case.

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

nlesource: Fail prepare() if no valid source pad is found

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

ges-launch: Be a bit more agressive sanitizing arguments

Otherwise GstStructure might fail parsing some fields
containing brackets

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

configure: Depend on gstreamer 1.9.2.1

timeline: Make get_groups public

Had to separate timeline_emit_group_added from timeline_add_group
to avoid emitting group-added when the project is being loaded.

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D1302

tests_: Make sure child-removed is emitted when ungrouping

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D1301

timeline: Fix documentation

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D1300

Remove obsolete dependency

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D1299

Automatic update of common submodule

From b18d820 to f980fd9

Automatic update of common submodule

From f49c55e to b18d820

tests: Move -DGES_TEST_FILES_PATH to common_cflags

As it is needed to build the utils lib

tests: Fix the way we get tests assets

Using __FILE__ won't work properly with meson.

test:validate: Handle new expected_failures Test argument

meson: Do not build libges against libcheck

meson: Properly find where other GStreamer plugins are when using subprojects

Revert "validate: Blacklist racy tests"

This reverts commit ce35412ff260fbd6e07b374bc3ca677053c277e0.

https://bugzilla.gnome.org/show_bug.cgi?id=769894 has been fixed

meson: Handle building the gir file when used as subproject

Add support for building GIR when used as subproject
Add an option to disable GIR generation
And bump version to 1.9.2

Back to development

Release 1.9.2

meson: doc: Fix building documentation when using subprojects

And check the presence of gtk-doc before building the documentation

ges: Rename parse_lex.h to ges-parse-lex.h

Fixes the build and makes it consistent with the meson build system.

Add support for Meson as alternative/parallel build system

https://github.com/mesonbuild/meson

ges-track-element: fix typos in control_binding_removed signal declaration

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

validate: Blacklist racy tests

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

ges-timeline: Demote some debugging statements

locking should be in a lower level to avoid too many messages

timeline-element: Reset pointer after freeing

dispose can be called multiple times, make sure we don't call functions
on free'd pointers.

ges: Do not rescale videos if the track aspect ratio changes

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

ges: Let the compositor do the scaling if mixing is enabled

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

tests:validate: Also test opus and theora in OGG

validate: Start also testing jpeg encoding

timeline: Keep transitions when moving the moving context between layers

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

ges-launcher: don't leak project uri string

ges_project_get_uri returns a cloned string so it should
be free'd after usage.

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D381

timeline: Ripple from start of clips and not the end.

Fixes https://phabricator.freedesktop.org/T7503

enums: Fix absolute text overlay alignment value

It needs to be in sync with GstBaseTextOverlayHAlign order.

title-source: Properly implement GESTimelineElement->lookup_child

GESTrackElement->lookup_child is deprecated and should be avoided
as much as possible.

clip: Reimplement look_child and iterate over children if needed

Otherwise in the case where children reimplement lookup_child to
handle some property renaming lookup fails.

tools: Fix printing commands help

ges:validate: Check that no extra decoding happens

uri-sources: Make sure to set decodebin 'caps' property

Fixes a regression where we decode streams twice,
this was introduced when we started creating NLE
object at GESTrackElement construct time.

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

timeline: Emit snap-ended on commit if needed

Commiting the timeline means that the current operations on the clips are over,
so we should concider snapping as done at that point

Fixes T7499

fixed ges-version.h not found issue during out of tree build

image-source: Do not concider inpoints

We have no restriction on inpoint for Images

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

tests:validate: Better choose tested rendering formats

Automatic update of common submodule

From f363b32 to f49c55e

Back to development

Release 1.9.1

ges-launch: unref GApplication properly

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

project_: improve get_uri doc

The return value of ges_project_get_uri should be freed
after usage.
Differential Revision: https://phabricator.freedesktop.org/D1142

Automatic update of common submodule

From ac2f647 to f363b32

tests: Make sure to run python tests against the build in tree

ges: Don't remove track elements from clips when removing from layer

And reuse the same previously created element when adding the clip
back to a layer, avoiding losing all setting done on clip children
in that situation

This is a behaviour change but previous behaviour was actually totally
unexpected and people working around that weird behaviour will moste
probably not care about that change

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

title_: Do not forget to link up child_added/removed vmethod

Otherwise effect handling is broken

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

auto-transitions: Do not remove auto transitions when moving neighboor from the same group

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

ges: fix various leaks with usage of ges_timeline_element_lookup_child

Some callers forgot to unref out child, pspec or both leading to leaks.

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

video-track: don't leak restriction caps in _sync_capsfilter_with_track()

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

ges-uri-asset: GstDiscoverer can return a valid info but a non-OK result, consider this an error

The asynchronous case in ges_uri_clip_asset_request_async() already considered
it an error, do the same in ges_uri_clip_asset_request_sync().

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

nlecomposition: Fix race condition in seek handling causing deadlocks

We might receive another seek from the application while the action task is
handling a previous seek (and thus setting seeking_itself to TRUE). To prevent
this seek to go through directly instead of being added as an action, also
check if the seek event was received from our action task thread or some other
thread.

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

ges: Emit GESMetontainer::notify-meta even if value is unset

ges: Allow passing `NULL` as a value to ges_meta_container_set_meta

Fixes T7430

g-i: pass compiler env to g-ir-scanner

It's what introspection.mak does as well. Should
fix spurious build failures on gnome-continuous
(caused by g-ir-scanner getting compiler details
via python which is broken in some environments
so passing the compiler details bypasses that).

uri-clip: make uri parameter of ges_uri_clip_new () const

To avoid compiler warning when using const string to create a new
GESUriClip as string is not modified and only passed to functions which
take a const string.

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

framepositionner: add a weak ref on track element to know when it is finalized

Otherwise if frame positionner is disposed after track element has been
finalized, it will raise a critical message because we will try to
disconnect a signal handler on a freed track element object.

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

audio-source: unref private capsfilter reference on dispose

Otherwise a capsfilter reference will be leaked since it has been got
using gst_bin_get_by_name.

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

audio-source: fix indentation

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

nlecomposition: ensure elements pending to be added are not leaked

When nlecomposition is finalized with pending add action or io,
associated elements are not unreffed as they should since caller gives
us the reference when calling gst_bin_add causing them to be leaked.
So to make sure we don't leak a reference on element when adding one to
the bin, each stage (action and pending_io) hold a reference on element
and release it when stage is done.

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

nlecomposition: fix nle_composition_remove_object info message

We don't add internal bin, we remove it.

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

ges: add some g-i annotations according to documentation

Mainly (transfer xxx) and (nullable). Also fix some typo.

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

asset: fix ges_asset_set_proxy() return value documentation

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

element: Also accept GParamSpec.owner_type name as a child property prefix

Makes it simpler for python users to be able to retrieve children
properties iterating over them.

track-element: gi: skip now deprecated children property getter/setter

Those are implemented with the exact same API at the GESTimelineElement
level now, and user of those APIs with high level languages will get the
exact same API.

formatter: Prefix all children properties in the XML formatter

Otherwise it will fail on properties that are mandatorily prefixed
like the newly added deinterlacing properties

ges: Remove timeline_emit_group_removed which slipped in the API by mistake

This is formally an API break but I am sure no one ever used that and
we should make sure the method is removed as soon as possible because
it has no reason to be exposed.

container: Handle setting children properties that need prefixing

video-source: Expose deinterlace-[fields, mode, tff] child properties

Letting some control over the deinterlacing to the users

ges: fix misc g-i annotations

g-i: use only "ges/ges.h" as c-include for introspection

This is the only header which shall be included by user. Otherwise some
language using gir to generate binding, e.g Vala, will includes all
headers files in alphabetical order which causes compilation errors due
to incomplete type.

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

timeline: rename "track-element" to "track_element" in select-tracks-for-object documentation

because "track-element" is not a valid identifier for a parameter and
will cause generated binding using GIR to be invalid. For instance in
Vala.

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

video-source: Do not ever plugin avdeinterlace

It is not feature compatible with deinterlace and is not safe to use

ges: #include glib.h for G_BEGIN_DECLS

ges-multi-file-source: Fix case of standard gobject macros

ges: Add some more functions to the docs and don't scan internal headers

Someone still should look through the unused.txt for more things to add or
hide.

ges: Add G_BEGIN_DECLS around all relevant declarations in headers

title: Do not concider inpoints

It does not make sense for titles
Handle element with no inpoint handling in the timeline

Fixes https://phabricator.freedesktop.org/T7319

Automatic update of common submodule

From 6f2d209 to ac2f647

tests: Rename positionner to positioner in the tests too

ges: Fix typo by renaming positionner to positioner

It's fortunately private API

framepositionner: Initialize all fields of the meta during initialization

GstMetas are not allocated with all fields initialized to zeroes.

ges: Do not try to set read only properties

When copying and splitting clips

Fixes T7375

transition: Lower done some debug loggin level