events: add null check for the signal of EventTarget

lib/internal/event_target.js

@@ -34,7 +34,7 @@ const {
     ERR_INVALID_THIS,
   }
 } = require('internal/errors');
-const { validateObject, validateString, validateInternalField } = require('internal/validators');
+const { validateAbortSignal, validateObject, validateString, validateInternalField } = require('internal/validators');
 
 const {
   customInspectSymbol,
@@ -575,6 +575,8 @@ class EventTarget {
     }
     type = String(type);
 
+    validateAbortSignal(signal, 'options.signal');
+
     if (signal) {
       if (signal.aborted) {
         return;

test/parallel/test-whatwg-events-add-event-listener-options-signal.js

@@ -4,6 +4,7 @@ require('../common');
 
 const {
   strictEqual,
+  throws,
 } = require('assert');
 
 // Manually ported from: wpt@dom/events/AddEventListenerOptions-signal.any.js
@@ -157,3 +158,11 @@ const {
   }, { once: true });
   et.dispatchEvent(new Event('foo'));
 }
+{
+  const et = new EventTarget();
+  [1, 1n, {}, [], null, true, 'hi', Symbol(), () => {}].forEach((signal) => {
+    throws(() => et.addEventListener('foo', () => {}, { signal }), {
+      name: 'TypeError',
+    });
+  });
+}

test/wpt/status/dom/events.json

@@ -11,7 +11,6 @@
   "AddEventListenerOptions-signal.any.js": {
     "fail": {
       "expected": [
-        "Passing null as the signal should throw",
         "Passing null as the signal should throw (listener is also null)"
       ]
     }