events: use property, primordials

PR-URL: #33775
Fixes: #33773
Reviewed-By: Denys Otrishko <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Zeyu Yang <[email protected]>
Reviewed-By: James M Snell <[email protected]>

Author: benjamingr

lib/internal/event_target.js

@@ -8,6 +8,7 @@ const {
   Set,
   Symbol,
   NumberIsNaN,
+  SymbolToStringTag,
 } = primordials;
 
 const {
@@ -67,7 +68,6 @@ class Event {
     // isTrusted is special (LegacyUnforgeable)
     Object.defineProperty(this, 'isTrusted', {
       get() { return false; },
-      set(ignoredValue) { return false; },
       enumerable: true,
       configurable: false
     });
@@ -132,10 +132,15 @@ class Event {
   stopPropagation() {
     this.#propagationStopped = true;
   }
-
-  get [Symbol.toStringTag]() { return 'Event'; }
 }
 
+Object.defineProperty(Event.prototype, SymbolToStringTag, {
+  writable: false,
+  enumerable: false,
+  configurable: true,
+  value: 'Event',
+});
+
 // The listeners for an EventTarget are maintained as a linked list.
 // Unfortunately, the way EventTarget is defined, listeners are accounted
 // using the tuple [handler,capture], and even if we don't actually make
@@ -305,14 +310,19 @@ class EventTarget {
 
     return `${name} ${inspect({}, opts)}`;
   }
-  get [Symbol.toStringTag]() { return 'EventTarget'; }
 }
 
 Object.defineProperties(EventTarget.prototype, {
   addEventListener: { enumerable: true },
   removeEventListener: { enumerable: true },
   dispatchEvent: { enumerable: true }
 });
+Object.defineProperty(EventTarget.prototype, SymbolToStringTag, {
+  writable: false,
+  enumerable: false,
+  configurable: true,
+  value: 'EventTarget',
+});
 
 class NodeEventTarget extends EventTarget {
   static defaultMaxListeners = 10;