Bug 119478 - 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
Summary: Implement HTML spec change to Adoption Agency Algorithm to not reverse the or...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2013-08-04 09:17 PDT by Michael[tm] Smith
Modified: 2016-09-08 13:59 PDT (History)
10 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Comment 1 Ryosuke Niwa 2013-08-04 18:53:36 PDT
I don't understand why this specification change was made when all browsers behave according to the old specification.
Comment 2 Michael[tm] Smith 2013-08-04 21:51:16 PDT
(In reply to comment #1)
> I don't understand why this specification change was made when all browsers behave according to the old specification.

Because the existing behavior is buggy in that it causes nodes to end up in the DOM in (from users perspective) an unexpected order; e.g., the text content of the following div ending up as "456123" instead of "123456".

  <div id=mike><b><i><s><u><tt><p></b>123</p></tt></u></s>456

Yeah, it's very invalid markup but dealing gracefully with invalid markup is entirely what this whole part of the parsing algorithm is all about. So this is a bug in the behavior of what the AAA is intended for, and if it's possible to have the spec get rid of the existing bug in the algorithm and handle that case more intuitively, then I think the right thing for Hixie or any other editor to do is to fix it in the spec if he can and then to try to see if browsers are willing to implement the revised algorithm or not.

Of course it's collectively up to browser implementors to decide if they want to make the change. But for a case like this, having concrete spec text to review in context seems to me a least to be a lot better than just having an isolated proposal written in an e-mail message or whatever.
Comment 3 Ian 'Hixie' Hickson 2013-08-14 13:31:31 PDT
While you're at it, this change to the AAA was also made recently:
   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 4 Simon Pieters 2016-05-17 23:41:46 PDT
Ping (see https://github.com/html5lib/html5lib-tests/issues/78 )
Comment 5 Alexey Proskuryakov 2016-09-08 13:59:39 PDT
<rdar://problem/28213632>