From f0bf37317604061dc60ebf858d6ad4dfc313dcdc 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
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: https://github.com/nodejs/node/pull/37970
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
---
 .eslintrc.js                             |  1 +
 doc/api/globals.md                       |  5 +++++
 lib/internal/bootstrap/node.js           | 23 +++++++++++++++++++++++
 test/common/index.js                     |  4 ++++
 test/parallel/test-global.js             |  1 +
 test/parallel/test-performance-global.js | 18 ++++++++++++++++++
 test/wpt/test-hr-time.js                 |  3 +--
 7 files changed, 53 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..35513e7754e806 100644
--- a/test/common/index.js
+++ b/test/common/index.js
@@ -287,6 +287,10 @@ if (global.gc) {
   knownGlobals.push(global.gc);
 }
 
+if (global.performance) {
+  knownGlobals.push(global.performance);
+}
+
 function allowGlobals(...allowlist) {
   knownGlobals = knownGlobals.concat(allowlist);
 }
diff --git a/test/parallel/test-global.js b/test/parallel/test-global.js
index ff34a812c592da..598f0c7b09f8f4 100644
--- a/test/parallel/test-global.js
+++ b/test/parallel/test-global.js
@@ -47,6 +47,7 @@ builtinModules.forEach((moduleName) => {
     'clearImmediate',
     'clearInterval',
     'clearTimeout',
+    'performance',
     'setImmediate',
     'setInterval',
     'setTimeout'
diff --git a/test/parallel/test-performance-global.js b/test/parallel/test-performance-global.js
new file mode 100644
index 00000000000000..416d017117e9f0
--- /dev/null
+++ b/test/parallel/test-performance-global.js
@@ -0,0 +1,18 @@
+'use strict';
+/* eslint-disable no-global-assign */
+
+require('../common');
+
+const perf_hooks = require('perf_hooks');
+const {
+  strictEqual
+} = require('assert');
+
+const perf = performance;
+strictEqual(globalThis.performance, perf_hooks.performance);
+performance = undefined;
+strictEqual(globalThis.performance, undefined);
+strictEqual(typeof perf_hooks.performance.now, 'function');
+
+// Restore the value of performance for the known globals check
+performance = perf;
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;
 `);