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 9581 - Steps for handling end tags in the "in foreign" mode may compare the name of an already-popped node
Summary: Steps for handling end tags in the "in foreign" mode may compare the name of ...
Status: RESOLVED FIXED
Alias: None
Product: HTML WG
Classification: Unclassified
Component: pre-LC1 HTML5 spec (editor: Ian Hickson) (show other bugs)
Version: unspecified
Hardware: PC Linux
: P1 critical
Target Milestone: ---
Assignee: Ian 'Hixie' Hickson
QA Contact: HTML WG Bugzilla archive list
URL: http://html5.org/tools/web-apps-track...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-04-22 14:33 UTC by Henri Sivonen
Modified: 2010-10-04 14:56 UTC (History)
4 users (show)

See Also:


Attachments

Description Henri Sivonen 2010-04-22 14:33:22 UTC
In the text added in rev 4803, the condition
"If node has the same tag name as the token"
is checked even if node has already been popped by the handling in the secondary insertion mode in the previous iteration.
Comment 1 Henri Sivonen 2010-05-19 07:14:45 UTC
Adjusting importance per discussion with Hixie.
Comment 2 Henri Sivonen 2010-05-31 11:01:04 UTC
http://hg.mozilla.org/mozilla-central/file/62c09fac599a/parser/html/javasrc/TreeBuilder.java#l3109 shows how I improvised around this bug.
Comment 3 Henri Sivonen 2010-06-30 08:56:53 UTC
the left-hand part of the |if| condition on line http://hg.mozilla.org/projects/htmlparser/rev/b08b9993cd5d#l1.57 fixes this bug
Comment 4 Henri Sivonen 2010-06-30 12:10:15 UTC
From #whatwg
14:32 < jgraham> hsivonen: It seems to work OK with the html5lib test cases if 
                 you break when node is no longer in the stack of open elements 
                 after step 5
Comment 5 Ian 'Hixie' Hickson 2010-07-14 00:38:18 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: Partially Accepted
Change Description: see diff given below
Rationale: This is fixed by the fix to bug 9580.