tmf: Correctly dispose sub-analyses in the statistics module

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

diff --git a/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/views/statistics/TmfStatisticsModule.java b/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/views/statistics/TmfStatisticsModule.java
index 0d03d4e54cacffed3eadc73cb6d71735927d354d..67c580faa2a01bf4d2ec31bc563f6bb2f6810aeb 100644 (file)
--- a/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/views/statistics/TmfStatisticsModule.java
+++ b/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/views/statistics/TmfStatisticsModule.java
@@ -82,6 +82,17 @@ public class TmfStatisticsModule extends TmfAbstractAnalysisModule
     // TmfAbstractAnalysisModule
     // ------------------------------------------------------------------------
 
+    @Override
+    public void dispose() {
+        /*
+         * The sub-analyses are not registered to the trace directly, so we need
+         * to tell them when the trace is disposed.
+         */
+        super.dispose();
+        totalsModule.dispose();
+        eventTypesModule.dispose();
+    }
+
     @Override
     protected boolean executeAnalysis(IProgressMonitor monitor) throws TmfAnalysisException {
         ITmfTrace trace = getTrace();