Fix #106 - Fix document.body not including content from parsed <body>

Fix #106 Before: ```js new DOMParser().parseFromString(`<html><body><div>asdf</div></body></html>`, "text/html").body.outerHTML // '<body></body>' ``` After: ```js new DOMParser().parseFromString(`<html><body><div>asdf</div></body></html>`, "text/html").body.outerHTML // '<body><div>asdf</div></body>' ```
WebReflection · Feb 9, 2022 · 8edc4e3 · 8edc4e3
1 parent b9774d6
commit 8edc4e3
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 3 deletions.
diff --git a/cjs/shared/parse-from-string.js b/cjs/shared/parse-from-string.js
@@ -59,7 +59,12 @@ const parseFromString = (document, isHTML, markupLanguage) => {
     onopentag(name, attributes) {
       let create = true;
       if (isHTML) {
-        if (ownerSVGElement) {
+        if (name === 'body') {
+          // Accessing the document.body property will create the <body>
+          node = document.body;
+          create = false;
+        }
+        else if (ownerSVGElement) {
           node = append(node, document.createElementNS(SVG_NAMESPACE, name), active);
           node.ownerSVGElement = ownerSVGElement;
           create = false;

diff --git a/esm/shared/parse-from-string.js b/esm/shared/parse-from-string.js
@@ -57,7 +57,12 @@ export const parseFromString = (document, isHTML, markupLanguage) => {
     onopentag(name, attributes) {
       let create = true;
       if (isHTML) {
-        if (ownerSVGElement) {
+        if (name === 'body') {
+          // Accessing the document.body property will create the <body>
+          node = document.body;
+          create = false;
+        }
+        else if (ownerSVGElement) {
           node = append(node, document.createElementNS(SVG_NAMESPACE, name), active);
           node.ownerSVGElement = ownerSVGElement;
           create = false;

diff --git a/test/html/document.js b/test/html/document.js
@@ -48,6 +48,11 @@ assert(document.all[1], document.querySelector('head'));
 assert(document.all[2], document.querySelector('title'));
 assert(document.all[3], document.querySelector('body'));
 
+document = (new DOMParser).parseFromString('<!DOCTYPE html><html><body><div>foo</div></body></html>', 'text/html');
+assert(document.body.tagName, 'BODY');
+assert(document.body, document.querySelector('body'));
+assert(document.body.textContent, 'foo');
+
 // global listener
 let triggered = false;
 window.addEventListener('test', function once() {

diff --git a/worker.js b/worker.js
@@ -8742,7 +8742,12 @@ const parseFromString = (document, isHTML, markupLanguage) => {
     onopentag(name, attributes) {
       let create = true;
       if (isHTML) {
-        if (ownerSVGElement) {
+        if (name === 'body') {
+          // Accessing the document.body property will create the <body>
+          node = document.body;
+          create = false;
+        }
+        else if (ownerSVGElement) {
           node = append$1(node, document.createElementNS(SVG_NAMESPACE, name), active);
           node.ownerSVGElement = ownerSVGElement;
           create = false;