From 8fdf8bf59f4ddd75362f1955282354e7cc2dfdfc Mon Sep 17 00:00:00 2001
From: James M Snell <jasnell@gmail.com>
Date: Mon, 29 Mar 2021 14:09:59 -0700
Subject: [PATCH] perf_hooks: make performance a global

Signed-off-by: James M Snell <jasnell@gmail.com>
---
 .eslintrc.js                             |  1 +
 doc/api/globals.md                       |  5 +++++
 lib/internal/bootstrap/node.js           | 23 +++++++++++++++++++++++
 test/common/index.js                     |  3 +++
 test/parallel/test-performance-global.js | 14 ++++++++++++++
 test/wpt/test-hr-time.js                 |  3 +--
 6 files changed, 47 insertions(+), 2 deletions(-)
 create mode 100644 test/parallel/test-performance-global.js

diff --git a/.eslintrc.js b/.eslintrc.js
index 3f47ddf45d0534..9579162dcf868c 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -331,5 +331,6 @@ module.exports = {
     globalThis: 'readable',
     btoa: 'readable',
     atob: 'readable',
+    performance: 'readable',
   },
 };
diff --git a/doc/api/globals.md b/doc/api/globals.md
index 831ef06bec6094..13e7f63d01b713 100644
--- a/doc/api/globals.md
+++ b/doc/api/globals.md
@@ -279,6 +279,10 @@ The `MessagePort` class. See [`MessagePort`][] for more details.
 
 This variable may appear to be global but is not. See [`module`][].
 
+## `performance`
+
+The [`perf_hooks.performance`][] object.
+
 ## `process`
 <!-- YAML
 added: v0.1.7
@@ -428,6 +432,7 @@ The object that acts as the namespace for all W3C
 [`console`]: console.md
 [`exports`]: modules.md#modules_exports
 [`module`]: modules.md#modules_module
+[`perf_hooks.performance`]: perf_hooks.md#perf_hooks_perf_hooks_performance
 [`process.nextTick()`]: process.md#process_process_nexttick_callback_args
 [`process` object]: process.md#process_process
 [`require()`]: modules.md#modules_require_id
diff --git a/lib/internal/bootstrap/node.js b/lib/internal/bootstrap/node.js
index d8f5c18d12d79b..2c195a6b8547d4 100644
--- a/lib/internal/bootstrap/node.js
+++ b/lib/internal/bootstrap/node.js
@@ -236,6 +236,11 @@ if (!config.noBrowserGlobals) {
 
   defineOperation(global, 'queueMicrotask', queueMicrotask);
 
+  defineLazyGlobal(global, 'performance', () => {
+    const { performance } = require('perf_hooks');
+    return performance;
+  });
+
   // Non-standard extensions:
   defineOperation(global, 'clearImmediate', timers.clearImmediate);
   defineOperation(global, 'setImmediate', timers.setImmediate);
@@ -481,3 +486,21 @@ function defineOperation(target, name, method) {
     value: method
   });
 }
+
+function defineLazyGlobal(target, name, loader) {
+  let value;
+  let overridden = false;
+  ObjectDefineProperty(target, name, {
+    enumerable: true,
+    configurable: true,
+    get() {
+      if (value === undefined && !overridden)
+        value = loader();
+      return value;
+    },
+    set(val) {
+      value = val;
+      overridden = true;
+    }
+  });
+}
diff --git a/test/common/index.js b/test/common/index.js
index 03eec756d0a19a..fa8581c6aee54d 100644
--- a/test/common/index.js
+++ b/test/common/index.js
@@ -274,6 +274,9 @@ let knownGlobals = [
   queueMicrotask,
 ];
 
+if (globalThis.performance)
+  knownGlobals.push(globalThis.performance);
+
 // TODO(@jasnell): This check can be temporary. AbortController is
 // not currently supported in either Node.js 12 or 10, making it
 // difficult to run tests comparitively on those versions. Once
diff --git a/test/parallel/test-performance-global.js b/test/parallel/test-performance-global.js
new file mode 100644
index 00000000000000..f430e29a974952
--- /dev/null
+++ b/test/parallel/test-performance-global.js
@@ -0,0 +1,14 @@
+'use strict';
+/* eslint-disable no-global-assign */
+
+require('../common');
+
+const perf_hooks = require('perf_hooks');
+const {
+  strictEqual
+} = require('assert');
+
+strictEqual(globalThis.performance, perf_hooks.performance);
+performance = undefined;
+strictEqual(globalThis.performance, undefined);
+strictEqual(typeof perf_hooks.performance.now, 'function');
diff --git a/test/wpt/test-hr-time.js b/test/wpt/test-hr-time.js
index 6add176f5fb995..36fdde8036c0cc 100644
--- a/test/wpt/test-hr-time.js
+++ b/test/wpt/test-hr-time.js
@@ -9,8 +9,7 @@ runner.setInitScript(`
   const { Blob } = require('buffer');
   global.Blob = Blob;
 
-  const { performance, PerformanceObserver } = require('perf_hooks');
-  global.performance = performance;
+  const { PerformanceObserver } = require('perf_hooks');
   global.PerformanceObserver = PerformanceObserver;
 `);