html: ignore <caption>, <col>, <tbody> etc. when parsing table fragments

Pass tests6.dat, test 36: <caption><col><colgroup><tbody><tfoot><thead><tr> | <tr> Pass tests through test 44: <body></body></html> R=nigeltao CC=golang-dev https://golang.org/cl/5494055

html: ignore <caption>, <col>, <tbody> etc. when parsing table fragments
Pass tests6.dat, test 36: <caption><col><colgroup><tbody><tfoot><thead><tr> | <tr> Pass tests through test 44: <body></body></html> R=nigeltao CC=golang-dev https://golang.org/cl/5494055
a0bd46e7 · Andrew Balholm · Nigel Tao · 315b361f · a0bd46e7 · a0bd46e7
Commit a0bd46e7 authored Dec 19, 2011 by Andrew Balholm Committed by Nigel Tao Dec 19, 2011
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 1 deletion

parse.go src/pkg/html/parse.go +7 -0

parse_test.go src/pkg/html/parse_test.go +1 -1

No files found.
--- a/src/pkg/html/parse.go
+++ b/src/pkg/html/parse.go
@@ -1206,6 +1206,13 @@ func inTableBodyIM(p *parser) bool {
 			add = true
 			data = "tr"
 			consumed = false
+		case "caption", "col", "colgroup", "tbody", "tfoot", "thead":
+			if !p.popUntil(tableScopeStopTags, "tbody", "thead", "tfoot") {
+				// Ignore the token.
+				return true
+			}
+			p.im = inTableIM
+			return false
 		default:
 			// TODO.
 		}

--- a/src/pkg/html/parse_test.go
+++ b/src/pkg/html/parse_test.go
@@ -172,7 +172,7 @@ func TestParser(t *testing.T) {
 		{"tests3.dat", -1},
 		{"tests4.dat", -1},
 		{"tests5.dat", -1},
-		{"tests6.dat", 36},
+		{"tests6.dat", 45},
 		{"tests10.dat", 13},
 	}
 	for _, tf := range testFiles {