snapshot pre-execution

nodejs · Dec 12, 2022 · 9646e62 · 9646e62
1 parent c330a62
commit 9646e62
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/lib/internal/bootstrap/switches/is_main_thread.js b/lib/internal/bootstrap/switches/is_main_thread.js
@@ -311,3 +311,7 @@ require('internal/source_map/source_map_cache');
 require('internal/modules/run_main');
 // Needed to refresh DNS configurations.
 require('internal/dns/utils');
+// Needed by almost all execution modes. It's fine to
+// load them into the snapshot as long as we don't run
+// any of the initialization.
+require('internal/process/pre_execution');
diff --git a/test/fixtures/inspector-global-function.mjs → test/fixtures/inspector-global-function.js b/test/fixtures/inspector-global-function.mjs → test/fixtures/inspector-global-function.js
diff --git a/test/sequential/test-inspector-break-when-eval.js b/test/sequential/test-inspector-break-when-eval.js
@@ -6,9 +6,7 @@ const { NodeInstance } = require('../common/inspector-helper.js');
 const fixtures = require('../common/fixtures');
 const { pathToFileURL } = require('url');
 
-// This needs to be a ESM to work around
-// https://bugs.chromium.org/p/chromium/issues/detail?id=1246905
-const script = fixtures.path('inspector-global-function.mjs');
+const script = fixtures.path('inspector-global-function.js');
 
 async function setupDebugger(session) {
   console.log('[test]', 'Setting up a debugger');