fix(web/worker) define event handlers in DOM order (#8334)

1 files changed, 5 insertions, 10 deletions

diff --git a/cli/rt/11_workers.js b/cli/rt/11_workers.js
index 36a2fd61b..62210dfae 100644
--- a/cli/rt/11_workers.js
+++ b/cli/rt/11_workers.js
@@ -4,6 +4,7 @@
   const core = window.Deno.core;
   const { Window } = window.__bootstrap.globalInterfaces;
   const { log } = window.__bootstrap.util;
+  const { defineEventHandler } = window.__bootstrap.webUtil;
 
   function createWorker(
     specifier,
@@ -87,9 +88,6 @@
           cancelable: false,
           data,
         });
-        if (this.onmessageerror) {
-          this.onmessageerror(msgErrorEvent);
-        }
         return;
       }
 
@@ -98,10 +96,6 @@
         data,
       });
 
-      if (this.onmessage) {
-        this.onmessage(msgEvent);
-      }
-
       this.dispatchEvent(msgEvent);
     };
 
@@ -116,9 +110,6 @@
       });
 
       let handled = false;
-      if (this.onerror) {
-        this.onerror(event);
-      }
 
       this.dispatchEvent(event);
       if (event.defaultPrevented) {
@@ -205,6 +196,10 @@
     }
   }
 
+  defineEventHandler(Worker.prototype, "error");
+  defineEventHandler(Worker.prototype, "message");
+  defineEventHandler(Worker.prototype, "messageerror");
+
   window.__bootstrap.worker = {
     Worker,
   };