happy-dom 13.0.5 → 13.0.7

package/src/nodes/node/Node.ts

@@ -11,14 +11,14 @@ import NodeUtility from './NodeUtility.js';
 import IAttr from '../attr/IAttr.js';
 import NodeList from './NodeList.js';
 import INodeList from './INodeList.js';
-import NodeCreationOwnerDocument from '../document/NodeCreationOwnerDocument.js';
+import NodeFactory from '../NodeFactory.js';
 
 /**
  * Node.
  */
 export default class Node extends EventTarget implements INode {
 	// Can be set before the Node is created.
-	public static [PropertySymbol.ownerDocument]: IDocument | null = null;
+	public static [PropertySymbol.ownerDocument]: IDocument | null;
 
 	// Public properties
 	public static readonly ELEMENT_NODE = NodeTypeEnum.elementNode;
@@ -71,13 +71,16 @@ export default class Node extends EventTarget implements INode {
 	 */
 	constructor() {
 		super();
-		if (
-			NodeCreationOwnerDocument.ownerDocument ||
-			(<typeof Node>this.constructor)[PropertySymbol.ownerDocument]
-		) {
-			this.ownerDocument =
-				NodeCreationOwnerDocument.ownerDocument ||
-				(<typeof Node>this.constructor)[PropertySymbol.ownerDocument];
+		if ((<typeof Node>this.constructor)[PropertySymbol.ownerDocument] !== undefined) {
+			this.ownerDocument = (<typeof Node>this.constructor)[PropertySymbol.ownerDocument];
+		} else {
+			const ownerDocument = NodeFactory.pullOwnerDocument();
+			if (!ownerDocument) {
+				throw new Error(
+					'Failed to construct "Node": No owner document in queue. Please use "NodeFactory" to create instances of a Node.'
+				);
+			}
+			this.ownerDocument = ownerDocument;
 		}
 	}
 
@@ -279,9 +282,7 @@ export default class Node extends EventTarget implements INode {
 	 * @returns Cloned node.
 	 */
 	public cloneNode(deep = false): INode {
-		NodeCreationOwnerDocument.ownerDocument = this.ownerDocument;
-		const clone = new (<typeof Node>this.constructor)();
-		NodeCreationOwnerDocument.ownerDocument = null;
+		const clone = NodeFactory.createNode<Node>(this.ownerDocument, <typeof Node>this.constructor);
 
 		// Document has childNodes directly when it is created
 		if (clone[PropertySymbol.childNodes].length) {

package/src/window/BrowserWindow.ts

@@ -150,11 +150,15 @@ import RequestImplementation from '../fetch/Request.js';
 import ResponseImplementation from '../fetch/Response.js';
 import RangeImplementation from '../range/Range.js';
 
-const ORIGINAL_SET_TIMEOUT = setTimeout;
-const ORIGINAL_CLEAR_TIMEOUT = clearTimeout;
-const ORIGINAL_SET_INTERVAL = setInterval;
-const ORIGINAL_CLEAR_INTERVAL = clearInterval;
-const ORIGINAL_QUEUE_MICROTASK = queueMicrotask;
+const TIMER = {
+	setTimeout: setTimeout,
+	clearTimeout: clearTimeout,
+	setInterval: setInterval,
+	clearInterval: clearInterval,
+	queueMicrotask: queueMicrotask,
+	setImmediate: setImmediate,
+	clearImmediate: clearImmediate
+};
 const IS_NODE_JS_TIMEOUT_ENVIRONMENT = setTimeout.toString().includes('new Timeout');
 
 /**
@@ -493,11 +497,6 @@ export default class BrowserWindow extends EventTarget implements IBrowserWindow
 	public readonly [PropertySymbol.readyStateManager] = new DocumentReadyStateManager(this);
 
 	// Private properties
-	#setTimeout: (callback: Function, delay?: number, ...args: unknown[]) => NodeJS.Timeout;
-	#clearTimeout: (id: NodeJS.Timeout) => void;
-	#setInterval: (callback: Function, delay?: number, ...args: unknown[]) => NodeJS.Timeout;
-	#clearInterval: (id: NodeJS.Timeout) => void;
-	#queueMicrotask: (callback: Function) => void;
 	#browserFrame: IBrowserFrame;
 	#innerWidth: number | null = null;
 	#innerHeight: number | null = null;
@@ -516,11 +515,6 @@ export default class BrowserWindow extends EventTarget implements IBrowserWindow
 		super();
 
 		this.#browserFrame = browserFrame;
-		this.#setTimeout = ORIGINAL_SET_TIMEOUT;
-		this.#clearTimeout = ORIGINAL_CLEAR_TIMEOUT;
-		this.#setInterval = ORIGINAL_SET_INTERVAL;
-		this.#clearInterval = ORIGINAL_CLEAR_INTERVAL;
-		this.#queueMicrotask = ORIGINAL_QUEUE_MICROTASK;
 
 		this.customElements = new CustomElementRegistry();
 		this.navigator = new Navigator(this);
@@ -648,6 +642,12 @@ export default class BrowserWindow extends EventTarget implements IBrowserWindow
 		this.XMLDocument = XMLDocument;
 		this.SVGDocument = SVGDocument;
 
+		// Override owner document
+		this.Document[PropertySymbol.ownerDocument] = null;
+		this.HTMLDocument[PropertySymbol.ownerDocument] = null;
+		this.XMLDocument[PropertySymbol.ownerDocument] = null;
+		this.SVGDocument[PropertySymbol.ownerDocument] = null;
+
 		// Document
 		this.document = new HTMLDocument();
 		(<IBrowserWindow>this.document.defaultView) = this;
@@ -961,7 +961,7 @@ export default class BrowserWindow extends EventTarget implements IBrowserWindow
 			!settings ||
 			!settings.disableErrorCapturing ||
 			settings.errorCapture === BrowserErrorCaptureEnum.tryAndCatch;
-		const id = this.#setTimeout(() => {
+		const id = TIMER.setTimeout(() => {
 			if (useTryCatch) {
 				WindowErrorUtility.captureError(this, () => callback(...args));
 			} else {
@@ -984,7 +984,7 @@ export default class BrowserWindow extends EventTarget implements IBrowserWindow
 		if (IS_NODE_JS_TIMEOUT_ENVIRONMENT && (!id || id.constructor.name !== 'Timeout')) {
 			return;
 		}
-		this.#clearTimeout(id);
+		TIMER.clearTimeout(id);
 		this.#browserFrame[PropertySymbol.asyncTaskManager].endTimer(id);
 	}
 
@@ -1002,7 +1002,7 @@ export default class BrowserWindow extends EventTarget implements IBrowserWindow
 			!settings ||
 			!settings.disableErrorCapturing ||
 			settings.errorCapture === BrowserErrorCaptureEnum.tryAndCatch;
-		const id = this.#setInterval(() => {
+		const id = TIMER.setInterval(() => {
 			if (useTryCatch) {
 				WindowErrorUtility.captureError(
 					this,
@@ -1028,7 +1028,7 @@ export default class BrowserWindow extends EventTarget implements IBrowserWindow
 		if (IS_NODE_JS_TIMEOUT_ENVIRONMENT && (!id || id.constructor.name !== 'Timeout')) {
 			return;
 		}
-		this.#clearInterval(id);
+		TIMER.clearInterval(id);
 		this.#browserFrame[PropertySymbol.asyncTaskManager].endTimer(id);
 	}
 
@@ -1044,7 +1044,7 @@ export default class BrowserWindow extends EventTarget implements IBrowserWindow
 			!settings ||
 			!settings.disableErrorCapturing ||
 			settings.errorCapture === BrowserErrorCaptureEnum.tryAndCatch;
-		const id = global.setImmediate(() => {
+		const id = TIMER.setImmediate(() => {
 			if (useTryCatch) {
 				WindowErrorUtility.captureError(this, () => callback(this.performance.now()));
 			} else {
@@ -1067,7 +1067,7 @@ export default class BrowserWindow extends EventTarget implements IBrowserWindow
 		if (IS_NODE_JS_TIMEOUT_ENVIRONMENT && (!id || id.constructor.name !== 'Immediate')) {
 			return;
 		}
-		global.clearImmediate(id);
+		TIMER.clearImmediate(id);
 		this.#browserFrame[PropertySymbol.asyncTaskManager].endImmediate(id);
 	}
 
@@ -1086,7 +1086,7 @@ export default class BrowserWindow extends EventTarget implements IBrowserWindow
 			!settings ||
 			!settings.disableErrorCapturing ||
 			settings.errorCapture === BrowserErrorCaptureEnum.tryAndCatch;
-		this.#queueMicrotask(() => {
+		TIMER.queueMicrotask(() => {
 			if (!isAborted) {
 				if (useTryCatch) {
 					WindowErrorUtility.captureError(this, <() => unknown>callback);

package/cjs/nodes/document/NodeCreationOwnerDocument.cjs

@@ -1,15 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-/**
- * When creating a new node, the ownerDocument property is set to the document object associated with the node's.
- * The ownerDocument property has to be available in the constructor of the node.
- *
- * This is used for setting current ownerDocument state when creating a new node.
- *
- * Another method for achieving this wich is also supported in Node, is to set a static property on the node class.
- * This may be necessary for sub-classes wich are bound to a document, but can cause problems in some cases when Node.js sets this.constructor to Reflect.comnstruct(), which is not the original class.
- */
-exports.default = {
-    ownerDocument: null
-};
-//# sourceMappingURL=NodeCreationOwnerDocument.cjs.map

package/cjs/nodes/document/NodeCreationOwnerDocument.cjs.map

@@ -1 +0,0 @@
-{"version":3,"file":"NodeCreationOwnerDocument.cjs","sourceRoot":"","sources":["../../../src/nodes/document/NodeCreationOwnerDocument.ts"],"names":[],"mappings":";;AAEA;;;;;;;;GAQG;AACH,kBAAoD;IACnD,aAAa,EAAE,IAAI;CACnB,CAAC"}

package/cjs/nodes/document/NodeCreationOwnerDocument.d.ts

@@ -1,15 +0,0 @@
-import IDocument from './IDocument.cjs';
-/**
- * When creating a new node, the ownerDocument property is set to the document object associated with the node's.
- * The ownerDocument property has to be available in the constructor of the node.
- *
- * This is used for setting current ownerDocument state when creating a new node.
- *
- * Another method for achieving this wich is also supported in Node, is to set a static property on the node class.
- * This may be necessary for sub-classes wich are bound to a document, but can cause problems in some cases when Node.js sets this.constructor to Reflect.comnstruct(), which is not the original class.
- */
-declare const _default: {
-    ownerDocument: IDocument | null;
-};
-export default _default;
-//# sourceMappingURL=NodeCreationOwnerDocument.d.ts.map

package/cjs/nodes/document/NodeCreationOwnerDocument.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"NodeCreationOwnerDocument.d.ts","sourceRoot":"","sources":["../../../src/nodes/document/NodeCreationOwnerDocument.ts"],"names":[],"mappings":"AAAA,OAAO,SAAS,MAAM,gBAAgB,CAAC;AAEvC;;;;;;;;GAQG;;mBAC8B,SAAS,GAAG,IAAI;;AAAjD,wBAEE"}

package/lib/nodes/document/NodeCreationOwnerDocument.d.ts

@@ -1,15 +0,0 @@
-import IDocument from './IDocument.js';
-/**
- * When creating a new node, the ownerDocument property is set to the document object associated with the node's.
- * The ownerDocument property has to be available in the constructor of the node.
- *
- * This is used for setting current ownerDocument state when creating a new node.
- *
- * Another method for achieving this wich is also supported in Node, is to set a static property on the node class.
- * This may be necessary for sub-classes wich are bound to a document, but can cause problems in some cases when Node.js sets this.constructor to Reflect.comnstruct(), which is not the original class.
- */
-declare const _default: {
-    ownerDocument: IDocument | null;
-};
-export default _default;
-//# sourceMappingURL=NodeCreationOwnerDocument.d.ts.map

package/lib/nodes/document/NodeCreationOwnerDocument.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"NodeCreationOwnerDocument.d.ts","sourceRoot":"","sources":["../../../src/nodes/document/NodeCreationOwnerDocument.ts"],"names":[],"mappings":"AAAA,OAAO,SAAS,MAAM,gBAAgB,CAAC;AAEvC;;;;;;;;GAQG;;mBAC8B,SAAS,GAAG,IAAI;;AAAjD,wBAEE"}

package/lib/nodes/document/NodeCreationOwnerDocument.js

@@ -1,13 +0,0 @@
-/**
- * When creating a new node, the ownerDocument property is set to the document object associated with the node's.
- * The ownerDocument property has to be available in the constructor of the node.
- *
- * This is used for setting current ownerDocument state when creating a new node.
- *
- * Another method for achieving this wich is also supported in Node, is to set a static property on the node class.
- * This may be necessary for sub-classes wich are bound to a document, but can cause problems in some cases when Node.js sets this.constructor to Reflect.comnstruct(), which is not the original class.
- */
-export default {
-    ownerDocument: null
-};
-//# sourceMappingURL=NodeCreationOwnerDocument.js.map

package/lib/nodes/document/NodeCreationOwnerDocument.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"NodeCreationOwnerDocument.js","sourceRoot":"","sources":["../../../src/nodes/document/NodeCreationOwnerDocument.ts"],"names":[],"mappings":"AAEA;;;;;;;;GAQG;AACH,eAAoD;IACnD,aAAa,EAAE,IAAI;CACnB,CAAC"}

package/src/nodes/document/NodeCreationOwnerDocument.ts

@@ -1,14 +0,0 @@
-import IDocument from './IDocument.js';
-
-/**
- * When creating a new node, the ownerDocument property is set to the document object associated with the node's.
- * The ownerDocument property has to be available in the constructor of the node.
- *
- * This is used for setting current ownerDocument state when creating a new node.
- *
- * Another method for achieving this wich is also supported in Node, is to set a static property on the node class.
- * This may be necessary for sub-classes wich are bound to a document, but can cause problems in some cases when Node.js sets this.constructor to Reflect.comnstruct(), which is not the original class.
- */
-export default <{ ownerDocument: IDocument | null }>{
-	ownerDocument: null
-};