Replace handwaving with more realistic steps

Author: foolip

source

@@ -124392,57 +124392,42 @@ dictionary <dfn dictionary>StreamHTMLUnsafeOptions</dfn> {
    <li><p>Let <var>writable</var> be a new WritableStream.</p></li>
 
    <li>
-    <p>👋 Sketch of the streams setup:</p>
+    <p>Let <var>parser</var> be an <span>HTML parser</span> using the <span>HTML fragment parsing
+    algorithm</span> given <var>this</var> as <span data-x="concept-frag-parse-context">context</span>
+    where the input is delivered in chunks instead of a fixed string.</p>
 
-    <ol>
-     <li><p>Incoming chunks must all be of the same type, either strings or a trusted types
-     wrapper. Check this on every chunk and treat mixing as an error.</p></li>
+    <p class="XXX">Refactor the fragment parser algorithm to really support being used in this way,
+    either as an object with an algorithm to feed the parser string chunks, or by making the
+    argument a readable which the parser reads from.</p>
+   </li>
 
-     <li><p>If the first chunk is not a trusted types wrapper and there is a default TT policy,
-     create a transform stream using <code data-x="">createTransformStream</code> from the default
-     policy. Then pipe chunks through that transform stream.</p></li>
+   <li><p>If <var>options</var>["<code data-x="dom-StreamHTMLUnsafeOptions-runScripts">runScripts</code>"],
+   configure <var>parser</var> to not mark scripts as already executed.</p></li>
 
-     <li><p>For trusted types handled "outside" (not by the internal transform stream) check that
-     the chunks are in the same order and not duplicated/filtered/reordered.</p></li>
-    </ol>
-   </li>
+   <li><p>Return <var>writable</var> and continue running the remaining steps <span>in parallel</span>.</p></li>
 
    <li>
-    <p>👋 Sketch of the parser setup:</p>
+    <p>For every <var>chunk</var> that is written to <var>writable</var>:</p>
 
     <ol>
-     <li><p>Let <var>parser</var> be a new fragment parser.</p></li>
-
-     <li><p>Add <span>this</span> to the <span>stack of open elements</span>.</p></li>
+     <li><p>Set <var>input</var> to the stringification of <var>chunk</var>. If this throws, abort
+     <var>writable</var> and abort these steps.</p></li>
 
-     <li><p>If <var>options</var>["<code data-x="dom-StreamHTMLUnsafeOptions-runScripts">runScripts</code>"],
-     don't mark scripts as already executed.</p></li>
+     <li>Place the <var>input</var> into the <span>input stream</span> of <var>parser</var>.</li>
 
-     <li><p>Write chunks into the parser as they are written to <var>writable</var>.</p></li>
+     <li><p>Let <var>parser</var> run until it has consumed all the characters just inserted into
+     the input stream.</p></li>
     </ol>
    </li>
-
-   <li><p>Return <var>writable</var>.</p></li>
   </ol>
   </div>
 
   <div class="example">
    <p>Do a thing like this:</p>
-   <pre><code class="js">const policy = trustedTypes.createPolicy("my-policy", {
-  createTransformStream() {
-    return new TransformStream({
-      transform(chunk, controller) {
-        // TODO: some buffering
-        controller.enqueue(sanitize(chunk));
-      }
-    });
-  }
-});
-
-const response = await fetch('/fragments/something');
-const transform = policy.createTransformStream();
+   <pre><code class="js">const response = await fetch('/fragments/something');
+const decoder = new TextDecoderStream();
 const writable = element.streamHTMLUnsafe();
-await response.body.pipeThrough(transform).pipeTo(writable);</code></pre>
+await response.body.pipeThrough(decoder).pipeTo(writable);</code></pre>
   </div>
 
   <div algorithm>