Age | Commit message (Collapse) | Author | Files | Lines |
|
Having a default value of 0 meant that a g_idle_add loop was constantly
running, causing each test to use 100% cpu.
This is no longer required. Using a 10ms interval brings down cpu usage
to a sane value
|
|
When g_option_context_parse fails, context and error variables are
not getting free'd which results in memory leaks being reported.
https://bugzilla.gnome.org/show_bug.cgi?id=753862
|
|
is not present.
When a file does not contain any stream info, then there is no need
to create the media info file as, it is not considered to be a valid file
and no validate checks are done for the same.
This skips unnecessary files like .txt, .dump files
https://bugzilla.gnome.org/show_bug.cgi?id=754006
|
|
|
|
When there are no media-info files present and --generate-media-info
option is not given, then it just fails without printing error.
Printing an error stating, use --generate-media-info if there are no
media info files.
When there are neither media files and media info files, print error
stating the same
https://bugzilla.gnome.org/show_bug.cgi?id=755087
|
|
When g_option_context_parse fails, context and error variables are not getting free'd
which results in memory leaks. Free'ing the same.
And replacing g_error_free with g_clear_error, which checks if the error being passed
is not NULL and sets the variable to NULL on free'ing.
https://bugzilla.gnome.org/show_bug.cgi?id=753862
|
|
Suppress this error, until the logic in libav is fixed.
https://bugzilla.gnome.org/show_bug.cgi?id=753268
|
|
Making debugging races leading to crashes easier to debug
|
|
|
|
|
|
The user might have scenarios specific to a particular pipeline, and the
application might have several pipelines running and scenarios that
apply on specific pipeline. We have to handle that valid use case.
|
|
They are reliable enough after 36b80edb7263118467dfcaee3923f7c964ae6bc8
in gst-plugins-base now.
|
|
Add missing gir include path for building with gst-uninstalled script
Differential Revision: https://phabricator.freedesktop.org/D461
|
|
While printing error in transcoding, gst-validate-transcoding-0.10 is being used.
Changing the same to 1.0
https://bugzilla.gnome.org/show_bug.cgi?id=756215
|
|
|
|
Summary:
Move variable declarations in the for block so we won't try re-free
tldir in case of early short circuiting of the 'for' code.
Depends on D348
Reviewers: thiblahute
Reviewed By: thiblahute
Differential Revision: https://phabricator.freedesktop.org/D349
|
|
Summary:
We were checking if the path was a full one but was using the
scenario_name instead of this path when trying to load the scenario.
Depends on D346
Reviewers: thiblahute
Reviewed By: thiblahute
Differential Revision: https://phabricator.freedesktop.org/D348
|
|
|
|
errors-for-leak-kinds should be set to definite, because almost every test case
, will have possibly lost memory, which may or may not be a leak.
And throwing error for all these cases doesn't seem to be correct.
https://bugzilla.gnome.org/show_bug.cgi?id=752754
|
|
valgrind logs
When creating the class names for media check, uri is being used,
instead of the path. Hence converting the uri using uri2path and creating
class name.
Add double quotes for valgrind logs, to support special characters like space
https://bugzilla.gnome.org/show_bug.cgi?id=752808
|
|
If not using the GError being passed on to media descriptor, writer and parser,
simply pass NULL instead of GError.
https://bugzilla.gnome.org/show_bug.cgi?id=753340
|
|
during error cases
writer_new_discover() API should be able to accept NULL GError and in case of
error, if GError is passed on as parameter, it should be propagated, else it
should be free'd.
https://bugzilla.gnome.org/show_bug.cgi?id=753340
|
|
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=753486
|
|
Makes it easier to know if the test passed or failed.
https://bugzilla.gnome.org/show_bug.cgi?id=754013
|
|
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
|
|
|
|
|
|
Reviewers: Mathieu_Du
Differential Revision: https://phabricator.freedesktop.org/D271
|
|
|
|
|
|
|
|
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
|
|
|
|
|
|
This way testsuite implementation can have the information
|
|
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.
|
|
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.
|
|
|
|
|
|
Fixed now
|
|
It apparently succeeds now.
|
|
|
|
|
|
And make sure to create a new dict for extra_env_vars when instanciating
GstValidateTest
|
|
It is used there but was set in each and every subclasses
|
|
|
|
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
|
|
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
|
|
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
|
|
mutex is being initialized but not cleared.
https://bugzilla.gnome.org/show_bug.cgi?id=752754
|