New issue
Advanced search Search tips

Issue 704534 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

HTML parser: <dialog> does not close <p>

Project Member Reported by zcorpan@gmail.com, Mar 23 2017

Issue description

Chrome Version: 59.0.3046.0 (Official Build) canary (64-bit)

Test: http://software.hixie.ch/utilities/js/live-dom-viewer/saved/4976

Result in Chrome canary:
log: <dialog><p></dialog>x
log: <p><dialog>
log: DONE

Expect:
log: DONE

The HTML parser does not close the p for </dialog> in the first test, and doesn't close the p for <dialog> in the second test.

HTML standard:
[[
A start tag whose tag name is one of: "address", "article", "aside", "blockquote", "center", "details", "dialog", "dir", "div", "dl", "fieldset", "figcaption", "figure", "footer", "header", "hgroup", "main", "menu", "nav", "ol", "p", "section", "summary", "ul"
If the stack of open elements has a p element in button scope, then close a p element.

Insert an HTML element for the token.
]]

[[
An end tag whose tag name is one of: "address", "article", "aside", "blockquote", "button", "center", "details", "dialog", "dir", "div", "dl", "fieldset", "figcaption", "figure", "footer", "header", "hgroup", "listing", "main", "menu", "nav", "ol", "pre", "section", "summary", "ul"
[...handling of end tag...]
]]

html5lib-tests PR:
https://github.com/html5lib/html5lib-tests/pull/91

Test passes in Firefox Nightly. Fails in Safari TP.
 

Comment 1 by dk...@chromium.org, Apr 5 2017

Should we move this bug to available? Is it on the right component?

Comment 2 by tkent@chromium.org, Apr 5 2017

Status: Available (was: Untriaged)
Project Member

Comment 3 by sheriffbot@chromium.org, Apr 6 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 4 by tkent@chromium.org, Apr 9 2018

Status: Available (was: Untriaged)
Can I take a look this issue if no one works on it?
Labels: -Hotlist-Recharge-Cold
> Can I take a look this issue if no one works on it?

Please take this.  But I'm afraid this issue isn't easy to fix.  I guess we need to update html5lib in WPT before fixing Chrome code.

Owner: donghee....@gmail.com
Status: Assigned (was: Available)
Status: Available (was: Assigned)
Fixing chrome code to solve this issue is easy but
since html5lib should be migrated into WPT test first.

Not easy for me right now.

Owner: ----

Sign in to add a comment