validate: Fix Validate plugin paths mixup

validate: fix typo in debug message

validate: fix some recurring typos

validate: fix validate-transcoding option descriptions

validate: improve run-time option descriptions

validate: No execute ON_ADDITION if a previous action has a playback-time

As this is what user will expect in this case.

For example with this scenario:

    set-state, state=null; playback-time=5
    set-property, target-element-name=dvbsrc0, property-name=delsys, property-value=11
    play;

validate:launcher: Add support for relative path while providing file path

Instead of providing full absolute path while validating the file, should be
able to provide the relative path with respect to the present directory.

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

validate: fix monitor leak when doing frame analysis

The monitor returned by gst_validate_monitor_factory_create() was never
unreffed.
Report instances now have to keep a ref, as suggested by the TODO, as
the reporter is no longer leaked.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D1012

validate: redefine default cat for the runner

The gstvalidate_debug may not be initialized like with the
validate/reporting which was crashing when run with GST_DEBUG=5.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D1004

validate: transcoding: fix encoding_profile leak

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D980

validate: transcoding: fix caps leaks

The 'all_raw_caps' list is never used and was just leaking caps.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D979

validate: scenario: always unref srcpad

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D959

validate: media-descriptor-writer: fix pad leaks

- the pad returned by gst_element_get_static_pad() was leaked.
- unref the pad from snode when updating it, not the pad passed as
callback to pad_added_cb()

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D958

validate: clean up action_types list on deinit

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D957

validate: keep a ref when registering an override

_add_override_from_struct() could, in theory, register more than once
the same override so we should not transfer the ref.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D956

validate: use g_object_unref() on GstValidateOverride

Those are GObject subclasses, not GstObject.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D955

validate: add mesa-related valgrind suppressions

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D954

validate: generate valgrind suppression traces

Makes fixing easier as then we can just re-use the generated trace.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D953

validate: fix typo if Xvfb is not installed

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D375

pad-monitor: Don't check for DISCONT flags on buffer in pull-mode

It only makes sense in push-mode

pad-monitor: Update checks for MISSING_DISCONT

* Some SEGMENT might be updates caused by calling gst_pad_set_offset(),
  which will send the same segment but with an updated offset and/or
  based field. For those segments, we don't require a DISCONT on the
  following buffer.
* Ignore differences in flags, they aren't relevant for now to figure
  out whether the segment is an update or not
* Ignore difference in 'position', it's only meant for internal usage
  by elements.
* Changes in the end position (stop in forward playback and start in
  reverse playback) are considering updates

Furthermore, also expect a DISCONT flag on the first buffer following
a STREAM_START.

gst-validate: Avoid overreading array

If we offset the argv table as argument, we need to decrement the number
of items in the array (argc) when iterating it

pad_monitor: Add a check for buffer DISCONT flag

The first buffer after a FLUSH or SEGMENT should have the DISCONT flag
set.

launcher: set gsettings-backend to prevent weird deadlocks

It seems like some sort of forking/dconf/gtype combination can
deadlock occasionally. Setting the gsettings backend to memory
makes it go away.

Same issue: https://cgit.freedesktop.org/gstreamer/gst-plugins-good/commit/tests/check/Makefile.am?id=8e2c1d1de56bddbff22170f8b17473882e0e63f9

validate: add non-seekable fragment file seek tests to the blacklist

The file has no index and is not easily seekable, seeking in these
kind of files isn't implemented.

debug-viewer: Allow running uninstalled with symlink

When uninstalled, we look at the directory of the executable to find
the resources. This patch uses realpath in replacement to abspath so
the path get expended, and symlink are followed.

validate: Stop using deprecated method

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

scenarios: Fix scrub_backward scenarios

We will be changing states, set the handles-state variable accordingly

Added menu opts to filter in instead of only out

Added also menu option to filter a log level and all above that

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

Added support to filter in instead of only out

Added support to filter a log level and all above it

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

filter: add more logging

menu: port menu.popup calls

Add 1 extra arg.

debug-viewer: use the gi GLib version rather than the old gobject glib package

debug-viewer: explicitely require GTK+3

debug-viewer: add more logging and some profiling how-to

The port is much slower than the gtk3 version. Try to figure why.

GstDebugViewer/Timeline: port timeline widgets to gtk3

GstDebugViewer: basic port to gtk3 and python gobject

gst-validate: Link with GIO and clean up CFLAGS/LIBADD/LDFLAGS

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

validate: use new gst_element_class_add_static_pad_template()

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

codecanalyzer: use new gst_element_class_add_static_pad_template()

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

Back to development

Release 1.8.0

validate: launcher: Make sure to properly setup all testsuites

When a first testsuite will set paths, it does not mean that we should
just register following testsuite test manager default tests.

So we need to make a difference between the media paths the user passed
with --media-path and the ones defined by the testsuite.

validate:launcher: Handle testslist files even running several testsuites

Only if those testsuites do not use the same Tester as we
currently can't know to what testsuite a test belongs.

launcher: Avoid caching all the debug logs in memory

And just write the temporary XML file on disc

validate: launcher: Blacklist dash and HLS failling seeking tests

As described in https://bugzilla.gnome.org/show_bug.cgi?id=764020

validate:testsuite: Blacklist validate.hls.playback.scrub_forward_seeking.hls_bibbop as it is racy

validate: pad-monitor: Check right segment after seek

After a seek we need to wait for the right segment (meaning the segment
with seqnum == last seek/flush stop seqnum) to check whether the segment.time
has been properly set.

validate: Accept pad return FLUSHING when the element is being teared down

In the case and element is in READY or is going to READY state, it can
always return GST_FLOW_FLUSHING.

Avoid a race where a demuxer sinkpad has not been set to FLUSHING when we are
still processing a buffer but downstream is already FLUSHING and thus
the demuxer is already returning FLUSHING.

validate: Use GstValidate logging system to print buffering avancement

So it can be used in the launcher.
And make sure to properly parse the info in the launcher.

validate:launcher: Remove dependency on wget

Release 1.7.91

validate: Add missing parenthesis to seek position check

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

validate: Fix overflow seek position comparision

MAX(0, ((gint64) priv->segment_start - priv->seek_pos_tol) will be a high
positive number thanks to being interpreted as unsigned values if
segment_start < seek_pos_tol. Fix this by explicitly checking for this case
and only doing the subtraction otherwise.

This fixes the problem from fdccffbb2e5885b3f8e7369cdbda45b6717ffab0
completely now.

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

validate: Fix seding mistakes

When we added namespace to make GI happy we ended up with
structure called like GstValidateMediaGstValidateMediaXXX.

monitor: Add critical issue for checking accurate seek results

If an accurate seek is accepted, the resulting segment.time should be
exactly the requested seek start value..

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

validate: Fix accurate seeking in paused failling condition

Validate: Fix consider, launch spelling mistakes

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

validate: Fix wrong condition check when adding tests

When listing tests, checking whether uri is present or not and displaying error.
But uri does notneed to be present in case of pipeline generator. So the condition check is wrong.
This results in validateelements testsuite not working. Hence modifying the condition to
not error out on valid cases.

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

Release 1.7.90

validate:launcher: Strip os.pathsep from extra env variables

We might be working with something that is not a path

validate: Add a missing new line to media info files

validate: Fix build

validate:launcher: Add a way to fail if test have been removed/added

validate: Fix annotations

Skipping all functions and methods that are not relevant and not easily
introspectable

Differential Revision: https://phabricator.freedesktop.org/D706

validate: Remove unimplemented function

Differential Revision: https://phabricator.freedesktop.org/D705

validate: Namespace all our structures and objects

Making GI a bit happier.

Those are not stable API anyway...

Differential Revision: https://phabricator.freedesktop.org/D704

validate: Misc annotation and gi friendly cleanups

Differential Revision: https://phabricator.freedesktop.org/D703

validate: Guarantee that we never create 2 monitors for the same object

Differential Revision: https://phabricator.freedesktop.org/D702

validate: Turn GstValidateRunner into a GstTracer

This way we do not need the LD_PRELOAD hack anymore

Add a new libgstvalidateplugin GStreamer plugin, making sure it shares
the exact same code as the library (exposing only the wanted symbols).

Fix the way we set where to install GstValidate plugins

Try to keep backward compatibility even if tracers should never be instantiated
after an GstElement has been instantiated.

Differential Revision: https://phabricator.freedesktop.org/D459

reporter: Properly reset g_log handler when reporter is destroyed

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D699

validate: Make _deinit thread safe

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D698

validate:launcher: handle spaces in transcode output path

When there are spaces in transcoding output path, then it fails. Hence adding the
path in double quotes

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

validate:launcher: Add proper check for is_seekable

The seekable variable in media_info file is of type string. When checking if the file
is seekable using is_seekable, it just returns the string, resulting in it always being true.
It should actually be comparing the string and returning true or false based on comparison

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

validate: fast_forward: Calculate proper playback-time for scenario

In case of fast-forward scenario, the playback-time is not set properly
as per increase in the rate. This is resulting in short media files of duration
less that 15 seconds to fail.

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

codecanalyzer: fix typo leading to implicit decl warning at build-time

codecanalyzer: do not use g_error if abort is not desired

Use g_printerr() instead.

g_error() calls abort after outputting the message
so these blocks' return statements and free()s
were unreachable.

Aditionally, fix wrong void returns on non-void
function, drop trailing whitespace before newline and
add \n's as needed (default handler won't add one).

codecanalyzer: add missing includes for g_printf()

codecanalyzer: Update README URL in help dialog

Previous one was a 404

configure: fix typos

pad-monitor: More fixes of previous commit

You'd think in 2016 compilers could complain when assigning/comparing
different types of enums ... *sigh*.

tools: Fix relative track switching

I have no idea where that "-2" came from, but it was obviously wrong.
Just use modulo "total number of streams" to get the proper track id.

validate: Remove debugging left overs

pad-monitor: Add support for GstPadEventFullFunc

This ensures our sink pad event wrapper is properly called if the
element implement a GstPadEventFullFunc instead of a regular one.

Removes all stray "buffer received before segment" issues with
queue/multiqueue

validate: Fix testsuite

Use fake elements instead of real ones in our tests so that
we control exactly the number of issues generated.

Until now we were trying to hide extra issues with a probe dropping
events and buffers but since 2dfa548f3645844082c3db65d96d87255701b3ad
"pad: Append hooks instead of prepending to call them in the order they were added"
in core, hidding will not work.

g-i: fix init section to avoid compiler warnings

runner: Report criticals when the reporter is in smart mode

media-descriptor-writer: track running time of buffers

PTS and DTS can be deceiving as a change in segment can dramatically change
playback synchronization. Track the running-time as well to properly
get any change in synchronization

Add a toplevel configure script to build components at once

And this way respect https://github.com/cgwalters/build-api

media-check: enable 'full' for files which reference are also 'full'

If the reference file has frames information, enable it automatically
so that the comparison file also has frames to be used.

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

media-descriptor: check if frame data matches

When comparing media descriptors, also check if the frames
match

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

media-descriptor-writer: refactor getting the streamnode by a pad

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

media-descriptor-writer: remove condition from inside loop

Only enter the loop if it indeed has a change of doing something

docs: fix typo

Retrive -> retrieve

validate: Add some documentation and annotiations

validate: Add a 'smart' reporting details mode

Making sure to show all informations for critical issues, but be synthetic for others

scenario: Set the default action execution interval to 10ms

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

codecanalyzer: Fix memory leaks when context parse fails

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

validate: media-descriptor-writer: Don't create media info when stream info 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

validate: launcher: Set scenario manager config before discovering testsuites