html: parse <optgroup> tags

Pass tests2.dat, test 34: <!DOCTYPE html><select><option><optgroup> | <!DOCTYPE html> | <html> | <head> | <body> | <select> | <option> | <optgroup> R=nigeltao CC=golang-dev https://golang.org/cl/5393045

html: parse <optgroup> tags
33075970 · Andrew Balholm · Nigel Tao · 28546ed5 · 33075970 · 33075970
Commit 33075970 authored Nov 16, 2011 by Andrew Balholm Committed by Nigel Tao Nov 16, 2011
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 2 deletions

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

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
@@ -1226,7 +1226,13 @@ func inSelectIM(p *parser) bool {
 			}
 			p.addElement(p.tok.Data, p.tok.Attr)
 		case "optgroup":
-			// TODO.
+			if p.top().Data == "option" {
+				p.oe.pop()
+			}
+			if p.top().Data == "optgroup" {
+				p.oe.pop()
+			}
+			p.addElement(p.tok.Data, p.tok.Attr)
 		case "select":
 			endSelect = true
 		case "input", "keygen", "textarea":

--- 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", 34},
+		{"tests2.dat", 35},
 		{"tests3.dat", 0},
 	}
 	for _, tf := range testFiles {