From 93f0b4d35bf72a1119b8a5bfea932b85faa1a4a6 Mon Sep 17 00:00:00 2001
From: Yash Ladha <yashladhapankajladha123@gmail.com>
Date: Tue, 16 Mar 2021 22:15:02 +0530
Subject: [PATCH] perf_hooks: add toJSON to performance class

Added toJSON method to the InternalPerformance class as per the
convention followed in other performance classes and per the spec:
https://www.w3.org/TR/hr-time/#tojson-method

Fixes: https://github.com/nodejs/node/issues/37623

PR-URL: https://github.com/nodejs/node/pull/37771
Fixes: https://github.com/nodejs/node/issues/37623
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 doc/api/perf_hooks.md                   |  9 +++++++++
 lib/perf_hooks.js                       | 14 ++++++++++++++
 test/parallel/test-tojson-perf_hooks.js | 14 ++++++++++++++
 3 files changed, 37 insertions(+)
 create mode 100644 test/parallel/test-tojson-perf_hooks.js

diff --git a/doc/api/perf_hooks.md b/doc/api/perf_hooks.md
index 11795b98dc71fa..3609ddfc042bf0 100644
--- a/doc/api/perf_hooks.md
+++ b/doc/api/perf_hooks.md
@@ -261,6 +261,14 @@ If the wrapped function returns a promise, a finally handler will be attached
 to the promise and the duration will be reported once the finally handler is
 invoked.
 
+### `performance.toJSON()`
+<!-- YAML
+added: REPLACEME
+-->
+
+An object which is JSON representation of the `performance` object. It
+is similar to [`window.performance.toJSON`][] in browsers.
+
 ## Class: `PerformanceEntry`
 <!-- YAML
 added: v8.5.0
@@ -1025,4 +1033,5 @@ require('some-module');
 [`child_process.spawnSync()`]: child_process.md#child_process_child_process_spawnsync_command_args_options
 [`process.hrtime()`]: process.md#process_process_hrtime_time
 [`timeOrigin`]: https://w3c.github.io/hr-time/#dom-performance-timeorigin
+[`window.performance.toJSON`]: https://developer.mozilla.org/en-US/docs/Web/API/Performance/toJSON
 [`window.performance`]: https://developer.mozilla.org/en-US/docs/Web/API/Window/performance
diff --git a/lib/perf_hooks.js b/lib/perf_hooks.js
index 1f9a665a3ca006..a92a040f2de839 100644
--- a/lib/perf_hooks.js
+++ b/lib/perf_hooks.js
@@ -59,6 +59,15 @@ class Performance extends EventTarget {
       timeOrigin: this.timeOrigin,
     }, opts)}`;
   }
+
+}
+
+function toJSON() {
+  return {
+    nodeTiming: this.nodeTiming,
+    timeOrigin: this.timeOrigin,
+    eventLoopUtilization: this.eventLoopUtilization()
+  };
 }
 
 class InternalPerformance extends EventTarget {}
@@ -105,6 +114,11 @@ ObjectDefineProperties(Performance.prototype, {
     configurable: true,
     enumerable: true,
     value: timeOriginTimestamp,
+  },
+  toJSON: {
+    configurable: true,
+    enumerable: true,
+    value: toJSON,
   }
 });
 
diff --git a/test/parallel/test-tojson-perf_hooks.js b/test/parallel/test-tojson-perf_hooks.js
new file mode 100644
index 00000000000000..2a716fc4478c29
--- /dev/null
+++ b/test/parallel/test-tojson-perf_hooks.js
@@ -0,0 +1,14 @@
+'use strict';
+
+require('../common');
+const assert = require('assert');
+const { performance } = require('perf_hooks');
+
+// Test toJSON for performance object
+{
+  assert.strictEqual(typeof performance.toJSON, 'function');
+  const jsonObject = performance.toJSON();
+  assert.strictEqual(typeof jsonObject, 'object');
+  assert.strictEqual(jsonObject.timeOrigin, performance.timeOrigin);
+  assert.strictEqual(typeof jsonObject.nodeTiming, 'object');
+}