#1102@minor: Uses vm.Script for setting globals instead of a string t…

…o improve performance.
capricorn86 · Sep 29, 2023 · 9700666 · 9700666
1 parent c68e51a
commit 9700666
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/packages/happy-dom/src/window/VMGlobalPropertyScript.ts b/packages/happy-dom/src/window/VMGlobalPropertyScript.ts
@@ -1,4 +1,6 @@
-export default `
+import { Script } from 'vm';
+
+export default new Script(`
 this.ArrayBuffer = globalThis.ArrayBuffer;
 this.Boolean = globalThis.Boolean;
 this.DataView = globalThis.DataView;
@@ -57,4 +59,4 @@ this.AbortSignal = globalThis.AbortSignal;
 this.Array = globalThis.Array;
 this.Object = globalThis.Object;
 this.Function = globalThis.Function;
-`;
+`);
diff --git a/packages/happy-dom/src/window/Window.ts b/packages/happy-dom/src/window/Window.ts
@@ -914,7 +914,7 @@ export default class Window extends EventTarget implements IWindow {
 
 			// Sets global properties from the VM to the Window object.
 			// Otherwise "this.Array" will be undefined for example.
-			VM.runInContext(VMGlobalPropertyScript, this);
+			VMGlobalPropertyScript.runInContext(this);
 		}
 	}
 }