parsing updated related to EOFs in tables. exciting

ryanking · ryanking · commit 5a07dfcd0981 · 2008-10-16T00:59:37.000Z
--HG--
extra : convert_revision : svn%3Aacbfec75-9323-0410-a652-858a13e371e0/trunk%401224
diff --git a/lib/html5/constants.rb b/lib/html5/constants.rb
@@ -1052,7 +1052,8 @@ def self._(str); str end
         _("Unexpected end table tag in caption. Generates implied end caption."),
       "end-html-in-innerhtml" => _("Unexpected html end tag in inner html mode."),
       "expected-self-closing-tag" => _("Expected a > after the /."),
-      "self-closing-end-tag" => _("Self closing end tag.")
+      "self-closing-end-tag" => _("Self closing end tag."),
+      "eof-in-table" => _("Unexpected end of file. Expected table content.")
   }
 
 end
diff --git a/lib/html5/html5parser/in_row_phase.rb b/lib/html5/html5parser/in_row_phase.rb
@@ -74,6 +74,10 @@ def endTagOther(name)
       @parser.phases[:inTable].processEndTag(name)
     end
 
+    def process_eof
+      @parser.phases[:inTable].process_eof
+    end
+
     protected
 
     # XXX unify this with other table helper methods
diff --git a/lib/html5/html5parser/in_table_body_phase.rb b/lib/html5/html5parser/in_table_body_phase.rb
@@ -17,6 +17,10 @@ def processSpaceCharacters(data)
       @parser.phases[:inTable].processSpaceCharacters(data)
     end
 
+    def process_eof
+      @parser.phases[:inTable].process_eof
+    end
+
     def startTagTr(name, attributes)
       clearStackToTableBodyContext
       @tree.insert_element(name, attributes)
diff --git a/lib/html5/html5parser/in_table_phase.rb b/lib/html5/html5parser/in_table_phase.rb
@@ -38,6 +38,14 @@ def processCharacters(data)
       end
     end
 
+    def process_eof
+      if @tree.open_elements.last.name != "html"
+        @parser.parse_error("eof-in-table")
+      else
+        assert @parser.innerHTML
+      end
+    end
+
     def startTagCaption(name, attributes)
       clear_stack_to_table_context
       @tree.activeFormattingElements.push(Marker)

Original file line number	Diff line number	Diff line change
`@@ -1052,7 +1052,8 @@ def self._(str); str end`
`1052`	`1052`	`_("Unexpected end table tag in caption. Generates implied end caption."),`
`1053`	`1053`	`"end-html-in-innerhtml" => _("Unexpected html end tag in inner html mode."),`
`1054`	`1054`	`"expected-self-closing-tag" => _("Expected a > after the /."),`
`1055`		`- "self-closing-end-tag" => _("Self closing end tag.")`
	`1055`	`+ "self-closing-end-tag" => _("Self closing end tag."),`
	`1056`	`+ "eof-in-table" => _("Unexpected end of file. Expected table content.")`
`1056`	`1057`	`}`
`1057`	`1058`
`1058`	`1059`	`end`