lazy load trace_timer

Author: H4ad

lib/internal/console/constructor.js

@@ -60,10 +60,22 @@ const {
   CHAR_UPPERCASE_C: kTraceCount,
 } = require('internal/constants');
 const kCounts = Symbol('counts');
-const { time, timeEnd, timeLog } = require('internal/util/trace_timer');
 
 const kTraceConsoleCategory = 'node,node.console';
 
+let _traceTimer;
+/**
+ * @returns {import('internal/util/trace_timer')}
+ */
+function lazyTraceTimer() {
+  if (_traceTimer) {
+    return _traceTimer;
+  }
+
+  _traceTimer = require('internal/util/trace_timer');
+  return _traceTimer;
+}
+
 const kMaxGroupIndentation = 1000;
 
 // Lazy loaded for startup performance.
@@ -393,21 +405,21 @@ const consoleMethods = {
   },
 
   time(label = 'default') {
-    time(this._times, kTraceConsoleCategory, 'console.time()', label, `time::${label}`);
+    lazyTraceTimer().time(this._times, kTraceConsoleCategory, 'console.time()', label, `time::${label}`);
   },
 
   timeEnd(label = 'default') {
     if (this[kInternalTimeLogImpl] === undefined)
       this[kInternalTimeLogImpl] = FunctionPrototypeBind(timeLogImpl, this);
 
-    timeEnd(this._times, kTraceConsoleCategory, 'console.timeEnd()', this[kInternalTimeLogImpl], label, `time::${label}`);
+    lazyTraceTimer().timeEnd(this._times, kTraceConsoleCategory, 'console.timeEnd()', this[kInternalTimeLogImpl], label, `time::${label}`);
   },
 
   timeLog(label = 'default', ...data) {
     if (this[kInternalTimeLogImpl] === undefined)
       this[kInternalTimeLogImpl] = FunctionPrototypeBind(timeLogImpl, this);
 
-    timeLog(this._times, kTraceConsoleCategory, 'console.timeLog()', this[kInternalTimeLogImpl], label, `time::${label}`, data);
+    lazyTraceTimer().timeLog(this._times, kTraceConsoleCategory, 'console.timeLog()', this[kInternalTimeLogImpl], label, `time::${label}`, data);
   },
 
   trace: function trace(...args) {

lib/internal/util/debuglog.js

@@ -11,9 +11,18 @@ const {
 } = primordials;
 
 const { inspect, format, formatWithOptions } = require('internal/util/inspect');
-const { time, timeLog, timeEnd } = require('internal/util/trace_timer');
 const { isTraceCategoryEnabled } = internalBinding('trace_events');
 
+let _traceTimer;
+function lazyTraceTimer() {
+  if (_traceTimer) {
+    return _traceTimer;
+  }
+
+  _traceTimer = require('internal/util/trace_timer');
+  return _traceTimer;
+}
+
 // `debugImpls` and `testEnabled` are deliberately not initialized so any call
 // to `debuglog()` before `initializeDebugEnv()` is called will throw.
 let debugImpls;
@@ -210,7 +219,7 @@ function debugWithTimer(set, cb) {
       return;
     }
 
-    time(tracesStores[set], kTraceCategory, 'debuglog.time', logLabel, traceLabel);
+    lazyTraceTimer().time(tracesStores[set], kTraceCategory, 'debuglog.time', logLabel, traceLabel);
   }
 
   /**
@@ -221,7 +230,7 @@ function debugWithTimer(set, cb) {
       return;
     }
 
-    timeEnd(tracesStores[set], kTraceCategory, 'debuglog.timeEnd', selectedLogImpl, logLabel, traceLabel);
+    lazyTraceTimer().timeEnd(tracesStores[set], kTraceCategory, 'debuglog.timeEnd', selectedLogImpl, logLabel, traceLabel);
   }
 
   /**
@@ -232,7 +241,7 @@ function debugWithTimer(set, cb) {
       return;
     }
 
-    timeLog(tracesStores[set], kTraceCategory, 'debuglog.timeLog', selectedLogImpl, logLabel, traceLabel, args);
+    lazyTraceTimer().timeLog(tracesStores[set], kTraceCategory, 'debuglog.timeLog', selectedLogImpl, logLabel, traceLabel, args);
   }
 
   function init() {

test/parallel/test-bootstrap-modules.js

@@ -44,7 +44,6 @@ expected.beforePreExec = new Set([
   'NativeModule internal/assert',
   'NativeModule internal/util/inspect',
   'NativeModule internal/util/debuglog',
-  'NativeModule internal/util/trace_timer',
   'NativeModule internal/timers',
   'NativeModule events',
   'Internal Binding buffer',