html: auto-close <dd> and <dt> elements

Pass tests2.dat, test 8: <!DOCTYPE html><dt><div><dd> | <!DOCTYPE html> | <html> | <head> | <body> | <dt> | <div> | <dd> Also pass tests through test 9: <script></x R=nigeltao CC=golang-dev https://golang.org/cl/5373083

html: auto-close <dd> and <dt> elements
06ef97e1 · Andrew Balholm · Nigel Tao · 631a575f · 06ef97e1 · 06ef97e1
Commit 06ef97e1 authored Nov 13, 2011 by Andrew Balholm Committed by Nigel Tao Nov 13, 2011
Hide whitespace changes
Inline Side-by-side

Showing with 19 additions and 1 deletion

parse.go src/pkg/html/parse.go +18 -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
@@ -637,6 +637,24 @@ func inBodyIM(p *parser) bool {
 			}
 			p.popUntil(buttonScopeStopTags, "p")
 			p.addElement(p.tok.Data, p.tok.Attr)
+		case "dd", "dt":
+			p.framesetOK = false
+			for i := len(p.oe) - 1; i >= 0; i-- {
+				node := p.oe[i]
+				switch node.Data {
+				case "dd", "dt":
+					p.oe = p.oe[:i]
+				case "address", "div", "p":
+					continue
+				default:
+					if !isSpecialElement[node.Data] {
+						continue
+					}
+				}
+				break
+			}
+			p.popUntil(buttonScopeStopTags, "p")
+			p.addElement(p.tok.Data, p.tok.Attr)
 		case "optgroup", "option":
 			if p.top().Data == "option" {
 				p.oe.pop()

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