Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: qunit/tests/cssEscaping.js

Issue 29374674: Issue 4864 - Start using ESLint for adblockpluschrome (Closed)
Left Patch Set: Add arrow-parens rule to match existing convention Created March 7, 2017, 10:48 a.m.
Right Patch Set: Use .includes again Created March 31, 2017, 8:37 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « qunit/common.js ('k') | qunit/tests/filterValidation.js » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 "use strict"; 1 "use strict";
2 2
3 (function()
3 { 4 {
4 const {Filter, ElemHideFilter} = require("filterClasses"); 5 const {Filter, ElemHideFilter} = require("filterClasses");
5 const {escapeCSS, quoteCSS} = require("filterComposer"); 6 const {escapeCSS, quoteCSS} = require("filterComposer");
6 7
7 module("CSS escaping"); 8 module("CSS escaping");
8 9
9 test("CSS escaping", () => 10 test("CSS escaping", () =>
10 { 11 {
11 function testSelector(opts) 12 function testSelector(opts)
12 { 13 {
(...skipping 78 matching lines...) Expand 10 before | Expand all | Expand 10 after
91 testEscape("x" + chr); 92 testEscape("x" + chr);
92 93
93 // Leading dashes must be escaped, when followed by certain characters. 94 // Leading dashes must be escaped, when followed by certain characters.
94 testEscape("-" + chr); 95 testEscape("-" + chr);
95 } 96 }
96 97
97 // Test some non-ASCII characters. However, those shouldn't 98 // Test some non-ASCII characters. However, those shouldn't
98 // require escaping. 99 // require escaping.
99 testEscape("\uD83D\uDE3B\u2665\u00E4"); 100 testEscape("\uD83D\uDE3B\u2665\u00E4");
100 }); 101 });
101 } 102 }());
LEFTRIGHT

Powered by Google App Engine
This is Rietveld