From 8dafaf74702bb7e15b6f653d432ff280f5e875ed Mon Sep 17 00:00:00 2001
From: Anne van Kesteren <annevk@annevk.nl>
Date: Fri, 27 Mar 2020 20:22:13 +0100
Subject: [PATCH] Make adoption account for DocumentFragment nodes with hosts

Tests: ...

Corresponding HTML PR: ...

Closes #813 and closes #814.
---
 dom.bs | 24 ++++++++++++++++++++++--
 1 file changed, 22 insertions(+), 2 deletions(-)

diff --git a/dom.bs b/dom.bs
index 85b43079..575f8ce7 100644
--- a/dom.bs
+++ b/dom.bs
@@ -5260,10 +5260,15 @@ when invoked, must run these steps:
 algorithm is passed <var>node</var> and <var>oldDocument</var>, as indicated in the <a>adopt</a>
 algorithm.
 
-<p>To <dfn export id=concept-node-adopt>adopt</dfn> a <var>node</var> into a <var>document</var>, run
-these steps:
+<p>To <dfn export id=concept-node-adopt>adopt</dfn> a <var>node</var> into a <var>document</var>,
+with an optional <var>forceDocumentFragmentAdoption</var>, run these steps:
 
 <ol>
+ <li>
+  <p>If <var>forceDocumentFragmentAdoption</var> is not given, then set it false.
+
+  <p class=note>This is needed for HTML's <{template}> element.
+
  <li><p>Let <var>oldDocument</var> be <var>node</var>'s <a for=Node>node document</a>.
 
  <li><p>If <var>node</var>'s <a for=tree>parent</a> is non-null, then <a for=/>remove</a>
@@ -5278,6 +5283,15 @@ these steps:
     <a>shadow-including inclusive descendants</a>:
 
     <ol>
+     <li>
+      <p>If <var>forceDocumentFragmentAdoption</var> is false, <var>inclusiveDescendant</var> is a
+      {{DocumentFragment}} <a for=/>node</a>, <var>inclusiveDescendant</var> is <var>node</var>, and
+      <var>node</var>'s <a for=DocumentFragment>host</a> is non-null, then
+      <a for=iteration>continue</a>.
+
+      <p class=note>This is only reasonable as long as all <a>adopt</a> callers remove the children
+      of <var>node</var>.
+
      <li><p>Set <var>inclusiveDescendant</var>'s <a for=Node>node document</a> to <var>document</var>.
 
      <li><p>If <var>inclusiveDescendant</var> is an <a for=/>element</a>, then set the
@@ -5307,6 +5321,12 @@ must run these steps:
  <li><p>If <var>node</var> is a <a for=/>shadow root</a>, then <a>throw</a> a
  "{{HierarchyRequestError!!exception}}" {{DOMException}}.
 
+ <li>
+  <p>If <var>node</var> is a {{DocumentFragment}} <a for=/>node</a> and its
+  <a for=DocumentFragment>host</a> is non-null, then return <var>node</var>.
+
+  <p class=note>Unfortunately this does not throw for web compatibility.
+
  <li><p><a>Adopt</a> <var>node</var> into <a>this</a>.
 
  <li><p>Return <var>node</var>.