Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(17)

Unified Diff: LayoutTests/css3/escape-dom-api.html

Issue 1330243002: CSS escape doesn't escape leading hyphen. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Updated patch to handle '--' Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | LayoutTests/css3/escape-dom-api-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/css3/escape-dom-api.html
diff --git a/LayoutTests/css3/escape-dom-api.html b/LayoutTests/css3/escape-dom-api.html
index b4cbfe6cd8517d718260bc47cf3c457976a70437..e2e358397f1abf9293d578a93a6f5b0d35831dfd 100644
--- a/LayoutTests/css3/escape-dom-api.html
+++ b/LayoutTests/css3/escape-dom-api.html
@@ -54,6 +54,8 @@
shouldBeEqualToString("CSS.escape('-8a')", "-\\38 a");
shouldBeEqualToString("CSS.escape('-9a')", "-\\39 a");
+ shouldBeEqualToString("CSS.escape('-')", "\\-");
mathias 2015/09/10 10:13:04 To match the updated source tests, add this:
+ shouldBeEqualToString("CSS.escape('--')", "--");
mathias 2015/09/10 10:20:25 This test expectation is incorrect. It should be `
shouldBeEqualToString("CSS.escape('--a')", "--a");
shouldBeEqualToString("CSS.escape('\x80\x2D\x5F\xA9')", "\x80\x2D\x5F\xA9");
« no previous file with comments | « no previous file | LayoutTests/css3/escape-dom-api-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698