901319 - Implement HTML spec change to 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

Status: RESOLVED FIXED

(Core :: DOM: HTML Parser, defect)

Description

[Michael[tm] Smith [:sideshowbarker]]

http://html5.org/r/8108
https://www.w3.org/Bugs/Public/show_bug.cgi?id=20319#c3

Comment 1

[Michael[tm] Smith [:sideshowbarker]]

http://lists.w3.org/Archives/Public/public-html/2013Aug/0002.html from Travis Leithead seems to indicate that at this point Microsoft is inclined to implement the spec change.

Comment 2

[Hixie (not reading bugmail)]

See also bug 884795 which resulted in another change to the AAA:
   http://html5.org/tools/web-apps-tracker?from=8138&to=8139
   https://www.w3.org/Bugs/Public/show_bug.cgi?id=22926

It makes conforming pages parse correctly (before, if you nested multiple identical formatting elements more than 3 deep, you'd end up corrupting your tree; that was only supposed to happen when the document was non-conforming).

Comment 3

[William Chen [:wchen]]

Attachment: Implement HTML spec change to 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

http://html5.org/r/8108

Comment 4

[Henri Sivonen (:hsivonen)]

Comment on attachment 8382844 [details] [diff] [review]
Implement HTML spec change to 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

r+, but instead of 

for (int j = 1; true; ++j) {

I'd prefer

int j = 0;
for (;;) {
  ++j;
  ...
}

to make the structure of the code match the structure of the spec more closely.

Comment 5

[William Chen [:wchen]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/c04e2e2b2944

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c04e2e2b2944