[html5] r8108 - [giow] (3) Adjust the Adoption Agency Algorithm to not reverse the order of node [...]

whatwg at whatwg.org whatwg at whatwg.org
Wed Jul 31 13:16:43 PDT 2013


Author: ianh
Date: 2013-07-31 13:16:42 -0700 (Wed, 31 Jul 2013)
New Revision: 8108

Modified:
   complete.html
   index
   source
Log:
[giow] (3) Adjust the Adoption Agency Algorithm to not reverse the order of nodes in the document, by removing nodes that we're not recreating from the stack of open elements.
Affected topics: HTML Syntax and Parsing

Modified: complete.html
===================================================================
--- complete.html	2013-07-31 19:54:23 UTC (rev 8107)
+++ complete.html	2013-07-31 20:16:42 UTC (rev 8108)
@@ -91534,24 +91534,23 @@
 
     <ol><li><p>Let <var title="">inner loop counter</var> be zero.</li>
 
-     <li><p><i>Inner loop</i>: If <var title="">inner loop counter</var> is greater than or equal to
-     three, then go to the next step in the overall algorithm.</li>
+     <li><p><i>Inner loop</i>: Increment <var title="">inner loop counter</var> by one.</li>
 
-     <li><p>Increment <var title="">inner loop counter</var> by one.</li>
-
      <li><p>Let <var title="">node</var> be the element immediately above <var title="">node</var>
      in the <a href=#stack-of-open-elements>stack of open elements</a>, or if <var title="">node</var> is no longer in the
      <a href=#stack-of-open-elements>stack of open elements</a> (e.g. because it got removed by this algorithm<!-- in
      particular, the step labeled "removal" below -->), the element that was immediately above <var title="">node</var> in the <a href=#stack-of-open-elements>stack of open elements</a> before <var title="">node</var>
      was removed.</li>
 
+     <li><p>If <var title="">node</var> is <var title="">formatting element</var>, then go to the
+     next step in the overall algorithm.</li>
+
+     <li><p>If <var title="">inner loop counter</var> is greater than three and <var title="">node</var> is in the <a href=#list-of-active-formatting-elements>list of active formatting elements</a>, then remove <var title="">node</var> from the <a href=#list-of-active-formatting-elements>list of active formatting elements</a>.</li>
+
      <li><p><!-- "removal" step: -->If <var title="">node</var> is not in the <a href=#list-of-active-formatting-elements>list of active
      formatting elements</a>, then remove <var title="">node</var> from the <a href=#stack-of-open-elements>stack of open
      elements</a> and then go back to the step labeled <i>inner loop</i>.</li>
 
-     <li><p>Otherwise, if <var title="">node</var> is <var title="">formatting element</var>, then
-     go to the next step in the overall algorithm.</li>
-
      <li><p><a href=#create-an-element-for-the-token>Create an element for the token</a> for which the element <var title="">node</var> was created, with <var title="">common ancestor</var> as the intended
      parent; replace the entry for <var title="">node</var> in the <a href=#list-of-active-formatting-elements>list of active formatting
      elements</a> with an entry for the new element, replace the entry for <var title="">node</var> in the <a href=#stack-of-open-elements>stack of open elements</a> with an entry for the new

Modified: index
===================================================================
--- index	2013-07-31 19:54:23 UTC (rev 8107)
+++ index	2013-07-31 20:16:42 UTC (rev 8108)
@@ -91534,24 +91534,23 @@
 
     <ol><li><p>Let <var title="">inner loop counter</var> be zero.</li>
 
-     <li><p><i>Inner loop</i>: If <var title="">inner loop counter</var> is greater than or equal to
-     three, then go to the next step in the overall algorithm.</li>
+     <li><p><i>Inner loop</i>: Increment <var title="">inner loop counter</var> by one.</li>
 
-     <li><p>Increment <var title="">inner loop counter</var> by one.</li>
-
      <li><p>Let <var title="">node</var> be the element immediately above <var title="">node</var>
      in the <a href=#stack-of-open-elements>stack of open elements</a>, or if <var title="">node</var> is no longer in the
      <a href=#stack-of-open-elements>stack of open elements</a> (e.g. because it got removed by this algorithm<!-- in
      particular, the step labeled "removal" below -->), the element that was immediately above <var title="">node</var> in the <a href=#stack-of-open-elements>stack of open elements</a> before <var title="">node</var>
      was removed.</li>
 
+     <li><p>If <var title="">node</var> is <var title="">formatting element</var>, then go to the
+     next step in the overall algorithm.</li>
+
+     <li><p>If <var title="">inner loop counter</var> is greater than three and <var title="">node</var> is in the <a href=#list-of-active-formatting-elements>list of active formatting elements</a>, then remove <var title="">node</var> from the <a href=#list-of-active-formatting-elements>list of active formatting elements</a>.</li>
+
      <li><p><!-- "removal" step: -->If <var title="">node</var> is not in the <a href=#list-of-active-formatting-elements>list of active
      formatting elements</a>, then remove <var title="">node</var> from the <a href=#stack-of-open-elements>stack of open
      elements</a> and then go back to the step labeled <i>inner loop</i>.</li>
 
-     <li><p>Otherwise, if <var title="">node</var> is <var title="">formatting element</var>, then
-     go to the next step in the overall algorithm.</li>
-
      <li><p><a href=#create-an-element-for-the-token>Create an element for the token</a> for which the element <var title="">node</var> was created, with <var title="">common ancestor</var> as the intended
      parent; replace the entry for <var title="">node</var> in the <a href=#list-of-active-formatting-elements>list of active formatting
      elements</a> with an entry for the new element, replace the entry for <var title="">node</var> in the <a href=#stack-of-open-elements>stack of open elements</a> with an entry for the new

Modified: source
===================================================================
--- source	2013-07-31 19:54:23 UTC (rev 8107)
+++ source	2013-07-31 20:16:42 UTC (rev 8108)
@@ -102244,11 +102244,8 @@
 
      <li><p>Let <var title="">inner loop counter</var> be zero.</p></li>
 
-     <li><p><i>Inner loop</i>: If <var title="">inner loop counter</var> is greater than or equal to
-     three, then go to the next step in the overall algorithm.</p></li>
+     <li><p><i>Inner loop</i>: Increment <var title="">inner loop counter</var> by one.</p></li>
 
-     <li><p>Increment <var title="">inner loop counter</var> by one.</p></li>
-
      <li><p>Let <var title="">node</var> be the element immediately above <var title="">node</var>
      in the <span>stack of open elements</span>, or if <var title="">node</var> is no longer in the
      <span>stack of open elements</span> (e.g. because it got removed by this algorithm<!-- in
@@ -102256,13 +102253,17 @@
      title="">node</var> in the <span>stack of open elements</span> before <var title="">node</var>
      was removed.</p></li>
 
+     <li><p>If <var title="">node</var> is <var title="">formatting element</var>, then go to the
+     next step in the overall algorithm.</p></li>
+
+     <li><p>If <var title="">inner loop counter</var> is greater than three and <var
+     title="">node</var> is in the <span>list of active formatting elements</span>, then remove <var
+     title="">node</var> from the <span>list of active formatting elements</span>.</p></li>
+
      <li><p><!-- "removal" step: -->If <var title="">node</var> is not in the <span>list of active
      formatting elements</span>, then remove <var title="">node</var> from the <span>stack of open
      elements</span> and then go back to the step labeled <i>inner loop</i>.</p></li>
 
-     <li><p>Otherwise, if <var title="">node</var> is <var title="">formatting element</var>, then
-     go to the next step in the overall algorithm.</p></li>
-
      <li><p><span>Create an element for the token</span> for which the element <var
      title="">node</var> was created, with <var title="">common ancestor</var> as the intended
      parent; replace the entry for <var title="">node</var> in the <span>list of active formatting




More information about the Commit-Watchers mailing list