Error handling

MicrosoftEdge · Nov 19, 2024 · 69c5921 · 69c5921
1 parent 1f93d63
commit 69c5921
Showing 1 changed file with 16 additions and 17 deletions.
diff --git a/idb-getallrecords/index.html b/idb-getallrecords/index.html
@@ -175,35 +175,34 @@ <h3></h3>
         newReadButton.addEventListener("click", async () => {
           setAsLoading();
           startMeasuring();
-
+          let error = null;
+
           try {
             await readInBatches(db, batchSize);
           } catch (e) {
-            if (useGetAllRecords) {
-              durationLabel.textContent = "Your browser doesn't support getAllRecords() yet.";
-            }
+            error = e;
+          } finally {
+            const duration = stopMeasuring();
+            setAsReady();
+            durationLabel.textContent = error ? "Your browser doesn't support getAllRecords()" : `Read in ${duration}ms`;
           }
-
-          const duration = stopMeasuring();
-          setAsReady();
-          durationLabel.textContent = `Read in ${duration}ms`;
+
         });
-
+        
         newReadReverseButton.addEventListener("click", async () => {
           setAsLoading();
           startMeasuring();
-
+          let error = null;
+
           try {
             await readInBatchesReverse(db, batchSize);
           } catch (e) {
-            if (useGetAllRecords) {
-              durationLabel.textContent = "Your browser doesn't support getAllRecords() yet.";
-            }
+            error = e;
+          } finally { 
+            const duration = stopMeasuring();
+            setAsReady();
+            durationLabel.textContent = error ? "Your browser doesn't support getAllRecords()" : `Read in ${duration}ms`;
           }
-
-          const duration = stopMeasuring();
-          setAsReady();
-          durationLabel.textContent = `Read in ${duration}ms`;
         });
 
         setAsReady();