From b1d6fa517cd99cbbad1523dddc6182da5a701122 Mon Sep 17 00:00:00 2001 From: Mike Samuel Date: Wed, 11 Jan 2012 18:47:03 -0500 Subject: [PATCH] html/template: reenable testcases and fix mis-escaped sequences. Tighter octal parsing broke some tests and were disabled in https://golang.org/cl/5530051 Those tests were broken. The CSS decoder was supposed to see CSS hex escape sequences of the form '\' +, but those escape sequences were instead being consumed by the template parser. This change properly escapes those escape sequences, and uses proper escaping for NULs. R=golang-dev, rsc, nigeltao CC=golang-dev https://golang.org/cl/5529073 --- src/pkg/html/template/escape_test.go | 22 ++++++++++------------ 1 file changed, 10 insertions(+), 12 deletions(-) diff --git a/src/pkg/html/template/escape_test.go b/src/pkg/html/template/escape_test.go index 0cac6a43ae..2ce1fb566a 100644 --- a/src/pkg/html/template/escape_test.go +++ b/src/pkg/html/template/escape_test.go @@ -300,23 +300,21 @@ func TestEscape(t *testing.T) { `

`, `

`, }, - // This test is broken by the fix to issue 2658. - // { - // "styleObfuscatedExpressionBlocked", - // `

`, - // `

`, - // }, + { + "styleObfuscatedExpressionBlocked", + `

`, + `

`, + }, { "styleMozBindingBlocked", `

`, `

`, }, - // This test is broken by the fix to issue 2658. - // { - // "styleObfuscatedMozBindingBlocked", - // `

`, - // `

`, - // }, + { + "styleObfuscatedMozBindingBlocked", + `

`, + `

`, + }, { "styleFontNameString", `

`, -- 2.50.0