validate:scenario: Fix 'duration' property of the pause action

We preparse it into and set it as GstClockTime in the
structures so make sure to use them as such.

validate:launcher: Concider unset MediaDescriptor duration has 'infinite'

validate: ignore a pixman leak which is fixed in master

http://phabricator.freedesktop.org/D128

validate: use a bigger hammer to ignore mesa related leaks

Looks like some tests are hitting a slightly different code path in udev but
the root bug is the same.

http://phabricator.freedesktop.org/D128

codecanalyzer: minor style fix

codecanalyzer: don't try to free uninitialized pointers

xml_files_path and hex_files_path variable are not initialized.
There are chances that corruption happens when uninitialized
variables are freed, so init them to NULL before use.

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

codecanalyzer: run gst-indent on code

validate: Update autogen.sh

And add a gst-validate.doap file.

Update common submodule

validate: use -debug versions of bins when running from source

Summary:
Those versions are using rpath instead of libtool's wrappers and so will be
faster to start and won't confuse valgrind.

Reviewers: thiblahute

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

move scenarios to data/scenarios

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

validate: add gstvalidate.supp valgrind suppression file

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

validate: scenario: fix structure and action leak

validate: report: fix GString leak when early returning

validate:pipelinemonitor: Print position only when in state >= PAUSED

Reviewers: Mathieu_Du

Reviewed By: Mathieu_Du

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

validate:launcher: Use full HD screen as default screen size in xvfb

validate:launcher: Add a way to run tests without displaying the output

Summary:
Adding a --no-display option and running Xvfb virtual frame buffer X
server.

Reviewers: Mathieu_Du

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

validate:launcher: Error out if valgrind is not available on the system

Summary:
When the user wants to use valgrind, make sure it is present on the
system before doing anything

Reviewers: gdesmott

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

validate:scenario: Stop scenario execution on stop action

And document it properly.

Summary:
The stop action was defined as "setting state to NULL" but
its actual goal is to stop the execution of the scenario. Make sure
that the scenario will not try to execute other actions when that
one has been executed.

Reviewers: Mathieu_Du

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

validate: Gracefully handle absence of TAG on streams

Summary: And do not segfault when it happens!

Reviewers: Mathieu_Du

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

validate:launcher: Pass GST_VALIDATE_SCENARIO to the subprocess env only

Summary:
And make sure to remove it from the env if the user has it in its main
environment.

Without that commit we ended up passing scenarios from previous tests
to the following ones where None were specified.

Reviewers: Mathieu_Du

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

validate: don't pass NULL to gst_caps_copy()

validate: use GstMediaDescriptorWriter as log handler

Allow us to catch warnings when running gst-validate-media-check-1.0.

validate: media-descriptor-writer: fix string leaks

validate:launcher: Handle git error properly

'OSError' exception is emitted but not handled properly when git is not
installed on running system.

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

validate: set GST_GL_XINITTHREADS

This ensure that XInitThreads is called and so gl contexts are properly
initialized.

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

Signed-off-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>

validate:launcher: Make validate the only default testsuite

Reviewers: Mathieu_Du

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

validate:launcher: Let the responsibility to update asset to the testsuite

Summary:
It makes it easier to make sure that the assets needed for a specific
testsuite are available when needed

Reviewers: Mathieu_Du

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

validate:tools: EOS handling is the responsibility of the scenario

Summary: If any scenario set

Reviewers: Mathieu_Du

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

validate:scenario: Add support for waiting on signals and messages

Reviewers: Mathieu_Du

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

validate:scenario: Report disabling plugin issues

Summary:
+ typedef GstValidateActionReturn so it can be used in the introspection
+ Add GST_VALIDATE_EXECUTE_ACTION_ERROR_REPORTED which should be used
  to tell Validate that something wrong happened so the sub action
  won't be executed, but that it should not report an error itself
  as it has already been handled in the action function.

Reviewers: Mathieu_Du

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

validate:scenario: Allow execution of disable-plugin as a config action

Summary:
And fix a bug where config actions were added to the list of action even
if they had already been executed

Reviewers: Mathieu_Du

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

validate:scenarios: Set seek_forward_backward min-media-duration=45

Reviewers: Mathieu_Du

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

validate:launcher: Set more env variable in the launcher command desc

Summary:
Adding if present:
  * LD_PRELOAD
  * DISPLAY
  * GST_VALIDATE_CONFIG
  * GST_VALIDATE_OVERRIDE

+  enhance the add_env_variable method to more easily set envvar from
current value

Reviewers: Mathieu_Du

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

validate: Add a method to easily get plugin configuration

Reviewers: Mathieu_Du

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

validate:plugins: Add support to all GApplication as a test apps

Summary:
Add a very simple plugin that will allow any GApplication to easily be
used with GstValidate using the LD_PRELOAD feature

Reviewers: Mathieu_Du

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

validate: let structs_from_filename be exported.

Summary: It is useful for plugins too

Reviewers: Mathieu_Du

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

validate:scenario: Update Action.repeat field when needed

Summary: And print the current repeat value of the action that have such a field

Reviewers: Mathieu_Du

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

validate:scenario: Alway execute a 'quit' action on EOS

Summary: Making scenario more usable with LD_PRELOAD

Reviewers: Mathieu_Du

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

validate:scenario: Set the main action structure in fill_structure

Reviewers: Mathieu_Du

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

validate:launcher: Use GST_VALIDATE_SCENARIO envvar to set scenarios

Summary:
Instead of concidering all apps will have a --set-scenario argument
which is not going to be the case as soon as we run the tests through
LD_PRELOAD

Reviewers: Mathieu_Du

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

validate:launcher: Move get_current_position from GstValidatePipelineTest to GstValidateTest

This is where it belongs

Reviewers: Mathieu_Du

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

validate: fix typo in scenario file format docs

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

validate: scenario: fix caps leak

validate: media-descriptor: fix filenode->caps leak

validate: transcoding: don't create a second mainloop

validate: transcoding: don't leak the requested sinkpad from decodebin

validate: pad-monitor: fix caps leak

Don't create othercaps when early returning.

validate: GstValidateMediaCheckTest should inherit from GstValidateTest

validate:launcher: Install config.py for non-development mode

Running installed gst-validate-launcher aborted with the following error.

File "lib/gst-validate-launcher/python/launcher/baseclasses.py", line 28, in <module>
  import config
ImportError: No module named config

This is because config.py is added but not installed
in ba6d209b3fd062f4e6bd889f81f1213cc12339ec.

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

validate:launcher: Show timeout seconds for timeout result message

Current timeout message doesn't show how many seconds a test took and
it is timeouted by normal timeout or hard timeout.

This patch changes the message like following.

1. normal timeout
old : validate.http.playback.reverse_playback.raw_video_mov: Timeout (Application timed out)
new : validate.http.playback.reverse_playback.raw_video_mov: Timeout (Application timed out: 120 secs)

2. hard timeout
old : validate.http.playback.reverse_playback.raw_video_mov: Timeout (Application timed out)
new : validate.http.playback.reverse_playback.raw_video_mov: Timeout (Hard timeout reached: 600 secs)

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

validate:launcher: Fix wrong test number with -j option

When '-j n' option is given, first n tests print test number 0.
This is caused by test_num part of 919db986052602dca452f05e284cfc857302d4f0.

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

validate:launcher: Avoid depending on PyGObject

Summary:
And rely on our knowledge of the configuration to figure out where the
suppression file has been installed

Reviewers: gdesmott

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

validate: media-descriptor-writer: don't leak info and streaminfo

validate: media-check: don't leak output_file and expected_file

validate: check VALGRIND_ERROR_CODE in Test as well

We were doing it only in GstValidateTest which was overriding the default
implementation.

validate: don't increase hard_timeout is if it's None

Some tests, like the media check ones, have None as hard_timeout.

validate: increase VALGRIND_TIMEOUT_FACTOR

5 wasn't enough for my poor laptop.

validate: increase the normal timeout as well when using valgrind

validate: media-descriptor: fix caps leak

gst_pad_get_current_caps() returns a reffed caps.

also call gst_deinit() in media-check and transcoding

More valgrind friendly.

validate: raise an error if valgrind detected issues

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

call gst_deinit() when we are done

More valgrind friendly.

validate: call gst_bus_remove_signal_watch()

We are supposed to call gst_bus_remove_signal_watch() for each gst_bus_add_signal_watch() call to prevent leaks.

validate: media-descriptor-parser: fix string leak

_set_content() doesn't actually consume @content so the caller is responsible
freeing it.

validate: pad-monitor: fix buffers list leak

validate: pad-monitor: fix caps leak

validate: fix a bunch of GstBus leaks

validate: store valgrind logs to its own file

validate: report: fix invalid read when destroying Report

Summary:
@report was invalid when we were trying to clear the mutex.

validate: scenario: remove weak pointer when destroying action

Free an invalid read when the scenario is destroyed after the action.

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

validate: report: fix invalid read when destroying Report

@report was invalid when we were trying to clear the mutex.

validate: fix string arguments leaks

We are responsible of freeing the string arguments parsed by GOptionContext.

validate: scenario: don't borrow @structure in _fill_action()

@structure was borrowed in some code path and wasn't in some other. Make it
clearer, and fix a leak, by always copying it.

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