analysis: no longer call overridable methods from constructors

Change-Id: I6b8e639f04e42ac840a114690a6ddbc639c5873c
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63650
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

diff --git a/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/internal/analysis/timing/ui/views/segmentstore/density/MouseDragZoomProvider.java b/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/internal/analysis/timing/ui/views/segmentstore/density/MouseDragZoomProvider.java
index 9001d9b38468eeae1729ee910a743374c678db97..97f2a4ef317b3bd452890421bab810d67adf775a 100644 (file)
--- a/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/internal/analysis/timing/ui/views/segmentstore/density/MouseDragZoomProvider.java
+++ b/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/internal/analysis/timing/ui/views/segmentstore/density/MouseDragZoomProvider.java
@@ -49,14 +49,14 @@ public class MouseDragZoomProvider extends BaseMouseProvider implements MouseLis
     }
 
     @Override
-    public void register() {
+    public final void register() {
         getChart().getPlotArea().addMouseListener(this);
         getChart().getPlotArea().addMouseMoveListener(this);
         ((IPlotArea) getChart().getPlotArea()).addCustomPaintListener(this);
     }
 
     @Override
-    public void deregister() {
+    public final void deregister() {
         if (!getChart().isDisposed()) {
             getChart().getPlotArea().removeMouseListener(this);
             getChart().getPlotArea().removeMouseMoveListener(this);

diff --git a/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/internal/analysis/timing/ui/views/segmentstore/density/MouseSelectionProvider.java b/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/internal/analysis/timing/ui/views/segmentstore/density/MouseSelectionProvider.java
index 8bc611bdbbaa029694e79cb75f555d61fb606325..bc756e2f5514df9a9f85ba1f02094521d6ddfdd3 100644 (file)
--- a/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/internal/analysis/timing/ui/views/segmentstore/density/MouseSelectionProvider.java
+++ b/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/internal/analysis/timing/ui/views/segmentstore/density/MouseSelectionProvider.java
@@ -52,14 +52,14 @@ public class MouseSelectionProvider extends BaseMouseProvider implements MouseLi
     }
 
     @Override
-    public void register() {
+    public final void register() {
         getChart().getPlotArea().addMouseListener(this);
         getChart().getPlotArea().addMouseMoveListener(this);
         ((IPlotArea) getChart().getPlotArea()).addCustomPaintListener(this);
     }
 
     @Override
-    public void deregister() {
+    public final void deregister() {
         if (!getChart().isDisposed()) {
             getChart().getPlotArea().removeMouseListener(this);
             getChart().getPlotArea().removeMouseMoveListener(this);

diff --git a/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/internal/analysis/timing/ui/views/segmentstore/density/SimpleTooltipProvider.java b/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/internal/analysis/timing/ui/views/segmentstore/density/SimpleTooltipProvider.java
index f42c0af9a6e20e6e96f43d1c9f6b021f87446eeb..62a6f020dd5ed9cbd596d5963250ff17cd369c91 100644 (file)
--- a/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/internal/analysis/timing/ui/views/segmentstore/density/SimpleTooltipProvider.java
+++ b/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/internal/analysis/timing/ui/views/segmentstore/density/SimpleTooltipProvider.java
@@ -44,12 +44,12 @@ public class SimpleTooltipProvider extends BaseMouseProvider implements MouseTra
     }
 
     @Override
-    public void register() {
+    public final void register() {
         getChart().getPlotArea().addMouseTrackListener(this);
     }
 
     @Override
-    public void deregister() {
+    public final void deregister() {
         if (!getChart().isDisposed()) {
             getChart().getPlotArea().removeMouseTrackListener(this);
         }