Don't handle TraceWidgetActivatedSignal for the selected experiment

This avoids unnecessary processing and broadcasting of signals if the experiment is already selected. Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
eclipse-cdt-cloud · Oct 19, 2021 · d0ec5c5 · d0ec5c5
1 parent 4a5201f
commit d0ec5c5
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/packages/react-components/src/trace-explorer/trace-explorer-opened-traces-widget.tsx b/packages/react-components/src/trace-explorer/trace-explorer-opened-traces-widget.tsx
@@ -77,9 +77,11 @@ export class ReactOpenTracesWidget extends React.Component<ReactOpenTracesWidget
     }
 
     protected doHandleTracesWidgetActivatedSignal(experiment: Experiment): void {
-        this._selectedExperiment = experiment;
-        const selectedIndex = this.state.openedExperiments.findIndex(openedExperiment => openedExperiment.UUID === experiment.UUID);
-        this.selectExperiment(selectedIndex);
+        if (this._selectedExperiment?.UUID !== experiment.UUID) {
+            this._selectedExperiment = experiment;
+            const selectedIndex = this.state.openedExperiments.findIndex(openedExperiment => openedExperiment.UUID === experiment.UUID);
+            this.selectExperiment(selectedIndex);
+        }
     }
 
     protected doHandleContextMenuEvent(event: React.MouseEvent<HTMLDivElement>, traceUUID: string): void {