Fix some bugs

Author: callpraths

assets/js/article/browser-task-queues/components/chronote.js

@@ -9,7 +9,6 @@ import { SetTimeoutByPartsNoteStore } from "../store/set-timeout-by-parts-note-s
 import { AwaitedPromiseNoteStore } from "../store/awaited-promise-note-store.js";
 import { UnawaitedPromiseNoteStore } from "../store/unawaited-promise-note-store.js";
 import { UnawaitedUnmeasuredPromiseNoteStore } from "../store/unawaited-unmeasured-promise-note-store.js";
-import { TimingReporter } from "../store/timing-reporter.js";
 
 export class Chronote extends LitElement {
     static get properties() {
@@ -27,7 +26,6 @@ export class Chronote extends LitElement {
 
     constructor() {
         super();
-        this.timingReporter = new TimingReporter(this);
     }
 
     // No styles in this component.
@@ -68,7 +66,7 @@ export class Chronote extends LitElement {
                 break;
             case "parts":
                 if (this.store === "setTimeoutByParts") {
-                    this.noteStore = new SetTimeoutByPartsNoteStore(this.parts ?? 1);
+                    this.noteStore = new SetTimeoutByPartsNoteStore(this, this.parts ?? 1);
                 }
                 break;
         }
@@ -83,11 +81,7 @@ export class Chronote extends LitElement {
     async saveNotes() {
         this.status = "saving";
 
-        const stopTimer = this.timingReporter.startTimer(
-            "chronotes-save-notes"
-        );
         await this.noteStore.save();
-        stopTimer();
 
         this.status = "ready";
     }

assets/js/article/browser-task-queues/perf-chart.js

@@ -85,4 +85,4 @@ function findOrCreateDataset(chart, label, color) {
     return dataset;
 }
 
-renderPerfGraph("perf-report-graph", ["xc-awaited-promise", "xc-unawaited-promise", "xc-unawaited-unmeasured-promise", "xc-set-timeout"]);
+renderPerfGraph("perf-report-graph", ["xc-awaited-promise", "xc-unawaited-promise", "xc-unawaited-unmeasured-promise", "xc-set-timeout", "xc-set-timeout-by-parts"]);

assets/js/article/browser-task-queues/store/set-timeout-by-parts-note-store.js

@@ -1,27 +1,39 @@
 import { compressParts } from "./common.js";
+import { TimingReporter } from "./timing-reporter.js";
 
 export class SetTimeoutByPartsNoteStore {
-    constructor(parts) {
+    constructor(elem, parts) {
         this.parts = parts;
         this.saveTimerHandles = new Array(parts);
+        this.timingReporter = new TimingReporter(elem);
     }
+
     async save(notes) {
+        const stopTimer = this.timingReporter.startTimer(
+            "set-timeout-by-parts-note-store-save"
+        );
         this.saveTimerHandles.forEach(
             (handle) => handle && clearTimeout(handle)
         );
+        return new Promise(async resolve => {
+            await Promise.all(
+                [...Array(this.parts).keys()].map(
+                    (i) =>
+                        new Promise((resolve) => {
+                            this.saveTimerHandles[i] = setTimeout(() => {
+                                compressParts(notes, this.parts, i);
+                                this.saveTimerHandles[i] = undefined;
+                                resolve();
+                            }, 0);
+                        })
+                )
+            );
+            stopTimer();
+            resolve();
+        })
+
+
 
-        return Promise.all(
-            [...Array(this.parts).keys()].map(
-                (i) =>
-                    new Promise((resolve) => {
-                        this.saveTimerHandles[i] = setTimeout(() => {
-                            compressParts(notes, this.parts, i);
-                            this.saveTimerHandles[i] = undefined;
-                            resolve();
-                        }, 0);
-                    })
-            )
-        );
     }
 }