Reset fetching flag in a try-finally statement

Fixes #66 Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
eclipse-cdt-cloud · Nov 24, 2020 · 35a92bf · 35a92bf
1 parent 7bde344
commit 35a92bf
Showing 1 changed file with 14 additions and 10 deletions.
diff --git a/timeline-chart/src/layer/time-graph-chart.ts b/timeline-chart/src/layer/time-graph-chart.ts
@@ -188,18 +188,22 @@ export class TimeGraphChart extends TimeGraphChartLayer {
             update
         )) {
             this.fetching = true;
-            const rowData = await this.providers.dataProvider(viewRange, resolution);
-            if (rowData) {
-                this.providedResolution = rowData.resolution;
-                this.providedRange = rowData.range;
-                this.setRowModel(rowData.rows);
-                this.removeChildren();
-                this.addRows(this.rows, this.rowController.rowHeight);
-                if (this.isNavigating) {
-                    this.selectStateInNavigation();
+            try {
+                const rowData = await this.providers.dataProvider(viewRange, resolution);
+                if (rowData) {
+                    this.providedResolution = rowData.resolution;
+                    this.providedRange = rowData.range;
+                    this.setRowModel(rowData.rows);
+                    this.removeChildren();
+                    this.addRows(this.rows, this.rowController.rowHeight);
+                    if (this.isNavigating) {
+                        this.selectStateInNavigation();
+                    }
                 }
+            } finally {
+                this.fetching = false;
+                this.isNavigating = false;
             }
-            this.fetching = false;
         }
     }