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

Side by Side Diff: qunit/tests/cssEscaping.js

Issue 29581937: Noissue - Use anonymous blocks instead of anonymous functions (Closed) Base URL: https://hg.adblockplus.org/adblockpluschrome/
Patch Set: Created Oct. 18, 2017, 2:10 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « ext/devtools.js ('k') | qunit/tests/filterValidation.js » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 "use strict"; 1 "use strict";
2 2
3 (function()
4 { 3 {
5 const {Filter, ElemHideFilter} = require("filterClasses"); 4 const {Filter, ElemHideFilter} = require("filterClasses");
6 const {escapeCSS, quoteCSS} = require("filterComposer"); 5 const {escapeCSS, quoteCSS} = require("filterComposer");
7 6
8 module("CSS escaping"); 7 module("CSS escaping");
9 8
10 test("CSS escaping", () => 9 test("CSS escaping", () =>
11 { 10 {
12 function testSelector(opts) 11 function testSelector(opts)
13 { 12 {
(...skipping 78 matching lines...) Expand 10 before | Expand all | Expand 10 after
92 testEscape("x" + chr); 91 testEscape("x" + chr);
93 92
94 // Leading dashes must be escaped, when followed by certain characters. 93 // Leading dashes must be escaped, when followed by certain characters.
95 testEscape("-" + chr); 94 testEscape("-" + chr);
96 } 95 }
97 96
98 // Test some non-ASCII characters. However, those shouldn't 97 // Test some non-ASCII characters. However, those shouldn't
99 // require escaping. 98 // require escaping.
100 testEscape("\uD83D\uDE3B\u2665\u00E4"); 99 testEscape("\uD83D\uDE3B\u2665\u00E4");
101 }); 100 });
102 }()); 101 }
OLDNEW
« no previous file with comments | « ext/devtools.js ('k') | qunit/tests/filterValidation.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld