remove unused options

Author: pabloerhard

doc/api/perf_hooks.md

@@ -1701,18 +1701,12 @@ Passing in a user-defined object instead of the result of a previous call to
 `eventLoopUtilization()` will lead to undefined behavior. The return values
 are not guaranteed to reflect any correct state of the event loop.
 
-## `perf_hooks.monitorEventLoopDelay([options])`
+## `perf_hooks.monitorEventLoopDelay()`
 
 <!-- YAML
 added: v11.10.0
 -->
 
-* `options` {Object}
-  * `resolution` {number} Deprecated. Previously the sampling rate in
-    milliseconds; sampling is now driven directly by the libuv event loop via
-    `uv_prepare_t`/`uv_check_t` hooks, so this value is ignored. It is still
-    validated to preserve backward compatible error behavior. Must be greater
-    than zero. **Default:** `10`.
 * Returns: {ELDHistogram}
 
 _This property is an extension by Node.js. It is not available in Web browsers._
@@ -1728,7 +1722,7 @@ the application is idle.
 ```mjs
 import { monitorEventLoopDelay } from 'node:perf_hooks';
 
-const h = monitorEventLoopDelay({ resolution: 20 });
+const h = monitorEventLoopDelay();
 h.enable();
 // Do something.
 h.disable();
@@ -1743,7 +1737,7 @@ console.log(h.percentile(99));
 
 ```cjs
 const { monitorEventLoopDelay } = require('node:perf_hooks');
-const h = monitorEventLoopDelay({ resolution: 20 });
+const h = monitorEventLoopDelay();
 h.enable();
 // Do something.
 h.disable();
@@ -2038,7 +2032,7 @@ Disables event loop delay sampling when the histogram is disposed.
 ```js
 const { monitorEventLoopDelay } = require('node:perf_hooks');
 {
-  using hist = monitorEventLoopDelay({ resolution: 20 });
+  using hist = monitorEventLoopDelay();
   hist.enable();
   // The histogram will be disabled when the block is exited.
 }

doc/type-map.json

@@ -59,7 +59,7 @@
   "Http2Session": "http2.html#class-http2session",
   "Http2Stream": "http2.html#class-http2stream",
   "Immediate": "timers.html#class-immediate",
-  "IntervalHistogram": "perf_hooks.html#class-intervalhistogram-extends-histogram",
+  "ELDHistogram": "perf_hooks.html#class-eldhistogram-extends-histogram",
   "LockManager": "worker_threads.html#class-lockmanager",
   "KeyAlgorithm": "webcrypto.html#class-keyalgorithm",
   "KeyObject": "crypto.html#class-keyobject",

lib/internal/perf/event_loop_delay.js

@@ -17,21 +17,12 @@ const {
   createELDHistogram,
 } = internalBinding('performance');
 
-const {
-  validateInteger,
-  validateObject,
-} = require('internal/validators');
-
 const {
   Histogram,
   kHandle,
   kMap,
 } = require('internal/histogram');
 
-const {
-  kEmptyObject,
-} = require('internal/util');
-
 const {
   markTransferMode,
 } = require('internal/worker/js_transferable');
@@ -73,18 +64,9 @@ class ELDHistogram extends Histogram {
 }
 
 /**
- * @param {{
- *   resolution : number
- * }} [options]
  * @returns {ELDHistogram}
  */
-function monitorEventLoopDelay(options = kEmptyObject) {
-  // TODO: remove this once backward compatibility is removed
-  validateObject(options, 'options');
-
-  const { resolution = 10 } = options;
-  validateInteger(resolution, 'options.resolution', 1);
-
+function monitorEventLoopDelay() {
   return ReflectConstruct(
     function() {
       markTransferMode(this, true, false);

test/sequential/test-performance-eventloopdelay.js

@@ -19,41 +19,9 @@ const { sleep } = require('internal/util');
   assert(!histogram.disable());
 }
 
-{
-  [null, 'a', 1, false, Infinity].forEach((i) => {
-    assert.throws(
-      () => monitorEventLoopDelay(i),
-      {
-        name: 'TypeError',
-        code: 'ERR_INVALID_ARG_TYPE',
-      }
-    );
-  });
-
-  [null, 'a', false, {}, []].forEach((i) => {
-    assert.throws(
-      () => monitorEventLoopDelay({ resolution: i }),
-      {
-        name: 'TypeError',
-        code: 'ERR_INVALID_ARG_TYPE',
-      }
-    );
-  });
-
-  [-1, 0, 2 ** 53, Infinity].forEach((i) => {
-    assert.throws(
-      () => monitorEventLoopDelay({ resolution: i }),
-      {
-        name: 'RangeError',
-        code: 'ERR_OUT_OF_RANGE',
-      }
-    );
-  });
-}
-
 {
   const s390x = os.arch() === 's390x';
-  const histogram = monitorEventLoopDelay({ resolution: 1 });
+  const histogram = monitorEventLoopDelay();
   histogram.enable();
   let m = 5;
   if (s390x) {