histogram: refactor to use min/max of validateNumber

Fix NumberIsNaN is called two times and change error code to ERR_OUT_OF_RANGE when RangeError is occurred in test.

Author: deokjinkim

lib/internal/histogram.js

@@ -186,10 +186,7 @@ class Histogram {
  percentile(percentile) {
  if (!isHistogram(this))
  throw new ERR_INVALID_THIS('Histogram');
- validateNumber(percentile, 'percentile');
-
- if (NumberIsNaN(percentile) || percentile <= 0 || percentile > 100)
- throw new ERR_INVALID_ARG_VALUE.RangeError('percentile', percentile);
+ validateNumber(percentile, 'percentile', 1, 100);
 
  return this[kHandle]?.percentile(percentile);
  }
@@ -201,10 +198,7 @@ class Histogram {
  percentileBigInt(percentile) {
  if (!isHistogram(this))
  throw new ERR_INVALID_THIS('Histogram');
- validateNumber(percentile, 'percentile');
-
- if (NumberIsNaN(percentile) || percentile <= 0 || percentile > 100)
- throw new ERR_INVALID_ARG_VALUE.RangeError('percentile', percentile);
+ validateNumber(percentile, 'percentile', 1, 100);
 
  return this[kHandle]?.percentileBigInt(percentile);
  }

test/sequential/test-performance-eventloopdelay.js

@@ -91,7 +91,7 @@ const { sleep } = require('internal/util');
  () => histogram.percentile(i),
  {
  name: 'RangeError',
- code: 'ERR_INVALID_ARG_VALUE'
+ code: 'ERR_OUT_OF_RANGE'
  }
  );
  });