snapshot pre-execution

Author: joyeecheung

lib/internal/bootstrap/browser.js

@@ -47,9 +47,6 @@ const {
 } = require('internal/event_target');
 exposeInterface(globalThis, 'Event', Event);
 exposeInterface(globalThis, 'EventTarget', EventTarget);
-// exposeLazyInterfaces(globalThis, 'internal/event_target', [
-//   'EventTarget', 'Event',
-// ]);
 exposeLazyInterfaces(globalThis, 'internal/worker/io', [
   'MessageChannel', 'MessagePort', 'MessageEvent',
 ]);

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');

test/fixtures/inspector-global-function.mjs renamed to test/fixtures/inspector-global-function.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');