Do not track location information for fragment-parsed nodes

Fixes #2968.
jsdom · Dec 2, 2021 · f9de3fd · f9de3fd
1 parent a61fdb8
commit f9de3fd
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 0 deletions.
diff --git a/lib/jsdom/browser/parser/html.js b/lib/jsdom/browser/parser/html.js
@@ -202,6 +202,7 @@ function parseFragment(markup, contextElement) {
 
   const config = {
     ...ownerDocument._parseOptions,
+    sourceCodeLocationInfo: false,
     treeAdapter: new JSDOMParse5Adapter(ownerDocument, { fragment: true })
   };
 

diff --git a/test/api/methods.js b/test/api/methods.js
@@ -133,6 +133,39 @@ describe("API: JSDOM class's methods", () => {
         endOffset: 36
       });
     });
+
+    it("should return undefined for nodes created by innerHTML", () => {
+      const dom = new JSDOM(`<p>Hello</p>`, { includeNodeLocations: true });
+      const para = dom.window.document.querySelector("p");
+
+      para.innerHTML = `<div></div>`;
+
+      const div = dom.window.document.querySelector("div");
+
+      assert.deepEqual(dom.nodeLocation(div), undefined);
+    });
+
+    it("should return undefined for nodes created by outerHTML", () => {
+      const dom = new JSDOM(`<p>Hello</p>`, { includeNodeLocations: true });
+      const para = dom.window.document.querySelector("p");
+
+      para.outerHTML = `<div></div>`;
+
+      const div = dom.window.document.querySelector("div");
+
+      assert.deepEqual(dom.nodeLocation(div), undefined);
+    });
+
+    it("should return undefined for nodes created by createContextualFragment", () => {
+      const dom = new JSDOM("", { includeNodeLocations: true });
+      const range = dom.window.document.createRange();
+
+      const fragment = range.createContextualFragment(`<p>Hello</p>`);
+
+      const node = fragment.querySelector("p");
+
+      assert.deepEqual(dom.nodeLocation(node), undefined);
+    });
   });
 
   describe("getInternalVMContext", { skipIfBrowser: true }, () => {