Olivier Crête
dad9d252bc
Revert "validate-utils: simplify _read_builtin ()"
...
This breaks the fast_forward scenario parsing.
This reverts commit 0cfff156b1
.
2015-06-02 16:46:15 -04:00
Luis de Bethencourt
ab65b1e743
validate-utils: clean error handling in _file_get_lines ()
2015-05-29 17:40:26 +01:00
Luis de Bethencourt
0cfff156b1
validate-utils: simplify _read_builtin ()
2015-05-29 16:47:48 +01:00
Vineeth T M
1e3084aa95
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
2015-05-29 15:40:55 +01:00
Thibault Saunier
e832785604
validate:scenario: Make sure to actually test position to execute actions
2015-05-20 14:16:14 +02:00
Guillaume Desmottes
9ef7943369
validate: add valgrind ignore supps for theora encoder
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D181
2015-05-18 10:16:28 +02:00
Guillaume Desmottes
dc2ba59ebb
validate: add more H264 valgrind supp rules
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D180
2015-05-18 10:16:05 +02:00
Thibault Saunier
4829e44147
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
2015-05-18 10:15:33 +02:00
Thibault Saunier
d89b4884c6
validate:launcher: extra_env_variables is a dictionnary
2015-05-14 17:44:17 +02:00
Thibault Saunier
dc2f4f11c6
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
2015-05-14 15:56:57 +02:00
Thibault Saunier
4ada36dbfc
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
2015-05-14 15:56:57 +02:00
Thibault Saunier
817ef82bb0
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
2015-05-14 15:56:57 +02:00
Thibault Saunier
d90f17a5b1
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
2015-05-14 15:56:56 +02:00
Thibault Saunier
92732c9f24
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
2015-05-14 15:56:56 +02:00
Thibault Saunier
53628bd7f2
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
2015-05-14 15:56:56 +02:00
Thibault Saunier
0f1f1db171
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
2015-05-14 15:56:56 +02:00
Thibault Saunier
2ccae6e500
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
2015-05-14 15:56:56 +02:00
Thibault Saunier
319e86c81c
validate: Do not unref twice the same list
...
Summary: Depends on D166
Reviewers: Mathieu_Du
Differential Revision: http://phabricator.freedesktop.org/D167
2015-05-14 15:56:56 +02:00
Thibault Saunier
2e8552eff0
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
2015-05-14 15:56:56 +02:00
Thibault Saunier
2548eafd11
validate: Avoid depending on Gst 1.4
...
Summary: Depends on D117
Reviewers: Mathieu_Du
Differential Revision: http://phabricator.freedesktop.org/D165
2015-05-14 15:56:56 +02:00
Thibault Saunier
b5d3622b31
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
2015-05-14 15:56:56 +02:00
Thibault Saunier
0969e79607
validate: Always git submodule update from the toplevel directory
...
Otherwise it fails with older git versions
2015-05-12 09:55:58 +02:00
Emanuele Aina
2c3d0c9eab
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
2015-05-11 21:56:27 +02:00
Emanuele Aina
1286989c31
validate: Go back to the validate dir after submodule init
...
https://bugzilla.gnome.org/show_bug.cgi?id=749162
2015-05-11 21:56:27 +02:00
Guillaume Desmottes
d548286209
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
2015-05-11 21:55:24 +02:00
Guillaume Desmottes
4927c65710
validate: disable QOS features when running with valgrind
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D156
2015-05-11 21:55:24 +02:00
Guillaume Desmottes
b54a22c9bc
validate: move element_has_klass() to utils
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D163
2015-05-11 21:55:24 +02:00
Guillaume Desmottes
f6681ff003
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
2015-05-11 21:55:24 +02:00
Guillaume Desmottes
2760da33a3
validate: allow to pass more than one file to GST_VALIDATE_CONFIG
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D161
2015-05-11 21:55:24 +02:00
Guillaume Desmottes
e5e7afd86f
validate: gst_validate_plugin_get_config() return 'core' conf if plugin is NULL
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D160
2015-05-11 21:55:24 +02:00
Guillaume Desmottes
92fa2b5681
validate: add gst_validate_deinit()
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D159
2015-05-11 21:55:24 +02:00
Guillaume Desmottes
596daaed74
validate: don't leak not maching config structures
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D158
2015-05-11 21:55:24 +02:00
Guillaume Desmottes
c2fef15266
validate: factor out create_config()
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D157
2015-05-11 21:55:24 +02:00
Guillaume Desmottes
63ca026b03
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
2015-05-11 21:55:24 +02:00
Thibault Saunier
ddc169196e
Update .gitignore
2015-05-11 19:41:08 +02:00
Guillaume Desmottes
0417e47e50
validate: unref last_caps when destroying pad monitor
...
Reviewers: thiblahute
Reviewed By: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D150
2015-05-07 11:19:57 +02:00
Guillaume Desmottes
5a5f718900
validate: fix typo in reverse_playback.scenario
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D148
2015-05-06 11:32:33 +02:00
Guillaume Desmottes
c2128e45af
validate: initialize position
...
Summary: Fix invalid read when executing without having the actual position.
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D147
2015-05-06 11:32:30 +02:00
Guillaume Desmottes
c08da0041d
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
2015-05-06 11:32:26 +02:00
Guillaume Desmottes
007f433d73
update gitignore
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D145
2015-05-04 14:48:26 +02:00
Guillaume Desmottes
b947e1c2ef
use the setup_sink_props_max_lateness config scenario with valgrind
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D141
2015-05-04 14:13:46 +02:00
Guillaume Desmottes
4a8d4ddbf6
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
2015-05-04 14:13:45 +02:00
Guillaume Desmottes
316a7bb74d
validate: add 'optional' action keyword
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D139
2015-05-04 13:24:21 +02:00
Guillaume Desmottes
d0a02df6e5
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
2015-05-04 13:24:21 +02:00
Guillaume Desmottes
db366e0a37
Fix 'stoped' typo
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D137
2015-05-01 18:19:39 +02:00
Guillaume Desmottes
4c70c57427
validate: add vg suppression for libdrm bug
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D134
2015-05-01 15:15:26 +02:00
Guillaume Desmottes
d1f1fab185
validate: ignore invalid read from libav aac decoding
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D133
2015-05-01 15:15:26 +02:00
Guillaume Desmottes
1c03eb0f71
validate: ignore libvpx valgrind errors
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D132
2015-05-01 15:15:26 +02:00
Guillaume Desmottes
133c415ade
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
2015-05-01 15:15:26 +02:00
Guillaume Desmottes
c5c39d88b1
validate: move look_for_file_in_source_dir and get_valgrind_suppression_file to utils
...
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D130
2015-05-01 15:15:26 +02:00