tmf: Bug 476481: NullTimeEvent should not affect time graph entry bounds

This fixes the process entry's Birth time in the Control Flow view.

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

diff --git a/tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/widgets/timegraph/model/TimeGraphEntry.java b/tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/widgets/timegraph/model/TimeGraphEntry.java
index ff0bb2e0b7e79bc95e33b6569def9d7b813724ae..c0b5eef39396c38dda5f1f14bdc6e6a84b21a256 100644 (file)
--- a/tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/widgets/timegraph/model/TimeGraphEntry.java
+++ b/tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/widgets/timegraph/model/TimeGraphEntry.java
@@ -174,6 +174,10 @@ public class TimeGraphEntry implements ITimeGraphEntry {
         } else {
             fEventList.add(event);
         }
+        if (event instanceof NullTimeEvent) {
+            /* A NullTimeEvent should not affect the entry bounds */
+            return;
+        }
         if (fStartTime == SWT.DEFAULT || start < fStartTime) {
             fStartTime = start;
         }
@@ -233,6 +237,10 @@ public class TimeGraphEntry implements ITimeGraphEntry {
         } else if (start == lastStart) {
             fZoomedEventList.set(lastIndex, event);
         }
+        if (event instanceof NullTimeEvent) {
+            /* A NullTimeEvent should not affect the entry bounds */
+            return;
+        }
         if (fStartTime == SWT.DEFAULT || start < fStartTime) {
             fStartTime = start;
         }