tmf.ui: Callstack the entry selection done only on the first match.

Fixes issue which caused many requests to be queued before
the sync operation is completed.

Bug: 500615
Change-Id: I563e37a2fb6c0a1cd1b93c801257d43d4ce03c38
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/80177
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI

diff --git a/tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/views/callstack/CallStackView.java b/tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/views/callstack/CallStackView.java
index 5d1efd58fb5d8a80ffb7d458a0a8429afa5b6c6d..32d68196c5ce78bc4b7f5a17b914f9769a723ce4 100644 (file)
--- a/tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/views/callstack/CallStackView.java
+++ b/tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/views/callstack/CallStackView.java
@@ -836,10 +836,10 @@ public class CallStackView extends AbstractTimeGraphView {
                             if (time == stackInterval.getStartTime()) {
                                 ITmfStateValue stackLevelState = stackInterval.getStateValue();
                                 if (stackLevelState.unboxInt() == callStackEntry.getStackLevel() || stackLevelState.isNull()) {
+                                    fSyncSelection = false;
                                     Display.getDefault().asyncExec(() -> {
                                         getTimeGraphCombo().setSelection(callStackEntry);
                                         getTimeGraphViewer().getTimeGraphControl().fireSelectionChanged();
-                                        fSyncSelection = false;
                                     });
                                 }
                             }