html: treat <image> as <img>

Pass tests1.dat, test 90: <p><image></p> | <html> | <head> | <body> | <p> | <img> Also pass test 91: <a><table><a></table><p><a><div><a> R=nigeltao CC=golang-dev https://golang.org/cl/5339052

html: treat <image> as <img>
ce4eec2e · Andrew Balholm · Nigel Tao · 3e94e757 · ce4eec2e · ce4eec2e
Commit ce4eec2e authored Nov 09, 2011 by Andrew Balholm Committed by Nigel Tao Nov 09, 2011
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 2 deletions

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

src/pkg/html/parse_test.go src/pkg/html/parse_test.go +3 -2

No files found.
--- a/src/pkg/html/parse.go
+++ b/src/pkg/html/parse.go
@@ -651,6 +651,9 @@ func inBodyIM(p *parser) (insertionMode, bool) {
 			}
 		case "base", "basefont", "bgsound", "command", "link", "meta", "noframes", "script", "style", "title":
 			return useTheRulesFor(p, inBodyIM, inHeadIM)
+		case "image":
+			p.tok.Data = "img"
+			return inBodyIM, false
 		default:
 			// TODO.
 			p.addElement(p.tok.Data, p.tok.Attr)

--- a/src/pkg/html/parse_test.go
+++ b/src/pkg/html/parse_test.go
@@ -133,7 +133,7 @@ func TestParser(t *testing.T) {
 		n int
 	}{
 		// TODO(nigeltao): Process all the test cases from all the .dat files.
-		{"tests1.dat", 89},
+		{"tests1.dat", 92},
 		{"tests2.dat", 0},
 		{"tests3.dat", 0},
 	}
@@ -210,6 +210,7 @@ var renderTestBlacklist = map[string]bool{
 	// The second <a> will be reparented to the first <table>'s parent. This
 	// results in an <a> whose parent is an <a>, which is not 'well-formed'.
 	`<a><table><td><a><table></table><a></tr><a></table><b>X</b>C<a>Y`: true,
-	// The second <a> will be reparented, similar to the case above.
+	// More cases of <a> being reparented:
 	`<a href="blah">aba<table><a href="foo">br<tr><td></td></tr>x</table>aoe`: true,
+	`<a><table><a></table><p><a><div><a>`:                                     true,
 }