This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 10096 - Adoption Agency 6.4 and 6.5 should be re-ordered
Summary: Adoption Agency 6.4 and 6.5 should be re-ordered
Status: RESOLVED FIXED
Alias: None
Product: HTML WG
Classification: Unclassified
Component: pre-LC1 HTML5 spec (editor: Ian Hickson) (show other bugs)
Version: unspecified
Hardware: Other other
: P1 critical
Target Milestone: ---
Assignee: Ian 'Hixie' Hickson
QA Contact: HTML WG Bugzilla archive list
URL: http://www.whatwg.org/specs/web-apps/...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-06 08:10 UTC by contributor
Modified: 2010-10-04 14:48 UTC (History)
6 users (show)

See Also:


Attachments

Description contributor 2010-07-06 08:10:35 UTC
Section: http://www.whatwg.org/specs/web-apps/current-work/#parsing-main-inbody

Comment:
Adoption Agency 6.4 and 6.5 should be re-ordered

Posted from: 98.210.108.185
Comment 1 Eric Seidel 2010-07-06 08:12:35 UTC
In implementing "active formatting element list" bookmarking for the adoption agency algorithm.  I found that steps 6.4 and 6.5 seem to be out of order:


6.4 Otherwise, if last node is the furthest block, then move the aforementioned bookmark to be immediately after the node in the list of active formatting elements.

6.5 Create an element for the token for which the element node was created, replace the entry for node in the list of active formatting elements with an entry for the new element, replace the entry for node in the stack of open elements with an entry for the new element, and let node be the new element.

Notice how 6.4 makes a bookmark relative to "node" and then 6.5 changes what "node" is.  The naive implementation would cause the bookmark's anchor node to fail to be found.  If you swap 6.4 and 6.5 the algorithm still does the same thing, but the anchor node is already properly updated to the new element.
Comment 2 Ian 'Hixie' Hickson 2010-07-14 19:17:59 UTC
EDITOR'S RESPONSE: This is an Editor's Response to your comment. If you are satisfied with this response, please change the state of this bug to CLOSED. If you have additional information and would like the editor to reconsider, please reopen this bug. If you would like to escalate the issue to the full HTML Working Group, please add the TrackerRequest keyword to this bug, and suggest title and text for the tracker issue; or you may create a tracker issue yourself, if you are able to do so. For more details, see this document:
   http://dev.w3.org/html5/decision-policy/decision-policy.html

Status: Accepted
Change Description: see diff given below
Rationale: Concurred with reporter's comments.
Comment 3 contributor 2010-07-14 19:19:37 UTC
Checked in as WHATWG revision r5163.
Check-in comment: Tweak the steps to reduce the likelihood of bugs implementing the AAA.
http://html5.org/tools/web-apps-tracker?from=5162&to=5163