CSS.escape('\0') should not throw.

third_party/WebKit/LayoutTests/css3/escape-dom-api.html

Index: third_party/WebKit/LayoutTests/css3/escape-dom-api.html
diff --git a/third_party/WebKit/LayoutTests/css3/escape-dom-api.html b/third_party/WebKit/LayoutTests/css3/escape-dom-api.html
index b1d31fb786ec3da18798e21b78837dd12e2a373c..948bd2ea154f637d96495c7c7fbf3d9e05d2d9c1 100644
--- a/third_party/WebKit/LayoutTests/css3/escape-dom-api.html
+++ b/third_party/WebKit/LayoutTests/css3/escape-dom-api.html
@@ -8,10 +8,10 @@
     // Test from https://github.com/mathiasbynens/CSS.escape
     description("Test window.CSS.escape()");
 
-    shouldThrow("CSS.escape('\0')");
-    shouldThrow("CSS.escape('a\0')");
-    shouldThrow("CSS.escape('\0b')");
-    shouldThrow("CSS.escape('a\0b')");
+    shouldBeEqualToString("CSS.escape('\0')", "\\0 ");

[Timothy Loh, 2015/12/03 04:08:03]

Shouldn't the expectation be "\\fffd "?

[ramya.v, 2015/12/03 06:33:46]

I've passed REPLACEMENT CHARACTER (U+FFFD) to be escaped as code point, when \0
is encounterd. Test results show '\\0 '. Please let me know if I need to correct
something. 

+    shouldBeEqualToString("CSS.escape('a\0')", "a\\0 ");
+    shouldBeEqualToString("CSS.escape('\0b')", "\\0 b");
+    shouldBeEqualToString("CSS.escape('a\0b')", "a\\0 b");
 
     shouldThrow("CSS.escape()");
     shouldBeEqualToString("CSS.escape(true)", "true");