platform/upstream/gstreamer.git
9 years agovalidate:launcher: Treat min-media-duration as a float
Thibault Saunier [Wed, 30 Sep 2015 16:13:28 +0000 (18:13 +0200)]
validate:launcher: Treat min-media-duration as a float

9 years agoscenarios: Set min-media-duration on switch_audio_track
Thibault Saunier [Wed, 30 Sep 2015 13:02:03 +0000 (15:02 +0200)]
scenarios: Set min-media-duration on switch_audio_track

9 years agovalidate:scenario: Add a way to define a timeout for actions execution
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

9 years agovalidatessim: Stop using deprecated gst_segment_to_position()
Sebastian Dröge [Sat, 26 Sep 2015 16:46:05 +0000 (18:46 +0200)]
validatessim: Stop using deprecated gst_segment_to_position()

9 years agoBack to development
Thibault Saunier [Fri, 25 Sep 2015 10:51:31 +0000 (12:51 +0200)]
Back to development

9 years agoRelease 1.6.0
Thibault Saunier [Fri, 25 Sep 2015 10:50:13 +0000 (12:50 +0200)]
Release 1.6.0

9 years agovalidate: scenarios: set need clock sync for switching subtitle track
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

9 years agoRelease 1.5.90
Thibault Saunier [Thu, 20 Aug 2015 15:58:36 +0000 (17:58 +0200)]
Release 1.5.90

9 years agoUpdate common submodule
Thibault Saunier [Fri, 21 Aug 2015 09:09:03 +0000 (11:09 +0200)]
Update common submodule

9 years agovalidate: Put the GstValidate testsuite version in the launcher config
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

9 years agovalidate/transcoding: Don't override the target state of the scenario when receiving...
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.

9 years agovalidate: Don't override the target state of the scenario when receiving BUFFERING...
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.

9 years agovalidate: Fix typo
Sebastian Dröge [Sun, 16 Aug 2015 15:59:00 +0000 (17:59 +0200)]
validate: Fix typo

9 years agovalidate/launcher: Blacklist some HLS seeking tests again
Sebastian Dröge [Sun, 16 Aug 2015 15:53:28 +0000 (17:53 +0200)]
validate/launcher: Blacklist some HLS seeking tests again

9 years agovalidate/launcher: Un-blacklist validate.dash.playback.reverse_playback.*
Thiago Santos [Sun, 16 Aug 2015 11:50:36 +0000 (08:50 -0300)]
validate/launcher: Un-blacklist validate.dash.playback.reverse_playback.*

Fixed now

9 years agovalidate/launcher: Un-blacklist validate.hls.playback.reverse_playback.*
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.

9 years agovalidate/launcher: De-blacklist some HLS tests
Sebastian Dröge [Sat, 15 Aug 2015 17:07:02 +0000 (19:07 +0200)]
validate/launcher: De-blacklist some HLS tests

9 years agovalidate/launcher: Blacklist some DASH tests
Sebastian Dröge [Sat, 15 Aug 2015 17:04:14 +0000 (19:04 +0200)]
validate/launcher: Blacklist some DASH tests

9 years agolauncher: Handle override files for media files
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

9 years agolauncher: Make sure MediaDescriptor is set in the GstValidateTest class itself
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

9 years agovalidate/launcher: Treat DASH like HLS in another place
Sebastian Dröge [Sat, 15 Aug 2015 14:19:24 +0000 (16:19 +0200)]
validate/launcher: Treat DASH like HLS in another place

9 years agovalidate: scenario: fix leak during error cases
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

9 years agovalidate: descriptor-writer: Handle error when stream info is not available
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

9 years agovalidate: descriptor-writer: Print proper error message when discover fails
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

9 years agovalidate:override-registry: fix memory leak
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

9 years agovalidate: tools: transcoding error due to wrong condition check
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

9 years agovalidate:launcher: Fix documentation
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

9 years agovalidate:launcher: escape the characters to remove bad range in regex
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

9 years agovalidate:launcher: improve uri generation for --medias-path
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

9 years agovalidate:launcher: skip setting up test suite for --medias-paths
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

9 years agovalidate: scenarios: Change start time for reverse playback
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

9 years agovalidate: launcher: Add a --force-sync option
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)

9 years agovalidate: Enable flac reverse playback tests
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.

9 years agovalidate: media-descriptor: Workaround file:// stream-id changing
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

9 years agovalidate: media-descriptor: Fix reading seekable record
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.

9 years agovalidate: media-descriptor: handle proper return values
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

9 years agovalidate: media-descriptor: Add comment before ignored return value
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

9 years agovalidate: media-descriptor: remove duplicate conditions
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

9 years agovalidate: media-descriptor: fix trivial spelling mistakes
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

9 years agovalidate:tools: set locale to all and change argument to FILENAME
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

9 years agovalidate: Add a method to get action->scenario in a thread safe way
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

9 years agovalidate-scenario: Use thread-safe GWeakRef
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.

9 years agovalidate-scenario: Use GLib functions to make sure GMainContext is used
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

9 years agoscenario: Only modify the actions from the main thread
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.

9 years agogitignore: Add more generated files
Olivier Crête [Fri, 24 Jul 2015 21:05:30 +0000 (17:05 -0400)]
gitignore: Add more generated files

9 years agovalidate-scenario: get duration from media_info if not able to query
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

9 years agovalidate:launcher: add quotes to the file path
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

9 years agovalidate:launcher: unquote the path to remove special characters
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

9 years agovalidate:launcher: return on error cases properly
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

9 years agovalidate:launcher: handle file path in --medias-paths
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

9 years agovalidate:launcher: skip default media path for --media-paths option
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

9 years agovalidate:ssim: Fix calls to the converters
Thibault Saunier [Thu, 16 Jul 2015 11:44:07 +0000 (13:44 +0200)]
validate:ssim: Fix calls to the converters

We were mixing them

9 years agovalidate:main.py: trivial document fixes
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

9 years agovalidate:gtk: Handle the case were we are 'pressing' only a modifier
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

9 years agovalidate: don't store the full description struct
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

9 years agovalidate:scenario: Report EVENT_SEEK_NOT_HANDLED as reported error on error
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

9 years agovalidate: Document the GST_VALIDATE_CONFIG environment variable
Thibault Saunier [Mon, 13 Jul 2015 11:10:15 +0000 (13:10 +0200)]
validate: Document the GST_VALIDATE_CONFIG environment variable

9 years agovalidate:scenario: Implement a config to set the interval between action calls
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.

9 years agoBack to development
Thibault Saunier [Wed, 24 Jun 2015 15:43:53 +0000 (17:43 +0200)]
Back to development

9 years agoRelease 1.5.2
Thibault Saunier [Wed, 24 Jun 2015 15:42:16 +0000 (17:42 +0200)]
Release 1.5.2

9 years agovalidate:scenario: Reset seeked_in_paused when wrong position detected
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

9 years agovalidate:ssim: Inform about min average and min minimum similarities
Thibault Saunier [Thu, 18 Jun 2015 09:09:26 +0000 (11:09 +0200)]
validate:ssim: Inform about min average and min minimum similarities

9 years agovalidate: spelling fixes
Tim-Philipp Müller [Sun, 14 Jun 2015 21:44:26 +0000 (22:44 +0100)]
validate: spelling fixes

analize != analyze

9 years agovalidate: media-check: add newline to end of print statements
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

9 years agovalidate: media-check: fix double unref in error code path
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

9 years agovalidate: tools: fix build
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

9 years agovalidate:ssim: Make position reporting parseable by the launcher
Thibault Saunier [Fri, 12 Jun 2015 10:10:55 +0000 (12:10 +0200)]
validate:ssim: Make position reporting parseable by the launcher

9 years agovalidate:ssim: Print better information about execution
Thibault Saunier [Fri, 12 Jun 2015 09:00:54 +0000 (11:00 +0200)]
validate:ssim: Print better information about execution

9 years agovalidate: print REPORTER->name when passed as source in validate_printf
Thibault Saunier [Fri, 12 Jun 2015 08:59:28 +0000 (10:59 +0200)]
validate: print REPORTER->name when passed as source in validate_printf

9 years agovalidate:launcher: Avoid printing twice env variables
Thibault Saunier [Fri, 12 Jun 2015 09:17:43 +0000 (11:17 +0200)]
validate:launcher: Avoid printing twice env variables

When printing test command.

9 years agovalidate:tools: Cleanup Makefile.am
Thibault Saunier [Tue, 9 Jun 2015 08:52:21 +0000 (10:52 +0200)]
validate:tools: Cleanup Makefile.am

Removing useless CFLAGS and LIBS

9 years agovalidate:media-descriptor-writer: cleanup get tag code
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

9 years agovalidate: Do not define GstValidateOverride type twice
Thibault Saunier [Mon, 8 Jun 2015 16:48:30 +0000 (18:48 +0200)]
validate: Do not define GstValidateOverride type twice

9 years agovalidate: Fix make distcheck
Thibault Saunier [Mon, 8 Jun 2015 16:20:33 +0000 (18:20 +0200)]
validate: Fix make distcheck

9 years agovalidate: Generate documentation for Validate plugins
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

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