validate: spelling fixes

analize != analyze

validate: media-check: add newline to end of print statements

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

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

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

validate:ssim: Make position reporting parseable by the launcher

validate:ssim: Print better information about execution

validate: print REPORTER->name when passed as source in validate_printf

validate:launcher: Avoid printing twice env variables

When printing test command.

validate:tools: Cleanup Makefile.am

Removing useless CFLAGS and LIBS

validate:media-descriptor-writer: cleanup get tag code

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

validate: Do not define GstValidateOverride type twice

validate: Fix make distcheck

validate: Generate documentation for Validate plugins

Summary: Depends on D215

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D216

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

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

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

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

validate: Properly clear the overrides registry on deinit

Summary: Depends on D208

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D209

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

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

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

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

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

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

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

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

validate: remove unused assignment

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D202

Revert "validate-utils: simplify _read_builtin ()"

This breaks the fast_forward scenario parsing.

This reverts commit 0cfff156b1d7013174652cdd25d3ad3f0571813e.

validate-utils: clean error handling in _file_get_lines ()

validate-utils: simplify _read_builtin ()

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

validate:scenario: Make sure to actually test position to execute actions

validate: add valgrind ignore supps for theora encoder

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D181

validate: add more H264 valgrind supp rules

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D180

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

validate:launcher: extra_env_variables is a dictionnary

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

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

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

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

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

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

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

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

validate: Do not unref twice the same list

Summary: Depends on D166

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D167

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

validate: Avoid depending on Gst 1.4

Summary: Depends on D117

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D165

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

validate: Always git submodule update from the toplevel directory

Otherwise it fails with older git versions

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

validate: Go back to the validate dir after submodule init

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

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

validate: disable QOS features when running with valgrind

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D156

validate: move element_has_klass() to utils

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D163

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

validate: allow to pass more than one file to GST_VALIDATE_CONFIG

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D161

validate: gst_validate_plugin_get_config() return 'core' conf if plugin is NULL

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D160

validate: add gst_validate_deinit()

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D159

validate: don't leak not maching config structures

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D158

validate: factor out create_config()

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D157

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

Update .gitignore

validate: unref last_caps when destroying pad monitor

Reviewers: thiblahute

Reviewed By: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D150

validate: fix typo in reverse_playback.scenario

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D148

validate: initialize position

Summary: Fix invalid read when executing without having the actual position.

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D147

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

update gitignore

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D145

use the setup_sink_props_max_lateness config scenario with valgrind

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D141

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

validate: add 'optional' action keyword

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D139

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

Fix 'stoped' typo

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D137

validate: add vg suppression for libdrm bug

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D134

validate: ignore invalid read from libav aac decoding

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D133

validate: ignore libvpx valgrind errors

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D132

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

validate: move look_for_file_in_source_dir and get_valgrind_suppression_file to utils

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D130

validate: Make sure to run submodule init from the root dir

validate:scenario: Fix 'duration' property of the pause action

We preparse it into and set it as GstClockTime in the
structures so make sure to use them as such.

validate:launcher: Concider unset MediaDescriptor duration has 'infinite'

validate: ignore a pixman leak which is fixed in master

http://phabricator.freedesktop.org/D128

validate: use a bigger hammer to ignore mesa related leaks

Looks like some tests are hitting a slightly different code path in udev but
the root bug is the same.

http://phabricator.freedesktop.org/D128

codecanalyzer: minor style fix

codecanalyzer: don't try to free uninitialized pointers

xml_files_path and hex_files_path variable are not initialized.
There are chances that corruption happens when uninitialized
variables are freed, so init them to NULL before use.

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

codecanalyzer: run gst-indent on code

validate: Update autogen.sh

And add a gst-validate.doap file.

Update common submodule

validate: use -debug versions of bins when running from source

Summary:
Those versions are using rpath instead of libtool's wrappers and so will be
faster to start and won't confuse valgrind.

Reviewers: thiblahute

Differential Revision: http://phabricator.freedesktop.org/D116

move scenarios to data/scenarios

Differential Revision: http://phabricator.freedesktop.org/D115

validate: add gstvalidate.supp valgrind suppression file

Differential Revision: http://phabricator.freedesktop.org/D115

validate: scenario: fix structure and action leak

validate: report: fix GString leak when early returning

validate:pipelinemonitor: Print position only when in state >= PAUSED

Reviewers: Mathieu_Du

Reviewed By: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D86

validate:launcher: Use full HD screen as default screen size in xvfb

validate:launcher: Add a way to run tests without displaying the output

Summary:
Adding a --no-display option and running Xvfb virtual frame buffer X
server.

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D105

validate:launcher: Error out if valgrind is not available on the system

Summary:
When the user wants to use valgrind, make sure it is present on the
system before doing anything

Reviewers: gdesmott

Differential Revision: http://phabricator.freedesktop.org/D104

validate:scenario: Stop scenario execution on stop action

And document it properly.

Summary:
The stop action was defined as "setting state to NULL" but
its actual goal is to stop the execution of the scenario. Make sure
that the scenario will not try to execute other actions when that
one has been executed.

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D103

validate: Gracefully handle absence of TAG on streams

Summary: And do not segfault when it happens!

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D99

validate:launcher: Pass GST_VALIDATE_SCENARIO to the subprocess env only

Summary:
And make sure to remove it from the env if the user has it in its main
environment.

Without that commit we ended up passing scenarios from previous tests
to the following ones where None were specified.

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D98

validate: don't pass NULL to gst_caps_copy()