TMF: Register analysis module to parameter provider only if trace active

Sometimes a module will register itself to a parameter provider, but it does
not correspond to the currently active trace, especially for modules from both
traces and experiments. This registers only the module for the currently
selected trace.

Change-Id: If29264c3823968530063cba4c6edd18c679baf69
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/29356
Tested-by: Hudson CI

TMF: TmfAnalysisManager is not meant to be instantiated

Change-Id: I126fa119c035bd8d8f021d78347bf85bd53acbac
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/29406
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

TMF: Add XML unit tests to the alltests package

Change-Id: I086fc0f58e057eaf6044d77c74242818439ac0b5
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/29331
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

ctf: Move the performance tests to the new infrastructure

Convert the existing performance tests to org.eclipse.test.performance
tests.

Change-Id: Idae9706d1e7203c30942c361c55bba9265615682
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/29357
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>

ctf: Flatten fixed-size structs

This patch will unroll nested arrays and structs retaining only floats,
enums and integers in a 1d array. This means we won't need to create
"useless" definitions.

Change-Id: Ia22730b6479df5b4a4f296e609bb98bdc4f1736e
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/25926
Tested-by: Hudson CI

lttng: Add an analysis benchmark

This patch adds a performance test for building a kernel state system.

Change-Id: I5350d30ea4beb7c6678139fd22e1a3432e8b9926
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/26225
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Hudson CI

lttng: Update README for building of LTTng p2 update site

Additionally, add state system plug-ins in list of plug-ins and
change default path name of lttngPackageDestination.

Change-Id: Ia69d532d86861c4246c3808355453d2f227e4b04
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/29029
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

lttng: Update documentation for sel/desel all in Experiment > Select traces

Change-Id: I868d0438525dae657d40ae5b828e73c66d290e8b
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/29007
Tested-by: Hudson CI

tmf: Introduce performance tests to TMF

Add separate source directories in test packages specifically for
performance tests based on org.eclipse.test.performance.

Change-Id: Ia28d5269a6ba81f0727eb2a65858c97566dbf1d9
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/28282
Tested-by: Hudson CI

ctf: Cleanup StructDeclaration

No need to keep separate lists for the fields and values of structs.
A LinkedHashMap preserves the insertion order, and Iterable defines
a specific iteration order.

No apparent performance impact, both versions hover around 975-980 ns/ev
on my machine.

Change-Id: I487b2c756d5827a7489a2ef65e550fb3d018cb72
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/29009
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Hudson CI

ctf: make BitBuffer's Constructor only take non-null ByteBuffers

Change-Id: I70490af35ea9d1ec559828d81fcb6b65b65d12ca
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28685
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

LTTNG: Bug 436857: Keep process selection in CPU Usage tree viewer

After a refresh to the tree's elements, the selected process was lost. Added
a method to set the selection in the abstract tree viewer.

Change-Id: Idc1ff4866593799e6f7571bbd0671364cae0ced1
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/28199
Tested-by: Hudson CI

Build using a target definition file

Change-Id: Ie406d8bfa1953fbee05a70703a63c45a9fda1247
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28392
Reviewed-by: Alexander Kurtakov <akurtako@redhat.com>
Tested-by: Alexander Kurtakov <akurtako@redhat.com>

Stop SWTBot test hangs by forcing dialog focus.

Change-Id: I4374f454e8e74f59649b6cd0381a0201d107e51c
Signed-off-by: Andrew Ferrazzutti <aferrazz@redhat.com>
Reviewed-on: https://git.eclipse.org/r/29002
Tested-by: Hudson CI
Reviewed-by: Alexander Kurtakov <akurtako@redhat.com>
Tested-by: Alexander Kurtakov <akurtako@redhat.com>

lttng: Relayd connector needs to send VIEWER_CLIENT_COMMAND as payload to be able to connect

ConnectResponse.fType should also not be populated from the stream as per
http://bugs.lttng.org/issues/728

Change-Id: Ic146eaf90a6c2135014bc6797c9a3d6f3352597f
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28897
Tested-by: Hudson CI

ctf: make BitBuffer's buffer @NonNull

This accelerates canread and thus every read

Change-Id: I53254b9d7760b9f7a4915751a9fedb4297a5ebb7
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28684

tmf: Some analyses don't call super() in their constructor

Change-Id: I5720736a9f6faa935f11220f077ab45d99a07467
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28895
Tested-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>

tmf: Fix trace package import changing \ to / in Windows

Change-Id: I5ef8ce8d7f6b112e70e85896b8bf76d2b3ea9c6e
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28892
Tested-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

lttng: Some relayd requests/responses have the wrong fixed size

Change-Id: I0783110bd488b15478c14ba7ad71116fa99d7653
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28898
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Hudson CI

ctf: make BitBuffer's ByteBuffer final

No more chances of someone swapping it while you're reading.
Remove an unnecessary alignment issue and add a proper byteorder check.

Change-Id: I344ed316489918dc0751c09c464fada3d4ac02ab
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28681
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

LTTng: Bug 433210: Populate the CPU usage view during analysis

Change-Id: Ieb69ff8999af84134e73a2219663ea0758fe59a8
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/28158
Tested-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

ctf: Introducing the ByteArrayDefinition

This patch introduces a special acceleration that will make sequences
and arrays read faster when it's an array of bytes or a  string, as the
CTF spec specifies that they are aligned on 8 bits and always 8 bits wide
per character.

Sequences and arrays can either be ArrayDefinitions or
ByteArrayDefintions.

This patch gives a 20% acceleration in sequential reading.

Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Change-Id: I7933268b7f9271f74f7d59339ea9b8343fdb021e
Reviewed-on: https://git.eclipse.org/r/26084
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Hudson CI

ss: Protect against special characters in state value strings

Change-Id: Ia0292769b4eab15219632ff0cccfcd5b86e00c97
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/28712
Tested-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>

tmf: Fix simple resource leak warnings

Many warnings left in ControlFlow and Resource Views, where they
get a reference to a (closeable) analysis module, and then the
compiler expects them to close it.

Since the view don't really use the module for anything other than
getting the state system, we could provide a utility class to get
the state system from an analysis module directly, without leaking
references. Shouldn't be hard to do, but outside of the scope of
this patch.

At some point we should revisit the ITmfTrace#getAnalysisModule(s)
methods. Maybe it is not such a good idea to expose closeable objects
publicly.

Change-Id: I0633d40260d0e2d37ad9da30fc6bb34d68cd5f38
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/26486
Tested-by: Hudson CI

TMF: Add a few NonNull annotations to remove warnings

Change-Id: I7408c012894bb6dc6bd92a94ec37fc77d515dd0f
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/28698
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Hudson CI

ctf: remove asserts from bitbuffer

Change-Id: I8a0dcff9f365a313c40032f4b5b4c2b95de91ea6
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28680
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

ctf: Fix simple resource leak warnings

Many warnings left though. For instance, the Stream/StreamInput/
StreamInputReader etc. constructor should create their inner Closeable
objects themselves (instead of having to pass say
new StreamInput(new StreamInputReader(...)) ). That way they can keep
a reference to it, and close() it in their own close() method.

Right now nobody keeps a reference to them, and they never get closed.

Change-Id: I708cfb7d2da12ea4d5c7c8f8a8bbcdf77263ddc1
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/26485
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Hudson CI

lttng: Enable "potential resource leak" warnings

These warnings were disabled, because they often were a bit annoying
to fix with pre-Java 7 syntax. Now that we can use try-with-resources,
those should be as straightforward as the "definite resource leak"
warnings. So we should enable them, and eventually work on fixing them!

Change-Id: Id1b2e63ff6908ca8e34fc07079853922554a0e22
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/26305
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Hudson CI

tmf: Use a Tree viewer, add sel/desel all in Experiment > Select traces

Change-Id: I9fcc86c5b974a871284d6d25878b0fb528ed5a0c
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28596
Tested-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

lttng: Fix hyperlinks in user guide

Change-Id: I19bcb6cec122a6d0c8cd6298628f068a02f9afe2
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28653
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>

TMF: Allow state systems to be built for experiments

Add the support of experiments to the state system build requests

Change-Id: I03b59eade2a9d719bff29359f777b8a21bacf979
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/24879
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

lttng: Disable deprecated-API-in-deprecated-code warning

Allow our deprecated code to call other deprecated APIs. Those are all
planned to be removed at the same time anway. This makes deprecating
APIs much less annoying. Doing so is nicer than outright breaking it!

Change-Id: Id403dad53735a31af67fe48163a7eb7218733ef7
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/28643
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>

lttng: Copy the project settings in the examples plugin too

This also copies the API tool's settings file, but the example plugin
doesn't have the API nature defined. So this file will be useless, but
it's easier to just copy it over than having special blacklists in the
script.

Change-Id: I056654a513bc226b2b39447b522fb2ddd9d0f952
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/28642
Tested-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>

ctf: add notion of ScopedDefinition

extract common code from variants and structs into a ScopedDefinition

Change-Id: I702dc54227efccace892b5787bc93a8a333c3e1a
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28492
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

ctf: Add get(byte[]) to BitBuffer

This adds a bulk reader for data.

Change-Id: I8a43b292b8240d070a94473861b7aca711474661
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/26083
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

btf: Initial Best Trace Format commit

The BTF trace format was defined by Timing Architects for the
Eclipse Automotive IWG.

The specification is available here:
http://wiki.eclipse.org/File:TA_BTF_Specification_2.1.3_Eclipse_Auto_IWG.pdf

Change-Id: I4726279d528d558507d06d2b0e8fd3f0149c810b
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/25892
Tested-by: Hudson CI

lttng: Remember last executed scripts in the Execute script dialog

Change-Id: I8c40713d40f2b8ac205f61f90aa8c68ce575e1e5
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28449
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

TMF: Fix formatting of MANIFEST.MF files

Change-Id: I455c47205fb96052c9699bdb7d5c60c06ee280d1
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/28510
Tested-by: Hudson CI

lttng: Add pde.runtime in target definition files

This allows using the Plug-in spy (Alt-Shift-F1) in the
runtime Eclipse.

Change-Id: I2a90421468f1d54285ee4970c6aa5bc0377fa9df
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28445
Tested-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>

lttng.control: add possibility to run a list of lttng commands

Change-Id: Ib391f02da6980f1d9381212bc448c12de2d69279
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/26757
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Hudson CI

tmf: fix api neglect from patch 28200

Change-Id: Iea78fb157fe51e2425506373df6fc7459366963c
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28491
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

LTTng: Bug 433214: Populate the UST memory view during analysis

The UST memory view is populated as data is available, while the analysis is
being executed.

Change-Id: I75cd4cb2c57f2f29b394bff5adb81c08537c0440
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/28150
Tested-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

TMF: Bug 436877: Set initial range in XY chart to the current range

Instead of using the trace's initial range, which may not correspond to the
currently visible range if the trace was already opened.

Change-Id: Iad5c8e7a579223d4c2c7d98d56c8ae7fb81c5743
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/28154
Tested-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

lttng: Clean help docs when doing a project clean like tmf help

Change-Id: Ieac5e68a020a7ba4190c829d255e26379fabfea9
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27414
Tested-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>

lttng: allow kernel analyses to be more permissive

Change-Id: I466cb013a691082422b7411d8e2e68de3527eb5b
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27825
Tested-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

tracing rcp: Use GTK2 on Linux

Change-Id: Ie088d9ecd4cd5892e0c4c164379ec3057282aa9f
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28237
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

TMF: Bug 416055: Fix behavior of synchronization view

Change-Id: I596e07eb8eb5cc688f0f1810d02f91cfc616a9b1
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/28200
Tested-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

TMF: Fix resource leak when trace indexer is re-created

That happens when a trace was created with the default constructor, a
default indexer is set and when the trace is initialized, a new indexer
is created again, but the previous one is not disposed and the file
remains opened.

Change-Id: I1cb163130563419c81833fd9d383a3eeb2a4f9ee
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/28221
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Hudson CI

tmf: Bug 437000: Queue overflow dragging events table thumb in Linux

Inhibit table refresh on the SWT.NONE slider selection event and perform
refresh on slider mouse up instead.

Change-Id: I38a6e94874f51636533cd6193ed9b2bc3e46b263
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/28235
Tested-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
(cherry picked from commit 66e92beddc67f51046f690d9da93a819fbc89fd0)
Reviewed-on: https://git.eclipse.org/r/28239

lttng: Fix 3.8 and 4.2 target definition for null annotations

Change-Id: Id8e5bac757dcfb6ae56a861b7f7868f238dfc174
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28155
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

[TMF] Bug: 436513 Custom Parser manager: renaming create a new parser and a parser is created

Change-Id: I5750bc051737233dc5aa1b1960df55dbe9f26457
Signed-off-by: Jonathan Rajotte Julien <jonathan.rajotte-julien@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27884
Tested-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
(cherry picked from commit d8757ff23eb659cc981f97ea319a1e1ca0b15960)
Reviewed-on: https://git.eclipse.org/r/28077

tmf: Bug 436751: NPE deleting unexpanded Tracing project

Change-Id: Id3e7cb3cf7152a2e841366c7e3b90853236b9542
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/28081
Tested-by: Hudson CI
(cherry picked from commit c9137e1c8cfb59bbe57677279b992f5ba0cfa864)
Reviewed-on: https://git.eclipse.org/r/28132

tmf: Bug 436738: NPE after deleting default custom parser

- Reload the default custom parser (if it exists) after deleting any
custom parser

- Add null checks when getting a trace type helper from a trace type id

Change-Id: If2b947ff6ce18371e9fdbe5a68f02a1b650fc27a
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/28067
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
(cherry picked from commit ed74d892ad2dfaafd64c43b6a1b619d402780438)
Reviewed-on: https://git.eclipse.org/r/28075

TMF: Add info for output extension point in analysis module documentation

Change-Id: Ia6bf075226a58031d46954af98c192663b347177
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/27986
Tested-by: Hudson CI

TMF: Bug 436576: Add null check on file field in XML state system module

Change-Id: If3eca378edd8e050114c0b14eebc7f943e28e5c5
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/27929
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Hudson CI
(cherry picked from commit c3c922bd0a9373bae387ef9b8c80d90ed8a4db9c)
Reviewed-on: https://git.eclipse.org/r/27985

ctf: Only download test traces if integration-test will execute

For example, if 'mvn package' is executed, the test traces don't
need to be downloaded but 'mvn install' does need them.

Change-Id: I75359969de7a706a29c91dc300a5e3a0ff0e143e
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27885
Tested-by: Hudson CI

tmf-guide: Update view tutorial using latest code

Change-Id: I78c0804750f2f5195a2b12ce8d8348a41f6b4992
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27877
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
(cherry picked from commit 9a572723bbd873eef657f2c40dc01924f3510775)
Reviewed-on: https://git.eclipse.org/r/27950

tmf-ctf: fix NPE in callsite handling (bug 436625)

Change-Id: Iaaecf310226eeb84cfa30c69035eb8f34cda3c91
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27991
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/28025
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>

tmf-rcp: Fix NPE at startup with --open (bug 436239)

Change-Id: Ie393ae61f31c4610373d919e38100ca095c53562
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27585
Tested-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27812

lttng-control: Fix dialog layout problem with GTK3

This patch also works fine with GTK2.

Change-Id: Iaec37cb84ff2815a2a7777ac202a4d05ab78217d
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27551
Tested-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27811

tmf: fix import of traces for type Generic CTF Traces (Bug 436383)

Change-Id: I5299858ba73573ca6a0cf10321b44d6601882c25
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27748
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27775
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>

Set next release to be 3.1.0.

Change-Id: I4731f53db522baccec56da9d51e7a37dc8a71b64
Reviewed-on: https://git.eclipse.org/r/27612
Tested-by: Hudson CI
Reviewed-by: Jeff Johnston <jjohnstn@redhat.com>
Tested-by: Jeff Johnston <jjohnstn@redhat.com>

tmf: Bug 436376: CustomXML Trace Parser Undefined behaviour on closely

The Custom XML parser now properly handles element names which are a
truncation of another element's name.

Change-Id: I5bbf7d4832976fc75d8e60a9ebc0e09d1463bd51
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/27771
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
(cherry picked from commit 33937d3c8a9f55f7f1505a6617cf978536a933ee)
Reviewed-on: https://git.eclipse.org/r/27853

tmf: Bug 436263: BufferedRandomAccessFile read does not return -1 on EOF

Change-Id: I06b056af1249604ec1113035c623a7da3eb90cbd
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/27624
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
(cherry picked from commit acc60287f48004b0293305627040cdf31431b9fe)
Reviewed-on: https://git.eclipse.org/r/27852

tmf: Bug 421475: Inaccurate seek for traces with timestamp transform

The CtfLocation now holds the original (untransformed) timestamp value.

Change-Id: I8f1b435469fd3c9ee32720fc367524d734abb145
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/26750
Tested-by: Hudson CI

tmf: Bug 436491: Fix Synchronize Traces handling

- Refresh supplementary files for each element as soon as they are
created.

- Dispose traces and experiment instances when they are no longer
needed.

- Update the experiment element traces only after instances have been
disposed.

- Fix TmfTraceElement.copy() to use the correct resource according to
its element path.

- Fix TmfTraceElement.copy() to avoid manually creating a new
TmfTraceElement and return the one created by the navigator resource
handler instead.

- Temporarily move synchronization.bin file during experiment update
then restore it to its original location.

Change-Id: I87b3be4dccaf2747fed70eabf63a2f521424b3cb
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/26761
Tested-by: Hudson CI

tmf: Bug 436258: Bookmark icon disappears on deletion even if there is

The TmfEventsTable now supports many bookmarks for the same rank by use
of a multimap. The tool tip is updated to show multiple bookmarks. The
remove and toggle bookmark actions now remove all bookmarks.

Change-Id: Ic22430ed642190e0c7dc58516d7f6759390329ca
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/27746
Tested-by: Hudson CI
(cherry picked from commit 671ac2131c879eba57ca86ab80ee86e5b0c5c554)
Reviewed-on: https://git.eclipse.org/r/27924

tmf: Fix tracetypeui type/experiment elements unnamed in the PDE editor

When an extension point element has many attributes we have to specify
which one to use as label in the PDE editor.

Change-Id: I5d4ad2dafbb3173217df1ac4bf2c1e43bcbb3cbf
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/27586
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Hudson CI
(cherry picked from commit 8acaefc99983781120298365ffbfcb6e19d0bf2c)
Reviewed-on: https://git.eclipse.org/r/27854

lttng-guide: Update for trace folders and removal of batch importing

Also, simplified all the anchor links

Change-Id: Ia91ea6238a31c4aba3f65c88cd9cab94a514f283
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/27532
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Hudson CI
Reviewed-on: https://git.eclipse.org/r/27851

lttng-guide: Update LTTng Kernel Analysis section

Update existing sections (e.g. Control Flow view) and add new section
for CPU Usage analysis and view.

Change-Id: Iee748e2c9837a101760a9b5531a9c0800273378a
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27492
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Hudson CI
Reviewed-on: https://git.eclipse.org/r/27565
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

TMF: Add analysis requirement API documentation

Change-Id: I275461c5514f9a09a46714ad4144a4d34e2d8d61
Signed-off-by: Guilliano Molaire <guilliamo-jaime.molaire@polymtl.ca>

TMF: Add documentation for the XML analysis

Change-Id: I305649c313b33ad22b6bfce4e7b6be322d84c704
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>

lttng-guide: Update Filters View section (cut, copy, paste)

Change-Id: I5680bdb8f378d35e8bc614dfc20382338a288652
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27516
Reviewed-on: https://git.eclipse.org/r/27567
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Fix error message when no trace type is set during open

Change-Id: I87d7a039891af6dbc2e87ed290c8891d3046257f
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27496
Tested-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27538

lttng-guide: Update the Statistics section with percentages

Change-Id: Ic7b0ca1aaef69c4aff4a07791c87c55a38e652b9
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27495
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27529

tmf-guide: update sequence diagram doc

Change-Id: I8db7063b81c82ba25cf4f105a197c4240589efc2
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27407
Tested-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27438

lttng-guide: Update LTTng Tracer Control chapter

The chapter was also moved on chapter level higher to make it more
visible.

Change-Id: Idb6989637b064c1289fdf5c9eab2bb03806825c5
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27408
Tested-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27418

lttng-control: Fix parsing and display of loglevels (bug 429193)

Change-Id: If42746ae361c4e9fe6210c2e1c9f1b253060f58e
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27310
Tested-by: Hudson CI
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27409

tmf: Make CtfTmfTestTrace null-friendly

Change-Id: I1450151fa8562ea52f66e7f1ad67c66c4bd354f9
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/27307
Tested-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

tmf: Fix histogram label background to be theme-friendly

The histogram text controls are now labels and no longer have their
background color set so that they can naturally inherit from their
parent's background whenever the color changes due to active/inactive
view part according to the theme.

The histogram filler area now takes its color from the parent
composite's current background color whenever it is redrawn.

Change-Id: I81ea92863186749dac4e11002b29899d80c4b97a
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/27214
Tested-by: Hudson CI

lttng: Sort the entries in the Resources View

Make the ResourcesEntry Comparable, so that the view can specify
how they should be sorted when entries are added to the list.

This also works for entries that are being added while the trace
is being read ("live" reading).

Change-Id: I5a99652b5b755ea9f2e76e32167ad0d15b76a320
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/27071
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Hudson CI

tmf: Add percentage on Statistics view

Change-Id: I70a565611e230330bc147bb92575d6606aa89311
Signed-off-by: Vincent Perot <vincent.perot@ericsson.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/26144
Tested-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf: Make TmfTraceType static

Can avoid calling getInstance() for every code that uses it.

Change-Id: Ic239ab59c5b2c9bfff2bc9ff0dc38877792a9b45
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/27221
Tested-by: Hudson CI

tmf: Fix custom parser wizard preview table

Since the preview table is built using a temporary trace file with a
fixed name, its persistent index was not being rebuilt when the parser
and/or the preview input was changed.

The preview table is now built using a trace class implementation that
is overridden to use the in-memory checkpoint indexer.

Change-Id: I5cd04f26dea7ab005823317fed68130477bf8e0d
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/27124
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Hudson CI

tmf: hide Batch import wizard

Change-Id: I3d2b0f9187e3870cece8e37addaca3e3aeacdd19
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27222
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Hudson CI

TMF: Display a help text for analyses from the helper

When an analysis cannot be executed, it used to show the same message as if the
trace was not opened. Now it displays a verbose message explaining why the
analysis cannot be executed.

Change-Id: I97f1a79ac5c76362ebb9764b95fc8a7af99f963b
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/26295
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Hudson CI

TMF: Validate if an analysis can be executed using the requirements

Change-Id: If4b51af84c4fc3dc7541629b69e950b0cc54cb00
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/26294
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Hudson CI

TMF: Strike-out analyses that cannot be executed

Also add an interface, ITmfStylecProjectModelElement to allow elements to
change the style of their text.

Change-Id: Iab0fa42b91539d5dbdd4a69917a2b35b3346ca05
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/26293
Tested-by: Hudson CI

tmf: internalize the trace import wizards

Change-Id: I8f0fdf0c47da543f2545b81a77ee7f66cfe33e56
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/26673
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

tmf: Move ImportTraceWizard messages in correct package

Change-Id: I90434653da549e94f5ea10940822930a41f7d20d
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/26672
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

Downgrade to Tycho 0.20.0 to fix build

Tycho 0.21.0-SNAPSHOT seems to fragile right now for us
to use for the Luna release. We needed 0.21 to work around
a bug in IcedTea but we can work around this issue by setting
the SWT_GTK3=0 environment variable when executing the build
instead of having it in the pom.xml. This is less convenient
but more prudent in order to stabilize the build.

Change-Id: I33fc64552c73a0cd8f2eac12fa27afab6af92557
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/27213
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

TMF: Remove XML header classes, they are not useful anymore

With the XmlUtils utility methods, there is no added value to wrap them in
a class and they are accessible from everywhere.

Change-Id: I66d15423ef4dadc18a77caef6af2d63faa426173
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/25584
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

ctf: Rename Stream* classes to CTFStream*

"Stream" will become very confusing with java.util.stream.Stream when
Java 8 comes. Rename those classes to CTFStream, for better namespacing.

Change-Id: Ifb239af0366553472463c50d8e6ed899e002f880
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/27067
Tested-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

lttng: Restrict version of jdt.annotation in Tycho target platform

This works around the problem of jdt.annotation 2.0 being used which
requires Java 8.

Change-Id: I042709d044e40c3eb1220c2cc2a5c5687c06c102
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/26967
Tested-by: Hudson CI

tmf: Use LinkedHashMap in state system Attribute class

By doing that the order in which sub-attributes are added is preserved.
The call stack view showed incorrect stack frames because of that.

Change-Id: I43c344d9d2667de625f9f776d76ab1bff144598a
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/26759
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

tmf: Support tree structure in SelectSupplementaryResourcesDialog

Change-Id: I62ccb5d80ace123621e2df4ddcd58dbf2fbf4d8e
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/26736
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

lttng: Use specific antlr version in targets

To work around bug in Kepler PDE with 0.0.0 versions
not using the latest version.

Change-Id: I34cfd5334e3d1bc4e510f3fa694cb52f0398d8b6
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/26729
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

tmf: Cleanup Statistics View classes a bit

- Remove useless interfaces
- Change protected fields to private, make some of them static
- Use Immutable List/Set where applicable

Change-Id: Ia351e820ea6d5988074d227d369db9f821374026
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/26476
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Remove unneeded null check in SyntheticEventProviderStub

Change-Id: I7df4764683fbb5dbdfdbea3309276930454a0dc9
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/26648
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf: Improve Delete Supplementary Files command

- Make all checkbox elements checked by default in the dialog

- Allow multiple selection as long as at least one of the selected
elements has supplementary files

- Update trace handler to support both traces and experiments, and
delete the experiment handler

- Ensure supplementary files for a particular trace are only shown once
in the list (e.g. same trace selected both under Traces folder and under
experiment, or two experiments selected that contain the same trace)

- Ensure only traces and experiments that have supplementary resources
to delete have their editors closed, and that their project is refreshed
only once.

Change-Id: I91f823c67c15da699c815fdf9daf9cfec5082ec9
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/25858
Tested-by: Hudson CI