platform/upstream/gstreamer.git
9 years agovalidate: Add a validate ssim plugin
Thibault Saunier [Mon, 8 Jun 2015 15:10:50 +0000 (17:10 +0200)]
validate: Add a validate ssim plugin

Summary:
 + Bump gst-video dependency to 1.4 as we need GstVideoConvert

 Depends on D213: validate: Mark gst_validate_report a G_GNUC_PRINTF

Reviewers: Mathieu_Du

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

9 years agovalidate: Mark gst_validate_report a G_GNUC_PRINTF
Thibault Saunier [Wed, 3 Jun 2015 10:43:52 +0000 (12:43 +0200)]
validate: Mark gst_validate_report a G_GNUC_PRINTF

Summary:
And fix the issue it raised

Reviewers: Mathieu_Du

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

Depends on D211

9 years agovalidate: Add a gst-validate-images-check tool
Thibault Saunier [Mon, 25 May 2015 11:41:04 +0000 (13:41 +0200)]
validate: Add a gst-validate-images-check tool

Summary:
That is a new tool that uses ssim algorithm to compare images

+ Add a GstValidateVideo internal library adding an helper Gssim class

Depends on D210

Reviewers: Mathieu_Du

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

9 years agovalidate:utils: Add a utility to get a GstClockTime from a structure
Thibault Saunier [Wed, 27 May 2015 17:35:15 +0000 (19:35 +0200)]
validate:utils: Add a utility to get a GstClockTime from a structure

Summary:
Properly handling the different types that can represent ClockTime

Make use of it in gst_validate_action_get_clocktime

API: gst_validate_utils_get_clocktime

Depends on D209

Reviewers: Mathieu_Du

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

9 years agovalidate: Properly clear the overrides registry on deinit
Thibault Saunier [Tue, 26 May 2015 16:45:45 +0000 (18:45 +0200)]
validate: Properly clear the overrides registry on deinit

Summary: Depends on D208

Reviewers: Mathieu_Du

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

9 years agovalidate: Add a way to check whether Validate is initialized
Thibault Saunier [Tue, 26 May 2015 11:58:15 +0000 (13:58 +0200)]
validate: Add a way to check whether Validate is initialized

Summary:
API:

  gst_validate_is_initialized

Depends on D207

Reviewers: Mathieu_Du

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

9 years agovalidate:override: Add a vmethod to check whether a monitor can attach it
Thibault Saunier [Tue, 26 May 2015 13:57:29 +0000 (15:57 +0200)]
validate:override: Add a vmethod to check whether a monitor can attach it

Summary: Depends on D206

Reviewers: Mathieu_Du

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

9 years agovalidate:override-registry: Make use of gst_validate_element_has_klass
Thibault Saunier [Tue, 26 May 2015 10:04:02 +0000 (12:04 +0200)]
validate:override-registry: Make use of gst_validate_element_has_klass

Summary:
+ Fix a minor mixup bug between klass_overrides and name_overrides

Depends on D205

Reviewers: Mathieu_Du

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

9 years agovalidate:override: Make overrides GObjects
Thibault Saunier [Tue, 26 May 2015 10:03:25 +0000 (12:03 +0200)]
validate:override: Make overrides GObjects

Summary:
This way we can subclass them getting a proper
context in the various override methods.

Depends on D204

Reviewers: Mathieu_Du

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

9 years agovalidate: Move plugins to the toplevel directory
Thibault Saunier [Mon, 25 May 2015 16:52:34 +0000 (18:52 +0200)]
validate:  Move plugins to the toplevel directory

Summary:
Otherwise we end up with circular / complicated dependencies between
Validate, its libraries, and the plugins

Depends on D203

Reviewers: Mathieu_Du

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

9 years agovalidate:scenario: Round up computed ClockTime values
Thibault Saunier [Wed, 27 May 2015 14:41:00 +0000 (16:41 +0200)]
validate:scenario: Round up computed ClockTime values

Otherwise we end up with rounding error and instead of
seeking to 0.1 we seek to 0.09999999999 for example

Reviewers: Mathieu_Du

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

9 years agovalidate:runner: Add a method to force exiting the runner
Thibault Saunier [Wed, 27 May 2015 11:18:33 +0000 (13:18 +0200)]
validate:runner: Add a method to force exiting the runner

This method is similar to runner_printf() but can be used
only once. The user needs to make sure all the pipeline
are in NULL state when this is called.

The method emits a "STOPPING" signal and at that point
overrides or monitors should do extra processing/checks if
needed.

+ Make use of it everywhere where it makes sense.

API:
  gst_validate_runner_exit
  GstValidateRunner::stopping signal

9 years agopad-monitor: Check that an ERROR GstMessage has been posted on GST_FLOW_ERROR
Olivier Crête [Wed, 3 Jun 2015 00:25:56 +0000 (20:25 -0400)]
pad-monitor: Check that an ERROR GstMessage has been posted on GST_FLOW_ERROR

Summary:
Before returning GST_FLOW_ERROR, an element must post an ERROR GstMessage,
enforce that.

Reviewers: thiblahute, Mathieu_Du

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

9 years agovalidate: remove unused assignment
Luis de Bethencourt [Wed, 3 Jun 2015 10:49:58 +0000 (11:49 +0100)]
validate: remove unused assignment

Reviewers: thiblahute

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

9 years agoRevert "validate-utils: simplify _read_builtin ()"
Olivier Crête [Tue, 2 Jun 2015 20:46:15 +0000 (16:46 -0400)]
Revert "validate-utils: simplify _read_builtin ()"

This breaks the fast_forward scenario parsing.

This reverts commit 0cfff156b1d7013174652cdd25d3ad3f0571813e.

9 years agovalidate-utils: clean error handling in _file_get_lines ()
Luis de Bethencourt [Fri, 29 May 2015 16:40:26 +0000 (17:40 +0100)]
validate-utils: clean error handling in _file_get_lines ()

9 years agovalidate-utils: simplify _read_builtin ()
Luis de Bethencourt [Fri, 29 May 2015 15:29:44 +0000 (16:29 +0100)]
validate-utils: simplify _read_builtin ()

9 years agovalidate-utils: sqrt(-1.0) leads to undefined result
Vineeth T M [Fri, 29 May 2015 14:40:04 +0000 (15:40 +0100)]
validate-utils: sqrt(-1.0) leads to undefined result

Using sqrt of -1 is not valid and leads to undefined results.
When comparing the return value of the fucntion in validate-scenario,
it is being checked with ret == -1, so it makes sense to just return -1 in error case.

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

9 years agovalidate:scenario: Make sure to actually test position to execute actions
Thibault Saunier [Wed, 20 May 2015 11:57:55 +0000 (13:57 +0200)]
validate:scenario: Make sure to actually test position to execute actions

9 years agovalidate: add valgrind ignore supps for theora encoder
Guillaume Desmottes [Fri, 15 May 2015 12:45:04 +0000 (14:45 +0200)]
validate: add valgrind ignore supps for theora encoder

Reviewers: thiblahute

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

9 years agovalidate: add more H264 valgrind supp rules
Guillaume Desmottes [Fri, 15 May 2015 12:26:35 +0000 (14:26 +0200)]
validate: add more H264 valgrind supp rules

Reviewers: thiblahute

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

9 years agovalidate:gtk: Use event->type directly
Thibault Saunier [Fri, 15 May 2015 10:57:49 +0000 (12:57 +0200)]
validate:gtk: Use event->type directly

gdk_event_get_event_type was introduced in Gtk 3.10 only

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

9 years agovalidate:launcher: extra_env_variables is a dictionnary
Thibault Saunier [Thu, 14 May 2015 15:43:40 +0000 (17:43 +0200)]
validate:launcher: extra_env_variables is a dictionnary

9 years agovalidate:launcher: Always set the protocol when creating a GstValidateMediaDescriptor
Thibault Saunier [Wed, 13 May 2015 13:30:23 +0000 (15:30 +0200)]
validate:launcher: Always set the protocol when creating a GstValidateMediaDescriptor

Summary: Depends on D174

Reviewers: Mathieu_Du, gdesmott

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

9 years agovalidate:launcher: Allow passing extra env variables to the tests
Thibault Saunier [Wed, 13 May 2015 13:29:43 +0000 (15:29 +0200)]
validate:launcher: Allow passing extra env variables to the tests

Summary: Depends on D173

Reviewers: Mathieu_Du, gdesmott

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

9 years agovalidate: overrides: Allow using regex for named overrides
Thibault Saunier [Wed, 13 May 2015 13:27:08 +0000 (15:27 +0200)]
validate: overrides: Allow using regex for named overrides

Summary:
And minor fixes

Depends on D172

Reviewers: Mathieu_Du, gdesmott

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

9 years agovalidate:scenario: Handle action execution after pipeline destruction
Thibault Saunier [Wed, 13 May 2015 10:18:18 +0000 (12:18 +0200)]
validate:scenario: Handle action execution after pipeline destruction

Summary:
It is possible to keep executing actions after the pipeline
has been destroyed.

API:
  GST_VALIDATE_ACTION_TYPE_DOESNT_NEED_PIPELINE

Depends on D171

Reviewers: Mathieu_Du

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

9 years agovalidate:scenario: Add a macro to get ActionType from an Action
Thibault Saunier [Wed, 13 May 2015 10:16:57 +0000 (12:16 +0200)]
validate:scenario: Add a macro to get ActionType from an Action

Summary: Depends on D170

Reviewers: Mathieu_Du

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

9 years agovalidate:scenario: Factor out code to check position
Thibault Saunier [Wed, 13 May 2015 10:13:17 +0000 (12:13 +0200)]
validate:scenario: Factor out code to check position

Summary:
Making simpler to follow the execute_next_action function.

Depends on D169

Reviewers: Mathieu_Du

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

9 years agovalidate:pipeline-monitor: Stop printing position when not possible
Thibault Saunier [Wed, 13 May 2015 09:27:25 +0000 (11:27 +0200)]
validate:pipeline-monitor: Stop printing position when not possible

Summary:
If from anything >= PAUSED to anything <= READY we can not query
pipeline position, so do not try to.

Depends on D168

Reviewers: Mathieu_Du

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

9 years agovalidate:scenario: Rename 'get_position_id' to 'execute_actions_source_id'
Thibault Saunier [Wed, 13 May 2015 09:20:42 +0000 (11:20 +0200)]
validate:scenario: Rename 'get_position_id' to 'execute_actions_source_id'

Summary: Depends on D167

Reviewers: Mathieu_Du

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

9 years agovalidate: Do not unref twice the same list
Thibault Saunier [Tue, 12 May 2015 10:07:13 +0000 (12:07 +0200)]
validate: Do not unref twice the same list

Summary: Depends on D166

Reviewers: Mathieu_Du

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

9 years agovalidate:scenario: Add a way to specify the pipeline on which a scenario applies
Thibault Saunier [Tue, 12 May 2015 08:58:19 +0000 (10:58 +0200)]
validate:scenario: Add a way to specify the pipeline on which a scenario applies

Summary:
From within the scenario itself.

Depends on D165

Reviewers: Mathieu_Du

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

9 years agovalidate: Avoid depending on Gst 1.4
Thibault Saunier [Tue, 12 May 2015 10:04:52 +0000 (12:04 +0200)]
validate: Avoid depending on Gst 1.4

Summary: Depends on D117

Reviewers: Mathieu_Du

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

9 years agovalidate: Add a gtk plugins that implements action types relative to Gtk
Thibault Saunier [Tue, 21 Apr 2015 13:29:15 +0000 (15:29 +0200)]
validate: Add a gtk plugins that implements action types relative to Gtk

Summary:
Currently the only supported action is gtk-put-event allowing press and
release keyboard keys.

Reviewers: Mathieu_Du

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

9 years agovalidate: Always git submodule update from the toplevel directory
Thibault Saunier [Tue, 12 May 2015 07:55:58 +0000 (09:55 +0200)]
validate: Always git submodule update from the toplevel directory

Otherwise it fails with older git versions

9 years agovalidate: Reshape documentation
Emanuele Aina [Sat, 9 May 2015 14:28:20 +0000 (16:28 +0200)]
validate: Reshape documentation

Fix some errors, use more Docbook tags and split each command reference
in its own file.

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

9 years agovalidate: Go back to the validate dir after submodule init
Emanuele Aina [Sat, 9 May 2015 14:23:06 +0000 (16:23 +0200)]
validate: Go back to the validate dir after submodule init

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

9 years agovalidate: ignore x264 valgrind errors
Guillaume Desmottes [Mon, 11 May 2015 15:08:37 +0000 (17:08 +0200)]
validate: ignore x264 valgrind errors

Summary: The x264 code is pretty hardcore so I just opened a bug for now.

Reviewers: thiblahute

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

9 years agovalidate: disable QOS features when running with valgrind
Guillaume Desmottes [Fri, 8 May 2015 14:33:50 +0000 (16:33 +0200)]
validate: disable QOS features when running with valgrind

Reviewers: thiblahute

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

9 years agovalidate: move element_has_klass() to utils
Guillaume Desmottes [Mon, 11 May 2015 12:24:32 +0000 (14:24 +0200)]
validate: move element_has_klass() to utils

Reviewers: thiblahute

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

9 years agovalidate: rename get_valgrind_suppression_file()
Guillaume Desmottes [Mon, 11 May 2015 10:22:25 +0000 (12:22 +0200)]
validate: rename get_valgrind_suppression_file()

Summary:
This function is actually not specific to valgrind so we can make it more
generic.

Reviewers: thiblahute

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

9 years agovalidate: allow to pass more than one file to GST_VALIDATE_CONFIG
Guillaume Desmottes [Mon, 11 May 2015 10:01:56 +0000 (12:01 +0200)]
validate: allow to pass more than one file to GST_VALIDATE_CONFIG

Reviewers: thiblahute

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

9 years agovalidate: gst_validate_plugin_get_config() return 'core' conf if plugin is NULL
Guillaume Desmottes [Mon, 11 May 2015 09:47:47 +0000 (11:47 +0200)]
validate: gst_validate_plugin_get_config() return 'core' conf if plugin is NULL

Reviewers: thiblahute

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

9 years agovalidate: add gst_validate_deinit()
Guillaume Desmottes [Mon, 11 May 2015 11:54:15 +0000 (13:54 +0200)]
validate: add gst_validate_deinit()

Reviewers: thiblahute

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

9 years agovalidate: don't leak not maching config structures
Guillaume Desmottes [Mon, 11 May 2015 12:25:49 +0000 (14:25 +0200)]
validate: don't leak not maching config structures

Reviewers: thiblahute

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

9 years agovalidate: factor out create_config()
Guillaume Desmottes [Mon, 11 May 2015 09:08:36 +0000 (11:08 +0200)]
validate: factor out create_config()

Reviewers: thiblahute

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

9 years agovalidate: display debug info when stopping because EOS
Guillaume Desmottes [Fri, 8 May 2015 14:28:11 +0000 (16:28 +0200)]
validate: display debug info when stopping because EOS

Summary:
Useful to know if we are executing the 'stop' command provided by the scenario
or not.

Reviewers: thiblahute

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

9 years agoUpdate .gitignore
Thibault Saunier [Mon, 11 May 2015 17:40:49 +0000 (19:40 +0200)]
Update .gitignore

9 years agovalidate: unref last_caps when destroying pad monitor
Guillaume Desmottes [Thu, 7 May 2015 09:19:57 +0000 (11:19 +0200)]
validate: unref last_caps when destroying pad monitor

Reviewers: thiblahute

Reviewed By: thiblahute

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

9 years agovalidate: fix typo in reverse_playback.scenario
Guillaume Desmottes [Tue, 5 May 2015 13:59:18 +0000 (15:59 +0200)]
validate: fix typo in reverse_playback.scenario

Reviewers: thiblahute

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

9 years agovalidate: initialize position
Guillaume Desmottes [Tue, 5 May 2015 10:46:38 +0000 (12:46 +0200)]
validate: initialize position

Summary: Fix invalid read when executing without having the actual position.

Reviewers: thiblahute

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

9 years agovalidate: be less specific when ignoring the pixman tls leak
Guillaume Desmottes [Tue, 5 May 2015 07:32:53 +0000 (09:32 +0200)]
validate: be less specific when ignoring the pixman tls leak

Summary: I hit the same big in a slightly different code path.

Reviewers: thiblahute

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

9 years agoupdate gitignore
Guillaume Desmottes [Mon, 4 May 2015 12:22:00 +0000 (14:22 +0200)]
update gitignore

Reviewers: thiblahute

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

9 years agouse the setup_sink_props_max_lateness config scenario with valgrind
Guillaume Desmottes [Thu, 30 Apr 2015 15:39:55 +0000 (17:39 +0200)]
use the setup_sink_props_max_lateness config scenario with valgrind

Reviewers: thiblahute

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

9 years agovalidate: add 'target-element-klass' property on set-property action
Guillaume Desmottes [Thu, 30 Apr 2015 15:22:19 +0000 (17:22 +0200)]
validate: add 'target-element-klass' property on set-property action

Summary:
This allows us to set a property on all the elements of the pipeline matching
a specific klass name.

Reviewers: thiblahute

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

9 years agovalidate: add 'optional' action keyword
Guillaume Desmottes [Fri, 1 May 2015 14:39:04 +0000 (16:39 +0200)]
validate: add 'optional' action keyword

Reviewers: thiblahute

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

9 years agovalidate: scenario: call _element_added_cb() on existing children
Guillaume Desmottes [Thu, 30 Apr 2015 13:39:23 +0000 (15:39 +0200)]
validate: scenario: call _element_added_cb() on existing children

Summary:
We want to have a chance to set property on all the elements of the pipelines,
including the existing children when the element is added.

Reviewers: thiblahute

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

9 years agoFix 'stoped' typo
Guillaume Desmottes [Wed, 29 Apr 2015 12:12:01 +0000 (14:12 +0200)]
Fix 'stoped' typo

Reviewers: thiblahute

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

9 years agovalidate: add vg suppression for libdrm bug
Guillaume Desmottes [Mon, 27 Apr 2015 13:57:13 +0000 (15:57 +0200)]
validate: add vg suppression for libdrm bug

Reviewers: thiblahute

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

9 years agovalidate: ignore invalid read from libav aac decoding
Guillaume Desmottes [Mon, 27 Apr 2015 13:14:10 +0000 (15:14 +0200)]
validate: ignore invalid read from libav aac decoding

Reviewers: thiblahute

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

9 years agovalidate: ignore libvpx valgrind errors
Guillaume Desmottes [Mon, 27 Apr 2015 12:48:54 +0000 (14:48 +0200)]
validate: ignore libvpx valgrind errors

Reviewers: thiblahute

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

9 years agovalidate: display the URL of ignored Valgrind bugs
Guillaume Desmottes [Mon, 27 Apr 2015 12:04:05 +0000 (14:04 +0200)]
validate: display the URL of ignored Valgrind bugs

Summary:
We don't want to forget about those so best to remind it when starting tests
as we do with blacklisted tests.

Reviewers: thiblahute

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

9 years agovalidate: move look_for_file_in_source_dir and get_valgrind_suppression_file to utils
Guillaume Desmottes [Mon, 27 Apr 2015 11:25:44 +0000 (13:25 +0200)]
validate: move look_for_file_in_source_dir and get_valgrind_suppression_file to utils

Reviewers: thiblahute

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

9 years agovalidate: Make sure to run submodule init from the root dir
Thibault Saunier [Thu, 30 Apr 2015 21:57:09 +0000 (23:57 +0200)]
validate: Make sure to run submodule init from the root dir

9 years agovalidate:scenario: Fix 'duration' property of the pause action
Thibault Saunier [Wed, 29 Apr 2015 11:22:11 +0000 (13:22 +0200)]
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.

9 years agovalidate:launcher: Concider unset MediaDescriptor duration has 'infinite'
Thibault Saunier [Tue, 28 Apr 2015 14:44:42 +0000 (16:44 +0200)]
validate:launcher: Concider unset MediaDescriptor duration has 'infinite'

9 years agovalidate: ignore a pixman leak which is fixed in master
Guillaume Desmottes [Wed, 22 Apr 2015 09:38:56 +0000 (11:38 +0200)]
validate: ignore a pixman leak which is fixed in master

http://phabricator.freedesktop.org/D128

9 years agovalidate: use a bigger hammer to ignore mesa related leaks
Guillaume Desmottes [Tue, 21 Apr 2015 13:57:57 +0000 (15:57 +0200)]
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

9 years agocodecanalyzer: minor style fix
Tim-Philipp Müller [Thu, 23 Apr 2015 11:33:26 +0000 (12:33 +0100)]
codecanalyzer: minor style fix

9 years agocodecanalyzer: don't try to free uninitialized pointers
Vineeth T M [Thu, 23 Apr 2015 06:53:12 +0000 (15:53 +0900)]
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

9 years agocodecanalyzer: run gst-indent on code
Tim-Philipp Müller [Thu, 23 Apr 2015 11:23:24 +0000 (12:23 +0100)]
codecanalyzer: run gst-indent on code

9 years agovalidate: Update autogen.sh
Thibault Saunier [Thu, 23 Apr 2015 09:44:24 +0000 (11:44 +0200)]
validate: Update autogen.sh

And add a gst-validate.doap file.

9 years agoUpdate common submodule
Thibault Saunier [Thu, 23 Apr 2015 09:24:14 +0000 (11:24 +0200)]
Update common submodule

9 years agovalidate: use -debug versions of bins when running from source
Guillaume Desmottes [Tue, 21 Apr 2015 09:00:58 +0000 (11:00 +0200)]
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

9 years agomove scenarios to data/scenarios
Guillaume Desmottes [Mon, 20 Apr 2015 13:24:46 +0000 (15:24 +0200)]
move scenarios to data/scenarios

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

9 years agovalidate: add gstvalidate.supp valgrind suppression file
Guillaume Desmottes [Mon, 20 Apr 2015 08:53:29 +0000 (10:53 +0200)]
validate: add gstvalidate.supp valgrind suppression file

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

9 years agovalidate: scenario: fix structure and action leak
Guillaume Desmottes [Mon, 13 Apr 2015 11:55:56 +0000 (13:55 +0200)]
validate: scenario: fix structure and action leak

9 years agovalidate: report: fix GString leak when early returning
Guillaume Desmottes [Fri, 10 Apr 2015 09:28:34 +0000 (11:28 +0200)]
validate: report: fix GString leak when early returning

9 years agovalidate:pipelinemonitor: Print position only when in state >= PAUSED
Thibault Saunier [Sun, 19 Apr 2015 09:57:36 +0000 (11:57 +0200)]
validate:pipelinemonitor: Print position only when in state >= PAUSED

Reviewers: Mathieu_Du

Reviewed By: Mathieu_Du

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

9 years agovalidate:launcher: Use full HD screen as default screen size in xvfb
Thibault Saunier [Sun, 19 Apr 2015 09:56:29 +0000 (11:56 +0200)]
validate:launcher: Use full HD screen as default screen size in xvfb

9 years agovalidate:launcher: Add a way to run tests without displaying the output
Thibault Saunier [Fri, 17 Apr 2015 18:37:21 +0000 (20:37 +0200)]
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

9 years agovalidate:launcher: Error out if valgrind is not available on the system
Thibault Saunier [Fri, 17 Apr 2015 17:56:17 +0000 (19:56 +0200)]
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

9 years agovalidate:scenario: Stop scenario execution on stop action
Thibault Saunier [Fri, 17 Apr 2015 17:28:19 +0000 (19:28 +0200)]
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

9 years agovalidate: Gracefully handle absence of TAG on streams
Thibault Saunier [Thu, 16 Apr 2015 11:40:08 +0000 (13:40 +0200)]
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

9 years agovalidate:launcher: Pass GST_VALIDATE_SCENARIO to the subprocess env only
Thibault Saunier [Thu, 16 Apr 2015 10:02:11 +0000 (12:02 +0200)]
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

9 years agovalidate: don't pass NULL to gst_caps_copy()
Guillaume Desmottes [Tue, 31 Mar 2015 13:10:11 +0000 (15:10 +0200)]
validate: don't pass NULL to gst_caps_copy()

9 years agovalidate: use GstMediaDescriptorWriter as log handler
Guillaume Desmottes [Tue, 31 Mar 2015 12:54:28 +0000 (14:54 +0200)]
validate: use GstMediaDescriptorWriter as log handler

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

9 years agovalidate: media-descriptor-writer: fix string leaks
Guillaume Desmottes [Tue, 31 Mar 2015 07:59:58 +0000 (09:59 +0200)]
validate: media-descriptor-writer: fix string leaks

9 years agovalidate:launcher: Handle git error properly
Young Han Lee [Wed, 15 Apr 2015 05:02:32 +0000 (14:02 +0900)]
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

9 years agovalidate: set GST_GL_XINITTHREADS
Guillaume Desmottes [Tue, 14 Apr 2015 10:31:32 +0000 (12:31 +0200)]
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>
9 years agovalidate:launcher: Make validate the only default testsuite
Thibault Saunier [Fri, 10 Apr 2015 16:19:40 +0000 (18:19 +0200)]
validate:launcher: Make validate the only default testsuite

Reviewers: Mathieu_Du

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

9 years agovalidate:launcher: Let the responsibility to update asset to the testsuite
Thibault Saunier [Fri, 10 Apr 2015 16:11:09 +0000 (18:11 +0200)]
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

9 years agovalidate:tools: EOS handling is the responsibility of the scenario
Thibault Saunier [Fri, 10 Apr 2015 11:29:47 +0000 (13:29 +0200)]
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

9 years agovalidate:scenario: Add support for waiting on signals and messages
Ramiro Polla [Fri, 13 Feb 2015 17:34:04 +0000 (18:34 +0100)]
validate:scenario: Add support for waiting on signals and messages

Reviewers: Mathieu_Du

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

9 years agovalidate:scenario: Report disabling plugin issues
Thibault Saunier [Fri, 6 Mar 2015 10:55:09 +0000 (11:55 +0100)]
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

9 years agovalidate:scenario: Allow execution of disable-plugin as a config action
Thibault Saunier [Fri, 6 Mar 2015 10:51:19 +0000 (11:51 +0100)]
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

9 years agovalidate:scenarios: Set seek_forward_backward min-media-duration=45
Thibault Saunier [Tue, 3 Mar 2015 09:16:20 +0000 (09:16 +0000)]
validate:scenarios: Set seek_forward_backward min-media-duration=45

Reviewers: Mathieu_Du

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

9 years agovalidate:launcher: Set more env variable in the launcher command desc
Thibault Saunier [Fri, 27 Feb 2015 23:20:43 +0000 (23:20 +0000)]
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

9 years agovalidate: Add a method to easily get plugin configuration
Thibault Saunier [Mon, 2 Mar 2015 10:03:08 +0000 (11:03 +0100)]
validate: Add a method to easily get plugin configuration

Reviewers: Mathieu_Du

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