Vineeth T M [Wed, 26 Aug 2015 01:36:51 +0000 (10:36 +0900)]
validate: launcher: Support relative path for folder names
Even though relative paths are supported, right now it does not
work when we give the path as 'media/' present in the current directory.
Adding support for the same.
https://bugzilla.gnome.org/show_bug.cgi?id=754100
Thibault Saunier [Wed, 30 Sep 2015 16:13:28 +0000 (18:13 +0200)]
validate:launcher: Treat min-media-duration as a float
Thibault Saunier [Wed, 30 Sep 2015 13:02:03 +0000 (15:02 +0200)]
scenarios: Set min-media-duration on switch_audio_track
Thibault Saunier [Wed, 30 Sep 2015 12:55:37 +0000 (14:55 +0200)]
validate:scenario: Add a way to define a timeout for actions execution
Reviewers: Mathieu_Du
Differential Revision: https://phabricator.freedesktop.org/D271
Sebastian Dröge [Sat, 26 Sep 2015 16:46:05 +0000 (18:46 +0200)]
validatessim: Stop using deprecated gst_segment_to_position()
Thibault Saunier [Fri, 25 Sep 2015 10:51:31 +0000 (12:51 +0200)]
Back to development
Thibault Saunier [Fri, 25 Sep 2015 10:50:13 +0000 (12:50 +0200)]
Release 1.6.0
eunhae choi [Wed, 16 Sep 2015 08:12:17 +0000 (17:12 +0900)]
validate: scenarios: set need clock sync for switching subtitle track
As soon as the track is changed, the pipeline state is set to NULL
by execution 'stop' action even if there is a 'playback-time' with 5sec.
If the AV sink is not synchronized,
audio fakesink and video fakesink has different position value.
When the validate request the position information of pipeline
to do 'stop' action, the audio fakesink response of the position query
with the bigger value than 5sec.
https://bugzilla.gnome.org/show_bug.cgi?id=755101
Thibault Saunier [Thu, 20 Aug 2015 15:58:36 +0000 (17:58 +0200)]
Release 1.5.90
Thibault Saunier [Fri, 21 Aug 2015 09:09:03 +0000 (11:09 +0200)]
Update common submodule
Thibault Saunier [Thu, 20 Aug 2015 14:35:15 +0000 (16:35 +0200)]
validate: Put the GstValidate testsuite version in the launcher config
This way testsuite implementation can have the information
Sebastian Dröge [Mon, 17 Aug 2015 15:20:07 +0000 (17:20 +0200)]
validate/transcoding: Don't override the target state of the scenario when receiving BUFFERING=100%
If the scenario handles the states and wants to stay in PAUSED, it's not a
good idea to change the state to PLAYING when receiving BUFFERING=100%. This
caused a race condition in varios seeking tests, most often in the dash scrub
seeking test.
Sebastian Dröge [Mon, 17 Aug 2015 12:27:33 +0000 (14:27 +0200)]
validate: Don't override the target state of the scenario when receiving BUFFERING=100%
If the scenario handles the states and wants to stay in PAUSED, it's not a
good idea to change the state to PLAYING when receiving BUFFERING=100%. This
caused a race condition in varios seeking tests, most often in the dash scrub
seeking test.
Sebastian Dröge [Sun, 16 Aug 2015 15:59:00 +0000 (17:59 +0200)]
validate: Fix typo
Sebastian Dröge [Sun, 16 Aug 2015 15:53:28 +0000 (17:53 +0200)]
validate/launcher: Blacklist some HLS seeking tests again
Thiago Santos [Sun, 16 Aug 2015 11:50:36 +0000 (08:50 -0300)]
validate/launcher: Un-blacklist validate.dash.playback.reverse_playback.*
Fixed now
Sebastian Dröge [Sun, 16 Aug 2015 10:26:16 +0000 (12:26 +0200)]
validate/launcher: Un-blacklist validate.hls.playback.reverse_playback.*
It apparently succeeds now.
Sebastian Dröge [Sat, 15 Aug 2015 17:07:02 +0000 (19:07 +0200)]
validate/launcher: De-blacklist some HLS tests
Sebastian Dröge [Sat, 15 Aug 2015 17:04:14 +0000 (19:04 +0200)]
validate/launcher: Blacklist some DASH tests
Thibault Saunier [Sat, 15 Aug 2015 14:40:11 +0000 (16:40 +0200)]
launcher: Handle override files for media files
And make sure to create a new dict for extra_env_vars when instanciating
GstValidateTest
Thibault Saunier [Sat, 15 Aug 2015 14:23:02 +0000 (16:23 +0200)]
launcher: Make sure MediaDescriptor is set in the GstValidateTest class itself
It is used there but was set in each and every subclasses
Sebastian Dröge [Sat, 15 Aug 2015 14:19:24 +0000 (16:19 +0200)]
validate/launcher: Treat DASH like HLS in another place
Vineeth TM [Fri, 7 Aug 2015 03:51:53 +0000 (12:51 +0900)]
validate: scenario: fix leak during error cases
When message_async is not called during error cases, needs_parsing GList is
not being freed resulting in leak. Hence free'ing the same in finalize.
https://bugzilla.gnome.org/show_bug.cgi?id=753339
Vineeth TM [Thu, 23 Jul 2015 06:51:09 +0000 (15:51 +0900)]
validate: descriptor-writer: Handle error when stream info is not available
There is no check to see if stream info is available. This leads to
assertion error. Adding proper error messages for the same and reported
the same as a validate warning message.
https://bugzilla.gnome.org/show_bug.cgi?id=752758
Vineeth TM [Thu, 23 Jul 2015 06:08:55 +0000 (15:08 +0900)]
validate: descriptor-writer: Print proper error message when discover fails
When discovering the files, there will be different kind of errors. If we print
the exact message, then it will be more helpful for user. Especially in the case
of missing plugins, displaying which plugin is missing as error message
https://bugzilla.gnome.org/show_bug.cgi?id=752758
Vineeth TM [Thu, 23 Jul 2015 04:35:04 +0000 (13:35 +0900)]
validate:override-registry: fix memory leak
mutex is being initialized but not cleared.
https://bugzilla.gnome.org/show_bug.cgi?id=752754
Vineeth TM [Thu, 23 Jul 2015 02:08:18 +0000 (11:08 +0900)]
validate: tools: transcoding error due to wrong condition check
when checking the restriction caps, not adding proper check, which
results in assertion error when calling gst_caps_from_string
https://bugzilla.gnome.org/show_bug.cgi?id=752749
Vineeth TM [Thu, 23 Jul 2015 00:18:46 +0000 (09:18 +0900)]
validate:launcher: Fix documentation
Fix some trivial spelling mistakes in documentation
and document about --update-media-info.
https://bugzilla.gnome.org/show_bug.cgi?id=752748
Vineeth TM [Tue, 21 Jul 2015 23:45:26 +0000 (08:45 +0900)]
validate:launcher: escape the characters to remove bad range in regex
When media file name consists of some special characters of the format
[b-a].mp3, then it fails with 'bad character range' error and exits.
call re.escape to escape the characters before using it in findall
https://bugzilla.gnome.org/show_bug.cgi?id=752650
Vineeth TM [Mon, 3 Aug 2015 23:35:16 +0000 (08:35 +0900)]
validate:launcher: improve uri generation for --medias-path
When --medias-paths option is being used, right now we have to
specify the full path, like /home/user/gst/master/media/
But when inside master directory, would like to specify only
media/ and expect it to work. Using os.path.abspath and create uri based on that.
This way we can either just pass media/ or pass the full path as parameters.
https://bugzilla.gnome.org/show_bug.cgi?id=752518
Vineeth TM [Fri, 17 Jul 2015 07:45:35 +0000 (16:45 +0900)]
validate:launcher: skip setting up test suite for --medias-paths
in validate.py, some mixer test generators are being added by default.
When passing --media-paths, i would not want to test these.
So instead of setting up the validate test suite, just call tester.register_defaults().
https://bugzilla.gnome.org/show_bug.cgi?id=752518
Vineeth TM [Tue, 4 Aug 2015 04:47:24 +0000 (13:47 +0900)]
validate: scenarios: Change start time for reverse playback
Right now reverse playback happens till the beginning of the media file.
But for files which are longer than 150 seconds,
Timeout 'Hard timeout reached: 150 secs' error happens. So we should set the
start time within 150 seconds.
https://bugzilla.gnome.org/show_bug.cgi?id=753216
Thibault Saunier [Tue, 4 Aug 2015 13:21:16 +0000 (15:21 +0200)]
validate: launcher: Add a --force-sync option
Which should put the testsuite in a clean state (basically using git
reset --hard for git based testsuite for example)
Nicolas Dufresne [Tue, 4 Aug 2015 12:53:17 +0000 (08:53 -0400)]
validate: Enable flac reverse playback tests
This is now supported and works as expected.
Nicolas Dufresne [Fri, 31 Jul 2015 14:50:24 +0000 (10:50 -0400)]
validate: media-descriptor: Workaround file:// stream-id changing
file:// base stream-id will vary depending on the file path. As we
don't expect everyone to use the same absolute path to place the
validate testsuite, the resulting stream-id changes. Because of that,
we can't match the stream-id in the recorded file, hence cannot do
further check. We work around this by doing what filesink would do,
which is compute a SHA256 of the URI which we can use to first
validate the ID is prefixed like expected, and decide if we should
consider the stream IDs the same or not.
https://bugzilla.gnome.org/show_bug.cgi?id=753079
Nicolas Dufresne [Fri, 31 Jul 2015 14:49:00 +0000 (10:49 -0400)]
validate: media-descriptor: Fix reading seekable record
Casting the result of g_strmp0 to boolean won't make gboolean
value 0 or 1. We need proper 0 and 1 so we can use == comparision.
Vineeth TM [Fri, 24 Jul 2015 06:36:27 +0000 (15:36 +0900)]
validate: media-descriptor: handle proper return values
while comparing the media descriptor with --expected-results, the return
values are not being handled properly, which results in wrong comparision
https://bugzilla.gnome.org/show_bug.cgi?id=748390
Nicolas Dufresne [Thu, 30 Jul 2015 19:14:13 +0000 (15:14 -0400)]
validate: media-descriptor: Add comment before ignored return value
As stated in the bug, this comparison failing is not a critical
error, warning is enough. Add a comment so nobody thinks it's a
coding error.
https://bugzilla.gnome.org/review?bug=748390
Vineeth TM [Wed, 22 Jul 2015 07:32:06 +0000 (16:32 +0900)]
validate: media-descriptor: remove duplicate conditions
when comparing tags, two conditions in if an else if are same
the correct way is to first check if both are NULL and return.
changed the condition accordingly.
https://bugzilla.gnome.org/show_bug.cgi?id=748390
Vineeth TM [Wed, 22 Jul 2015 07:07:19 +0000 (16:07 +0900)]
validate: media-descriptor: fix trivial spelling mistakes
replace comparse_stream with compare_streams
https://bugzilla.gnome.org/show_bug.cgi?id=748390
Vineeth TM [Wed, 29 Jul 2015 07:42:48 +0000 (16:42 +0900)]
validate:tools: set locale to all and change argument to FILENAME
When file name consists of characters from other languages, say korean,
then it throws an error
Error initializing: Invalid byte sequence in conversion input
Hence setting locale to all to fix this.
And changing the media-info argument to type G_OPTION_ARG_FILENAME
https://bugzilla.gnome.org/show_bug.cgi?id=752945
Thibault Saunier [Sat, 25 Jul 2015 08:54:19 +0000 (10:54 +0200)]
validate: Add a method to get action->scenario in a thread safe way
API:
gst_validate_action_get_scenario
Olivier Crête [Fri, 24 Jul 2015 20:47:57 +0000 (16:47 -0400)]
validate-scenario: Use thread-safe GWeakRef
Since _set_done() is meant to be thread safe,
it can not be used with g_object_add_weak_pointer(),
instead, one must use GWeakRef. But since it is in the API,
document that fact and add a couple assertions to make sure
it doesn't get broken in the future.
Olivier Crête [Fri, 24 Jul 2015 20:25:38 +0000 (16:25 -0400)]
validate-scenario: Use GLib functions to make sure GMainContext is used
Olivier Crête [Fri, 24 Jul 2015 20:19:46 +0000 (16:19 -0400)]
scenario: Only modify the actions from the main thread
The action's content is not protected by a mutex, so only
modify it from the main thread.
Olivier Crête [Fri, 24 Jul 2015 21:05:30 +0000 (17:05 -0400)]
gitignore: Add more generated files
Vineeth T M [Fri, 17 Jul 2015 14:42:22 +0000 (23:42 +0900)]
validate-scenario: get duration from media_info if not able to query
In case of files, which don't have duration in header, baseparse
estimates the duration only after 1.5 seconds. But Async_done event
is sent before the duration is estimated, which results in error.
If duration query fails, getting the duration from the media-info being
passed through --set-media-info. If media-info is also not set,
printing an error message and throwing error.
https://bugzilla.gnome.org/show_bug.cgi?id=752521
Vineeth T M [Mon, 20 Jul 2015 10:37:41 +0000 (19:37 +0900)]
validate:launcher: add quotes to the file path
When folder name contains space or other special characters,
it fails to recognise the same and error is thrown. Adding the path
inside to recognise the same
https://bugzilla.gnome.org/show_bug.cgi?id=752611
Vineeth T M [Mon, 20 Jul 2015 10:35:34 +0000 (19:35 +0900)]
validate:launcher: unquote the path to remove special characters
When getting path from url using, url2path, it is returning
special characters (%20 for space etc..), instead of returning
plain path. path.unquote replaces the same..
https://bugzilla.gnome.org/show_bug.cgi?id=752611
Vineeth TM [Mon, 20 Jul 2015 08:27:56 +0000 (17:27 +0900)]
validate:launcher: return on error cases properly
When folder name contains spaces during --medias-paths, it does not
create the media info, but still it shows as passed.
Returing failed during this case
https://bugzilla.gnome.org/show_bug.cgi?id=752611
Vineeth TM [Wed, 22 Jul 2015 06:20:54 +0000 (15:20 +0900)]
validate:launcher: handle file path in --medias-paths
right now --medias-paths accepts only directories. Added support to
accept file path as well.
https://bugzilla.gnome.org/show_bug.cgi?id=752692
Vineeth T M [Thu, 16 Jul 2015 11:27:11 +0000 (20:27 +0900)]
validate:launcher: skip default media path for --media-paths option
when --media-paths is specified, then no need to check the default media.
And add Force argument to let testsuite force the inclusion of
default media directory.
https://bugzilla.gnome.org/show_bug.cgi?id=752461
Thibault Saunier [Thu, 16 Jul 2015 11:44:07 +0000 (13:44 +0200)]
validate:ssim: Fix calls to the converters
We were mixing them
Vineeth T M [Thu, 16 Jul 2015 01:28:18 +0000 (10:28 +0900)]
validate:main.py: trivial document fixes
gst-validate-launch is being used instead of gst-validate-launcher
in a couple of places.
https://bugzilla.gnome.org/show_bug.cgi?id=752455
Thibault Saunier [Tue, 14 Jul 2015 18:31:59 +0000 (20:31 +0200)]
validate:gtk: Handle the case were we are 'pressing' only a modifier
Guillaume Desmottes [Tue, 14 Jul 2015 16:28:18 +0000 (18:28 +0200)]
validate: don't store the full description struct
Summary:
When running valgrind we'll have 2 scenarios loaded (the normal one and
"setup_sink_props_max_lateness.scenario"). The loading code shouldn't assume
which one will contain the description it actually care about and so just look
for the fields it actually needs.
Reviewers: thiblahute
Differential Revision: http://phabricator.freedesktop.org/D199
Thibault Saunier [Tue, 14 Jul 2015 16:16:40 +0000 (18:16 +0200)]
validate:scenario: Report EVENT_SEEK_NOT_HANDLED as reported error on error
Thibault Saunier [Mon, 13 Jul 2015 11:10:15 +0000 (13:10 +0200)]
validate: Document the GST_VALIDATE_CONFIG environment variable
Thibault Saunier [Mon, 13 Jul 2015 11:05:41 +0000 (13:05 +0200)]
validate:scenario: Implement a config to set the interval between action calls
Allowing users to decide the time between which the action should be
executed. In some cases executing on idle might lead to action not
being executed fast enough so the user might want to force an interval
in that case.
Thibault Saunier [Wed, 24 Jun 2015 15:43:53 +0000 (17:43 +0200)]
Back to development
Thibault Saunier [Wed, 24 Jun 2015 15:42:16 +0000 (17:42 +0200)]
Release 1.5.2
Thibault Saunier [Wed, 24 Jun 2015 14:06:06 +0000 (16:06 +0200)]
validate:scenario: Reset seeked_in_paused when wrong position detected
+ Lower some debug output to LOG
Thibault Saunier [Thu, 18 Jun 2015 09:09:26 +0000 (11:09 +0200)]
validate:ssim: Inform about min average and min minimum similarities
Tim-Philipp Müller [Sun, 14 Jun 2015 21:44:26 +0000 (22:44 +0100)]
validate: spelling fixes
analize != analyze
Wonchul Lee [Fri, 29 May 2015 07:45:25 +0000 (16:45 +0900)]
validate: media-check: add newline to end of print statements
https://bugzilla.gnome.org/show_bug.cgi?id=750089
Wonchul Lee [Tue, 9 Jun 2015 00:10:42 +0000 (09:10 +0900)]
validate: media-check: fix double unref in error code path
Writer would get unrefed twice when it could not parse the file.
https://bugzilla.gnome.org/show_bug.cgi?id=750606
Tim-Philipp Müller [Sat, 13 Jun 2015 18:25:17 +0000 (19:25 +0100)]
validate: tools: fix build
/usr/bin/ld: gst-validate-images-check.o: undefined reference to symbol 'gst_init'
/home/tpm/gst/glib-master/gstreamer/gst/.libs/libgstreamer-1.0.so.0: error adding symbols: DSO missing from command line
Thibault Saunier [Fri, 12 Jun 2015 10:10:55 +0000 (12:10 +0200)]
validate:ssim: Make position reporting parseable by the launcher
Thibault Saunier [Fri, 12 Jun 2015 09:00:54 +0000 (11:00 +0200)]
validate:ssim: Print better information about execution
Thibault Saunier [Fri, 12 Jun 2015 08:59:28 +0000 (10:59 +0200)]
validate: print REPORTER->name when passed as source in validate_printf
Thibault Saunier [Fri, 12 Jun 2015 09:17:43 +0000 (11:17 +0200)]
validate:launcher: Avoid printing twice env variables
When printing test command.
Thibault Saunier [Tue, 9 Jun 2015 08:52:21 +0000 (10:52 +0200)]
validate:tools: Cleanup Makefile.am
Removing useless CFLAGS and LIBS
Wonchul Lee [Tue, 9 Jun 2015 02:14:58 +0000 (11:14 +0900)]
validate:media-descriptor-writer: cleanup get tag code
https://bugzilla.gnome.org/show_bug.cgi?id=750609
Thibault Saunier [Mon, 8 Jun 2015 16:48:30 +0000 (18:48 +0200)]
validate: Do not define GstValidateOverride type twice
Thibault Saunier [Mon, 8 Jun 2015 16:20:33 +0000 (18:20 +0200)]
validate: Fix make distcheck
Thibault Saunier [Mon, 8 Jun 2015 15:11:51 +0000 (17:11 +0200)]
validate: Generate documentation for Validate plugins
Summary: Depends on D215
Reviewers: Mathieu_Du
Differential Revision: http://phabricator.freedesktop.org/D216
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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.
Luis de Bethencourt [Fri, 29 May 2015 16:40:26 +0000 (17:40 +0100)]
validate-utils: clean error handling in _file_get_lines ()
Luis de Bethencourt [Fri, 29 May 2015 15:29:44 +0000 (16:29 +0100)]
validate-utils: simplify _read_builtin ()
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
Thibault Saunier [Wed, 20 May 2015 11:57:55 +0000 (13:57 +0200)]
validate:scenario: Make sure to actually test position to execute actions
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
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
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
Thibault Saunier [Thu, 14 May 2015 15:43:40 +0000 (17:43 +0200)]
validate:launcher: extra_env_variables is a dictionnary
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