The default bug view has changed. See this FAQ.

[HTML5] Implement Comment end bang state

RESOLVED FIXED

Status

()

Core
HTML: Parser
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: hsivonen, Assigned: hsivonen)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
http://www.whatwg.org/specs/web-apps/current-work/#comment-end-bang-state

This state missed the landing. It's important to get it in ASAP to enable testing.
(Assignee)

Comment 1

8 years ago
Created attachment 386724 [details] [diff] [review]
Implement comment end bang and comment end space states

Note that there are also changes related to error reporting with < in various places in attributes. These changes should all be no-ops in C++.

There are also various whitespace changes in the .java file.
Attachment #386724 - Flags: superreview?(mrbkap)
Attachment #386724 - Flags: review?(jonas)
Comment on attachment 386724 [details] [diff] [review]
Implement comment end bang and comment end space states

Given that neither me or blake really reviewed the generated code yet, lets just check this in with rs=sicking (rs meaning rubber stamp). We'll need to go through it in detail later before turning the parser on by default.
Attachment #386724 - Flags: superreview?(mrbkap)
Attachment #386724 - Flags: review?(jonas)
(Assignee)

Comment 3

8 years ago
http://hg.mozilla.org/mozilla-central/rev/f4f94ee39e4b
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.