perf(ext/web): optimize timer cancellation (#16316)

Towards #16315 

It created a bunch of Error objects and rejected the promise. This patch
changes `op_sleep` to resolve with `true` if it was cancelled.

1 files changed, 6 insertions, 10 deletions

diff --git a/ext/web/02_timers.js b/ext/web/02_timers.js
index 7c0bc1e3a..0fc808796 100644
--- a/ext/web/02_timers.js
+++ b/ext/web/02_timers.js
@@ -18,7 +18,6 @@
     // deno-lint-ignore camelcase
     NumberPOSITIVE_INFINITY,
     PromisePrototypeThen,
-    ObjectPrototypeIsPrototypeOf,
     SafeArrayIterator,
     SymbolFor,
     TypeError,
@@ -245,7 +244,12 @@
     // 1.
     PromisePrototypeThen(
       sleepPromise,
-      () => {
+      (cancelled) => {
+        if (!cancelled) {
+          // The timer was cancelled.
+          removeFromScheduledTimers(timerObject);
+          return;
+        }
         // 2. Wait until any invocations of this algorithm that had the same
         // global and orderingIdentifier, that started before this one, and
         // whose milliseconds is equal to or less than this one's, have
@@ -278,14 +282,6 @@
           currentEntry = currentEntry.next;
         }
       },
-      (err) => {
-        if (ObjectPrototypeIsPrototypeOf(core.InterruptedPrototype, err)) {
-          // The timer was cancelled.
-          removeFromScheduledTimers(timerObject);
-        } else {
-          throw err;
-        }
-      },
     );
   }