validate: override-registry: fix structs list leak

The list returned by _lines_get_strutures() needs to be deeply freed.

validate: scenario: fix scenarios leak

validate: reporter: fix message leak

validate: report: don't shadow the GString variable

We were leaking the GString as it's freed outside of the block.

validate: report: fix GStrv leak

validate: report: fix GStrv leak

We borrow the content of the GStrv but were leaking the array itself.

launcher: try using gst.supp as valgrind suppressions file

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

validate: install gst.supp

Will be used when running tests inside Valgrind.

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

launcher: add valgrind support

Add a --valgrind option to gst-validate-launcher to run the tests inside
Valgrind and tune GLib's memory allocator accordingly.

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

validate: move scenarios to validate/scenarios/

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

validate:launcher: Fix small typo

validate:launcher: Make sure to show apps specific options in the help

validate:launcher: Fix test number printing

validate:launcher Rename _other_testsuite_for_tester

To _check_tester_has_other_testsuite

validate:launcher: Cache all the tests in the runner

This way we do not have to re ask all the test managers
what tests should be run.

validate:launcher: Add a way to simply run validate default tests on uris

Summary:
This allows us to easily run all the scenarios on a particular file doing:

  $ gst-validate-launcher validate --validate-check-uri file:///some/media/file.webm

Reviewers: Mathieu_Du

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

validate: keep executing actions even after linking up following execution

When linking actions execution without waiting on execution context, then
idle callback should keep being called so following action keep being
executed.

validate: Add more files to .gitignore

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

validate:scenario: Rename get_position to execute_next_action

That function was wrongly called and did not correspond to what it
actually does.

Revert "validate: add non flushing seek support"

This reverts commit 3ff55dcc3119b39e7c86044159db8bce49a2dc3a.

Regressions on the test server, apparently linked to this patchset.

Revert "validate: use segments to detect success of flushing seeks too"

This reverts commit c47cc7ba90e96ffaefe201087428ef448670f3be.

Regressions on the test server, apparently linked to this patchset.

Revert "validate: expect a buffer with discontinuity after a seek"

This reverts commit 87064b6994e36203b6976d436feda809068f1497.

Regressions on the test server, apparently linked to this patchset.

validate:launcher: Make sure TIMEOUTs do not get converted to ERROR

This was a regression introduced in c0e3d2e4f190fc9627897cc3d3d016448cb5dbe9

validate: expect a buffer with discontinuity after a seek

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

validate: use segments to detect success of flushing seeks too

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

validate: add non flushing seek support

and a couple scenarios using them

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

validate:launcher: Do not forget to install apps/__init__.py

It is a python module that should be usable by external apps/testsuites

validate:scenario: Handle not mandatory action types

Summary:
There is currently no way to handle the fact that action types
might be handled only by a specific application but not handling
this action types would not cause any difference for the good execution
of the scenario as a whole

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

validate:launcher: Better handle GST debug log outputs redirection

validate:launcher: First rely on the presence of criticals to set tests result

In the case of external applications they might not set their exist
code bases on the result of validate so we should rely on what
validates as to say first.

validate:report: Allow registering of issue types through the introspection

Fixing annotations and make GstValidateIssue refcounted

We break the ABI in that commit but I do not expect anyone to register
issue type outside GstValidate yet.

Add padding in the structures so we can avoid breaking the ABI again later.

validate:monitor: Do not requiere a GstObject as target

We can work with any GObject and that allows applications to write
monitors for other aspects too

validate:scenario: Alway execute sub action on action running SYNC

Move methods around to avoid needing on top prototypes

validate:docs: Rename gst-validate-launch to gst-validate-launcher

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

validate:scenario: Do not execute last sub action twice when ASYNC

validate:scenario: Properly print sub action as if they were main actions

validate:docs: Fix typos in Scenario File Format

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

validate:launcher Do not use git -C as it is relatively recent

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

validate: Build the preload so when possible

validate:launcher: Print the long help in less when possible

validate:launcher: Mention testsuite implementation in the help

validate: Define GST_PLUGIN_LDFLAGS as needed

validate: Determine development mode using git hash value

Development mode has been determined by whether the launcher is in git
repo
or not. This could be wrong when the launcher is installed to
subdirectory of other project's git repo, such as jhbuild. It is normal
to install compiled output to subdirectory of your jhbuild.

Changed logic gets the first commit hash of current git repo and
compares it with gst-devtools' the first commit hash.

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

validate:launcher: Fix typo s/FILE_EXTENDION/FILE_EXTENSION/g

validate:scenario: Fix GstValidateAction ABI adding a private structure

This way we can easily extend the structure and avoid needing using
a union and such

validate: Test buffer outside of received range.

Summary: As part of the preparation for a port to tracer.

Test Plan: This is a test, we won't test tests

Reviewers: tsaunier

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

codecanalyzer: fix codec detection with git master

The names might be 'MPEG-2 (Simple Profile)' now.
Shouldn't really rely on codec name strings here
in the first place, but use caps instead.

validate: Properly notify user about missing plugins

This way it is clear in gst-validate-launcher that the failure is due
to a missing plugin

validate: Fix wrong sizeof usage

sizeof(int) is always <= sizeof(gpointer)

validate: launcher: Use cElementTree for XML parsing

Using cElementTree instead of ElementTree speeds up parsing of media
descriptor files.

The total time spent parsing XML files drops from ~0.64 s to ~0.24 s,
leading to faster initialisation times for gst-validate-launcher.

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

validate: Print actions directly from the scenario

Avoiding user to have to print them in each and every action type
implementation.

This requires adding some API to prepare actions before printing them.
Preparing action in that case mean parsing the values contained in the
GstStructure parsing equations and setting back the actual value
afterward

API:
  * GstValidatePrepateAction
  * gst_validate_action_type_set_prepare_function

validate: Fix build on windows

Check where libgstvalidate.dll is installed and use that base folder to
figure out where GstValidate plugins are installed

validate: Do not forget to dist _full variant of scrubing scenarios

validate: Create a gst-validate-types.h header where we define types

And include it from validate.h.

This way we avoid to need to typedef GstValidateAction twice, which is
a C11 feature

validate-report: Fix valist usage

a va_list always 'exists' (it's a struct). It therefore can't be NULL
(and can't be tested)

Just use the regular print variant where appropriate.

validate:scenario: Document locking

validate:scenario: Make get_position happen on idle

Summary:
- Add a way to force action to be executed in their own GSource dispatch, disabling chain action execution

API:
  GstValidateScenario::execute-on-idle property

validate:scenario: Add a method to retrieve all remaining actions

Not only the next one as it was not making much sense!

API:
  - gst_validate_scenario_get_next_action
  + gst_validate_scenario_get_actions

validate:utils: Fix some annotations

validate:scenario: Allow link up of action executions for overriden types

Exposing a GstValidateActionType.overriden_type field
And properly expose gst_validate_execute_action

validate:reporter: Always print reports in the Gst debug system

validate: Misc fixes

validate: Properly annotate gst_validate_register_action_type*

It does not return any reference to the type

Revert "validate:scenario: Add a way to specify action structure size"

This reverts commit b976319ef7f977b8ce910c4b8aa1a843da3b264f.

Now that the exact same structure can be used to represent different
action types, we can not rely on the structure size to stuff
informations into the action.  Users should just make use of
GstMiniObject.qdata.

validate:scenario: We do not own any ref in GstValidateExecuteAction

And gst_validate_action_set_done might very well unref the last
reference to the action

validate: Make sure that the latest action type registration is kept

Avoiding to change the behaviour!

validate:scenario: Handle scenario repeat property with sub actions

And port change_state_intensive.scenario to it

validate:scenario: Add the notion of sub actions

Sub action will allow user to executed action *right* after the
previous action has been completed, meaning in the end that both
action can be considered as one single action.

+ Factor out a function to fill an GstValidateAction structure from a
  GstStructure
+ Factor out a function to set action playback time

validate: Document some env variable usage

validate: Add an API to cleanly register action type from plugins

API:
    gst_validate_register_action_type_dynamic

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

validate: Rename libfaultinjector to libgstvalidatefaultinjector

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

validate: Use plugin name as implementer_namespace when registering action type

And document it as a good practice as it will allow us to map plugins
and action types

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

validate: Move the fault_injection plugin to gst/plugins/

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

validate: Use an actual GstRegistry to track our plugins

Keeping everything internal for now

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

validate: Implement fault_injection as a Gs(tValidate)Plugin

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

validate:scenario: Add a way to specify action structure size

And return the register GstValidateActionType on registration

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

scenario: Do not concider we are seek_in_paused if executing a new action

The new action might change the position on purpose and we should not
fail in that case.

Also at that point we know the test of position after the seek has
been executed

+ Minor cosmetic fixes

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

validate:launcher:baseclasses: Avoid raising axception when all getting scenarios

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

launcher: Use gst-integration-testsuites FDO git repo

And make sure that people that were using the old repo get the origin
repo properly updated.

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

validate: launcher: Allow discovering scenario from full path

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

validate: scenario: Add a method to get the following action to be executed

API:
  + gst_validate_scenario_get_next_action

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

validate: Add helper functions ti print actions

API:
  + gst_validate_scenario_get_next_action
  + gst_validate_reporter_report_simple

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

validate: Minor documentation fixes

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

validate: Set seek_with_stop as needing at least 2secs media files

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

validate: Fix the check of action that can be *not* executed

The check was wrong and we ended up allowing seek actions to no be
executed.

API:
    GST_VALIDATE_ACTION_TYPE_NO_EXECUTION_NOT_FATAL

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

validate:scenario; Advertise action types that will be executed on addition

Adding a flag to the action type

And make that code thread safe.

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

validate:scenario: Add a Flag fore ActionType that need clocks sync

And cleanly use it to set the need-clock-sync field in
the scenario properties

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

validate: Add the notion of INTERLACED actions

An interlaced action is an action that will be executed ASYNC but
without that will not block following actions during its execution.
The action should be set to done later on at any point during the
execution of the scenario.

API:
  + GST_VALIDATE_EXECUTE_ACTION_INTERLACED
  + GST_VALIDATE_ACTION_TYPE_INTERLACED

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

validate: Add a way to retrieve register actoin type from outside

API:
  * GstValidateActionType
  * gst_validate_get_action_type

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

validate:launcher: Implement a FakeMediaDescriptor

This allows us to more cleanly implement Simple pipeline test
generation

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

validate:scenario: Make action->scenario public API

It can be usefull for action type implementers

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

validate:scenario: Add a disable-plugin action type

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

validate: Properly advertise the wait action as ASYNC

And add some printing when executing the set-property action

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

validate: Enhance support for simple pipeline test generation

The GstValidatePipelineGenerator was quite limited in term
of configuration for user who just want to specify pipelines
to run with/without scenario.

Enhance the API so that we can properly configure that.

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

validate: Wait for switch-track to complete before executing next action

This action type can take some time, we need to make sure that the
combiner/input-selector element properly pushed a buffer marked
as DISCONT to concider the action is done.

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

validate:scenario: Avoid waiting for 50ms between actions

We should be able to execute the next action as soon as the previous
one is fully completed, make sure the code tries to do that and does
not artificially add some waiting time.

And make sure if the gst_validate_action_set_done is called from outside
our execution thread, we do not try to execute anything

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

scenario: Add a signal to notify user when the scenario is DONE executing

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

validate: launcher: Print test name in Result

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

validate: launcher: Add option to run tests in parallel

Patch 4/4 to implement parallel test execution.

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

validate: launcher: Print test number on result

With parallel test execution, it will be hard to track which result
relates to which test. Therefore, the test number should be printed
along with the results as well.

Patch 3/4 to implement parallel test execution.

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