validate-launcher: loggable: fix various pyflakes / uncaught pep8 issues.

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

validate-launcher: baseclasses: fix various pyflakes / uncaught pep8 issues.

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

validate-launcher: pep8ify sources.

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

validate-utils: downgrade ERROR to DEBUG.

This function is called in places where it is legit for it
to return NULL.

launcher: add a way to specify an application directory.

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

launcher: Don't implement product-specific TestManagers.

This manager will be moved in GES.

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

validate-launcher: restructure filesystem

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

validate: Fix compiler warning about implicit enum type conversion

gst-validate-reporter.c:119:39: error: implicit conversion from enumeration type
      'GstValidateReportingDetails' to different enumeration type
      'GstValidateInterceptionReturn' [-Werror,-Wenum-conversion]
  GstValidateInterceptionReturn ret = GST_VALIDATE_SHOW_UNKNOWN;
                                ~~~   ^~~~~~~~~~~~~~~~~~~~~~~~~
gst-validate-reporter.c:124:11: error: implicit conversion from enumeration type
      'GstValidateReportingDetails' to different enumeration type
      'GstValidateInterceptionReturn' [-Werror,-Wenum-conversion]
    ret = iface->get_reporting_level (reporter);
        ~ ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
gst-validate-reporter.c:127:10: error: implicit conversion from enumeration type
      'GstValidateInterceptionReturn' to different enumeration type
      'GstValidateReportingDetails' [-Werror,-Wenum-conversion]
  return ret;
  ~~~~~~ ^~~

build: We install all headers system wide for now.

Will be fixed when the API is deemed stable enough

validate: rename GstValidateReportingLevel.

Removes the confusion with GstValidateReportLevel.

Modeled on GstDebugGraphDetails.

validate: Verify that elements always send a segment before pushing EOS

EOS is some kind of data flow and thus a segment event should always be
pushed before the EOS is sent

validate: Properly check that the seqnum of the EOS is always properly set

In the pipeline, an EOS should always have the same seqnum of the
previous SEGMENT event that was received. If the segment is the result
of a seek, it should always be the same as the seek seqnum too.

+ (Mathieu Duponchelle): fix reporting and concatenation tests.

validate: Add support for text based override files

Allowing user to easily determine the severity of issue
types in a config file

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

validate: Remove unused method

gst_media_descriptor_add_frame is not used anywhere

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

validate: Move GstStructure file parsing into utils

So it can be reused, at least in GstValidate.

validate: report: Simplify the issue ID registering using GQuarks

+ Remove unused issue types

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

validate: Check all buffers when we have the info from MediaDescriptor

We now check that each buffer is the expected one for each buffer that
come into the decoder.

+ Fix some minor leaks in test-utils

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

validate:media-descriptor-parser: Add a way to create from a string

So it is simple to make use of it from the testsuite

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

report: g_critical are CRITICAL issues!

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

validate: launcher: Fix printing of errors in final report

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

validate: Add the possibility to generate media infos with frame descs

+ Fix a little issue when the generation fails.

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

validate: MediaDescriptors: Add md5sum to buffer informations

In the media descriptor files, we now have the md5sum of the actual
content of encoded buffers so that we can check that the buffer content is
perfectly what is was supposed to be.

+ Fix the check of whether a frame is a keyframe in the string
  comparison (g_ascii_strcasecmp  return 0 if string matches)

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

validate: Move some method between GstMediaDescriptorParser and GstMediaDescriptor

So that method land where they actually belong.

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

validate: Add a way to pass a MediaDescriptor around monitors

And add an option in gst-validate so that the user can define what
media descriptor file to use.

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

validate:media-descriptor: Handle stream with no tags

It was segfaulting before.

validate: Add more files to gitignore

validate: Move some method between GstMediaDescriptorParser and GstMediaDescriptor

So that method land where they actually belong.

validate-report / reporter: rework the way we repeat issues.

+ runner: update reports count algorithm.

pad-monitor: mark the peer pad as EOS too.

When a sink pad gets EOS, its src pad monitor should also
be marked as EOS (helpful with issue concatenation).

validate-pad-monitor / runner: Check per-object reporting levels.

validate-report: Set conditions in which a report can't be 

validate-report: Add a reporting level field and setter.

validate-runner: implement synthetic report.

+ Fix criticals logic in validate_runner_printf
+ Update padmonitor tests
+ Split validate_report_printf function.

validate-runner: Implement REPORT_NONE for global reporting.

Yeah that was tough. Helpful already though, for example:
GST_VALIDATE_REPORT_LEVEL=none,x:all gst-validate src name=x ! sink
will only report issues reported by the source.

+ Add test.

tests: Check monitors correctly determine their reporting level.

+ [API] gst_validate_reporter_get_reporting_level

validate-runner / monitor: Let the user single out pads.

That's some pretty specific code but it should be helpful.
The following syntax can be used : element-name::pad-name.

+ Free return of gst_object_get_name.

validate-runner / reporter: Sanitize reports refcounting.

The previous code worked but was confusing, the runner didn't actually
take the ref it was releasing later.

+ Fix indentation.

tests: Test reports refcounts.

+ Set the element monitor on the element as qdata.

validate-monitor: Determine the reporting level at setup.

validate-runner: Add code to parse GST_VALIDATE_REPORT_LEVEL.

+ Extend the tests.
+ [API] gst_validate_runner_get_default_reporting_level
+ [API] gst_validate_runner_get_reporting_level_for_name

validate-runner: report-level initial work.

+ Defines reporting levels and document them.
+ Add API to get the default level.
+ fix indentation.
+ fix some typos.
+ Add the beginning of a reporting test.

validate-pad-monitor: concatenate issues.

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

The process is to check for a similar report in intercept_report on
the pads of the upstream element, set that report as the master report
of the intercepted report, and return REPORTER_KEEP instead
of REPORTER_REPORT.

test-utils: add a create_and_monitor element function.

validate-reporter: Add some methods

+ gst_validate_reporter_get_reports
+ gst_validate_reporter_get_reports_count

validate-report: Add the notion of master / shadow reports.

A master report is a report that has been detected by a monitor
to stem from the same issue. It thus contains a list of
"shadow reports" which it will browse when printing itself.

validate-report: Make the ref / unref functions safer.

tests/padmonitor: Correcly strdup the result of get_metadata.

The const pointer was becoming invalid after the first call to add_metadata,
and we ended up setting corrupted data on the second call.

validate-reporter: Add return value to intercept_report.

It will allow to drop, keep or report reports.

validate-pad-monitor: Reimplement reporter interface.

+ Do nothing there for now, except chain up.

validate-reporter: add gst_validate_reporter_get_report.

+ Add locking.

Back to development

Release 1.4.0

validate: Print position if it could properly be queried

Otherwize we will print meaningless garbage.

validate:launcher: Minor enhancement in the documentation

validate: Rename action type playback_time to playback-time

Keeping backward compatiblity with the old naming

validate: Rename gst_validate_add_action_type to gst_validate_register_action_type

The _register naming corresponds much better to what the method does
and makes it more similar to how we refer to this kind of action in
GStreamer.

It is a last minute API change, but that API should not change anymore
after 1.4 is released.

validate: Fix the addition of playback_time in the parameter types

validate: Rename --list-action-types to --inspect-action-type

Making clearer the meaning of the parameter and closer to the
usual naming in the GStreamer land.

validate: Add the 'flags' for the seek action type

This was always a mandatory field but was not documented

validate: mishandled pointer criticals

Free glist of criticals

Signed-off-by: Anuj Jaiswal <anuj.jaiswal@samsung.com>
https://bugzilla.gnome.org/show_bug.cgi?id=736313

validate:docs: Add documentation about the default testsuite

validate-runner: switch to using a GList for the reports.

+ Return a copy of that list in get_reports.
+ update tests.

validate-runner: Hide implementation.

gst-validate-runner: Add locking for the reports list.

Release 1.3.90

validate:launcher: Factorize code to get a MediaDescriptor name for classname

Instead of copy/pasting that code badly

validate/docs: Add location of Scenario/Action defines

validate: (performance issue)refactor to remove duplicate assignment

Signed-off-by: Anuj Jaiswal <anuj.jaiswal@samsung.com>
https://bugzilla.gnome.org/show_bug.cgi?id=736412

validate: docs: Always dist the launcher directoty

Fixing make distcheck

validate: Get the Runner reports in order of arrival

Making sure they are printed in the right order

validate: Start a testsuite

Currently implemented tests are:
  * Settup and cleanup on monitor is done properly
  * Some tests in the PadMonitor are done properly, namely:
        - Buffer before segment
        - Buffer outside segment
        - First buffer running time is always 0
        - The Demuxer flow aggregation is properly checked

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

validate/private: Avoid double typdef

Instead just include required (public and local) header

gst-validate-scenario.h:43:44: error: redefinition of typedef 'GstValidateActionParameter' is a C11 feature [-Werror,-Wtypedef-redefinition]

timeline: disable tooltip while scrolling

Hide some columns by default

columns: optimize cell data functions a little

Fix crash when copying row to clipboard

timeline: fix incorrect position after scrolling using the timeline

Apparently events are dropped internally, so the last position after you stop
dragging can be off.

models: store line offsets in arrays

Use pango markup instead of attributes

Attributes don't work from introspection, so this blocks porting to gtk3.

In MessageColumn, admit that multiple highlighters don't actually work.

findbar: highlight multiple matches in a message

timeline: only redraw updated parts of the graph

Improves rendering performance a lot.

timeline: fix position rectangle missing on first click

Regression caused by previous commit.

timeline: only redraw required areas when updating position rectangle

Fix crash when range filtering

Regression from 25cfe9 (timeline: make log level calculation a lot faster).

timeline: clean up widget drawing

timeline: make log level calculation a lot faster

This is the step where the timeline graph gets colored with the individual log
level colors. It's roughly 4.5 times faster now. Probably can be made even
better, the code also needs a cleanup.

Data: improve stripped log file loading performance

A ~9% improvement for files without colors. This now slightly outperforms the
code before color support was added.

Data: also yield while loading unparseable files

Otherwise, the UI would be blocked while loading something big that is not a
log file at all.

Remove color stripping script

Data: add support for colored log files

Adds a ~5% penalty for loading stripped files.

timeline: remove broken actions from context menu

Hide lines before/after doesn't work as expected in this case.

Data: fix parsing of lines missing filename or function name

E.g. ffmpeg.

timeline: grab when scrolling in TimelineWidget

Also use gdk_event_request_motions.

timeline: move mouse handling into TimelineWidget

timeline: replace self.connect calls with vmethod overrides

setup: fix build

timeline: don't select row when changing position in the timeline

Behaves just like the scrollbar now.

timeline: stop scanning the file while filtering

timeline: small cleanup

Modernize menus a little

A bit in preparation to gtk3 app menus.