Merge pull request #2 from HTMLParseErrorWG/comments-parse-errors

Comments parse errors

Author: inikulin

tokenizer/domjs.test

@@ -154,7 +154,10 @@
             "description":"--!NUL in comment ",
             "doubleEscaped":true,
             "input":"<!----!\\u0000-->",
-            "output":["ParseError", "ParseError", ["Comment", "--!\\uFFFD"]]
+            "output":["ParseError", "ParseError", ["Comment", "--!\\uFFFD"]],
+            "errors":[
+                { "code": "unexpected-null-character", "line": 1, "col": 8 }
+            ]
         },
         {
             "description":"space EOF after doctype ",

tokenizer/escapeFlag.test

@@ -16,7 +16,10 @@
 "initialStates":["RCDATA state", "RAWTEXT state"],
 "lastStartTag":"xmp",
 "input":"foo<!--></xmp><!-->baz</xmp>",
-"output":[["Character", "foo<!-->"], ["EndTag", "xmp"], "ParseError", ["Comment", ""], ["Character", "baz"], ["EndTag", "xmp"]]},
+"output":[["Character", "foo<!-->"], ["EndTag", "xmp"], "ParseError", ["Comment", ""], ["Character", "baz"], ["EndTag", "xmp"]],
+"errors":[
+    { "code": "abrupt-comment", "line": 1, "col": 19 }
+]},
 
 {"description":"Commented entities in RCDATA",
 "initialStates":["RCDATA state"],

tokenizer/pendingSpecChanges.test

@@ -2,6 +2,8 @@
 
 {"description":"<!---- >",
 "input":"<!---- >",
-"output":["ParseError", "ParseError", ["Comment","-- >"]]}
-
+"output":["ParseError", "ParseError", ["Comment","-- >"]],
+"errors":[
+    { "code": "eof-in-comment", "line": 1, "col": 9 }
+]}
 ]}

tokenizer/test1.test

@@ -94,7 +94,10 @@
 
 {"description":"Unfinished comment",
 "input":"<!--comment",
-"output":["ParseError", ["Comment", "comment"]]},
+"output":["ParseError", ["Comment", "comment"]],
+"errors":[
+    { "code": "eof-in-comment", "line": 1, "col": 12 }
+]},
 
 {"description":"Start of a comment",
 "input":"<!-",
@@ -104,17 +107,30 @@
 ]},
 
 {"description":"Short comment",
- "input":"<!-->",
- "output":["ParseError", ["Comment", ""]]},
+"input":"<!-->",
+"output":["ParseError", ["Comment", ""]],
+"errors":[
+    { "code": "abrupt-comment", "line": 1, "col": 5 }
+]},
+
 
 {"description":"Short comment two",
- "input":"<!--->",
- "output":["ParseError", ["Comment", ""]]},
+"input":"<!--->",
+"output":["ParseError", ["Comment", ""]],
+"errors":[
+    { "code": "abrupt-comment", "line": 1, "col": 6 }
+]},
 
 {"description":"Short comment three",
  "input":"<!---->",
  "output":[["Comment", ""]]},
 
+{"description":"Nested comment",
+"input":"<!-- <!--test-->",
+"output":[["Comment", " <!--test"]],
+"errors":[
+    { "code": "nested-comment", "line": 1, "col": 10 }
+]},
 
 {"description":"Ampersand EOF",
 "input":"&",

tokenizer/test2.test

@@ -153,7 +153,10 @@
 
 {"description":"Comment with dash",
 "input":"<!---x",
-"output":["ParseError", ["Comment", "-x"]]},
+"output":["ParseError", ["Comment", "-x"]],
+"errors":[
+    { "code": "eof-in-comment", "line": 1, "col": 7 }
+]},
 
 {"description":"Entity + newline",
 "input":"\nx\n&gt;\n",