bootstrap: include event_target into the built-in snapshot · nodejs/node@508e830 · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 633
• Pull requests 592
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit 508e830

Browse filesBrowse the repository at this point in the historyBrowse files

joyeecheung

authored and

targos

committed

bootstrap: include event_target into the built-in snapshot

Since the module has to be loaded during bootstrap anyway. PR-URL: #45849 Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com>

1 parent dd77c05 commit 508e830
Copy full SHA for 508e830

1 file changed

+5-3Lines changed: 5 additions & 3 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• lib/internal/bootstrap
  • browser.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎lib/internal/bootstrap/browser.js‎

Copy file name to clipboardExpand all lines: lib/internal/bootstrap/browser.js

+5-3Lines changed: 5 additions & 3 deletions

Original file line number	Diff line number	Diff line change
@@ -42,9 +42,11 @@ defineOperation(globalThis, 'setTimeout', timers.setTimeout);
42	42	exposeLazyInterfaces(globalThis, 'internal/abort_controller', [
43	43	'AbortController', 'AbortSignal',
44	44	]);
45		- exposeLazyInterfaces(globalThis, 'internal/event_target', [
46		- 'EventTarget', 'Event',
47		- ]);
	45	+ const {
	46	+ EventTarget, Event,
	47	+ } = require('internal/event_target');
	48	+ exposeInterface(globalThis, 'Event', Event);
	49	+ exposeInterface(globalThis, 'EventTarget', EventTarget);
48	50	exposeLazyInterfaces(globalThis, 'internal/worker/io', [
49	51	'MessageChannel', 'MessagePort', 'MessageEvent',
50	52	]);

0 commit comments

Comments

0 (0)