perf_hooks: make Performance extend EventTarget

Refs: https://www.w3.org/TR/hr-time/#sec-performance PR-URL: #37621 Backport-PR-URL: #37832 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
nodejs · Mar 29, 2021 · 4dabb99 · 4dabb99
1 parent 76779bf
commit 4dabb99
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/lib/perf_hooks.js b/lib/perf_hooks.js
@@ -54,6 +54,9 @@ const {
   NODE_PERFORMANCE_MILESTONE_ENVIRONMENT
 } = constants;
 
+const {
+  EventTarget,
+} = require('internal/event_target');
 const L = require('internal/linkedlist');
 const kInspect = require('internal/util').customInspectSymbol;
 
@@ -418,8 +421,9 @@ class PerformanceObserver {
   }
 }
 
-class Performance {
+class Performance extends EventTarget {
   constructor() {
+    super();
     this[kIndex] = {
       [kMarks]: new SafeSet()
     };

diff --git a/test/wpt/status/hr-time.json b/test/wpt/status/hr-time.json
@@ -1,7 +1,4 @@
 {
-  "basic.any.js": {
-    "fail": "self.performance.addEventListener is not a function"
-  },
   "idlharness.any.js": {
     "skip": "TODO: update IDL parser"
   },