html: don't emit text token for empty raw text elements.

Pass tests1.dat, test 99: <script></script></div><title></title><p><p> | <html> | <head> | <script> | <title> | <body> | <p> | <p> Also pass tests through test 105: <ul><li><ul></li><li>a</li></ul></li></ul> R=nigeltao CC=golang-dev https://golang.org/cl/5373043

html: don't emit text token for empty raw text elements.
ddc5ec64 · Andrew Balholm · Nigel Tao · 23ffbe61 · ddc5ec64 · ddc5ec64
Commit ddc5ec64 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 3 deletions

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

token.go src/pkg/html/token.go +4 -2

No files found.
--- 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", 99},
+		{"tests1.dat", 106},
 		{"tests2.dat", 0},
 		{"tests3.dat", 0},
 	}
@@ -213,4 +213,5 @@ var renderTestBlacklist = map[string]bool{
 	// 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,
+	`<a><table><td><a><table></table><a></tr><a></table><a>`:                  true,
 }
--- a/src/pkg/html/token.go
+++ b/src/pkg/html/token.go
@@ -552,8 +552,10 @@ func (z *Tokenizer) Next() TokenType {
 	z.data.end = z.raw.end
 	if z.rawTag != "" {
 		z.readRawOrRCDATA()
-		z.tt = TextToken
+		if z.data.end > z.data.start {
-		return z.tt
+			z.tt = TextToken
+			return z.tt
+		}
 	}
 	z.textIsRaw = false