html: auto-close <p> elements when starting <form> element.

Pass tests2.dat, test 26: <!doctypehtml><p><form> | <!DOCTYPE html> | <html> | <head> | <body> | <p> | <form> Also pass tests through test 32: <!DOCTYPE html><!-- X R=nigeltao CC=golang-dev https://golang.org/cl/5369114

html: auto-close <p> elements when starting <form> element.
b91d8225 · Andrew Balholm · Nigel Tao · a619da9f · b91d8225 · b91d8225
Commit b91d8225 authored Nov 15, 2011 by Andrew Balholm Committed by Nigel Tao Nov 15, 2011
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 1 deletion

parse.go src/pkg/html/parse.go +6 -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
@@ -619,6 +619,12 @@ func inBodyIM(p *parser) bool {
 			// TODO: detect <select> inside a table.
 			p.im = inSelectIM
 			return true
+		case "form":
+			if p.form == nil {
+				p.popUntil(buttonScopeStopTags, "p")
+				p.addElement(p.tok.Data, p.tok.Attr)
+				p.form = p.top()
+			}
 		case "li":
 			p.framesetOK = false
 			for i := len(p.oe) - 1; i >= 0; i-- {

--- a/src/pkg/html/parse_test.go
+++ b/src/pkg/html/parse_test.go
@@ -134,7 +134,7 @@ func TestParser(t *testing.T) {
 	}{
 		// TODO(nigeltao): Process all the test cases from all the .dat files.
 		{"tests1.dat", -1},
-		{"tests2.dat", 26},
+		{"tests2.dat", 33},
 		{"tests3.dat", 0},
 	}
 	for _, tf := range testFiles {