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

Unified Diff: lib/snippets.js

Issue 29761597: Issue 6538, 6781 - Implement script parsing for snippets (Closed) Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set: Revert Patch Set 8 Created July 12, 2018, 10:56 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | test/snippets.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/snippets.js
===================================================================
--- a/lib/snippets.js
+++ b/lib/snippets.js
@@ -18,16 +18,20 @@
"use strict";
/**
* @fileOverview Snippets implementation.
*/
const {Filter} = require("./filterClasses");
+const singleCharacterEscapes = new Map([
+ ["n", "\n"], ["r", "\r"], ["t", "\t"]
+]);
+
let filters = new Set();
/**
* Container for snippet filters
* @class
*/
let Snippets = {
/**
@@ -70,8 +74,82 @@
if (filter.isActiveOnDomain(domain))
result.push(filter.script);
}
return result;
}
};
exports.Snippets = Snippets;
+
+/**
+ * Parses a script and returns a list of all its commands and their arguments
+ * @param {string} script
+ * @return {Array.<string[]>}
+ */
+function parseScript(script)
+{
+ let tree = [];
+
+ let escape = false;
+ let withinQuotes = false;
+
+ let unicodeEscape = null;
+
+ let call = [];
+ let argument = "";
+
+ for (let character of script.trim() + ";")
+ {
+ if (unicodeEscape != null)
+ {
+ unicodeEscape += character;
+
+ if (unicodeEscape.length == 4)
+ {
+ let codePoint = parseInt(unicodeEscape, 16);
+ if (!isNaN(codePoint))
+ argument += String.fromCodePoint(codePoint);
+
+ unicodeEscape = null;
+ }
+ }
+ else if (escape)
+ {
+ escape = false;
+
+ if (character == "u")
+ unicodeEscape = "";
+ else
+ argument += singleCharacterEscapes.get(character) || character;
+ }
+ else if (character == "\\")
+ {
+ escape = true;
+ }
+ else if (character == "'")
+ {
+ withinQuotes = !withinQuotes;
+ }
+ else if (withinQuotes || character != ";" && !/\s/u.test(character))
+ {
+ argument += character;
+ }
+ else
+ {
+ if (argument)
+ {
+ call.push(argument);
+ argument = "";
+ }
+
+ if (character == ";" && call.length > 0)
+ {
+ tree.push(call);
+ call = [];
+ }
+ }
+ }
+
+ return tree;
+}
+
+exports.parseScript = parseScript;
« no previous file with comments | « no previous file | test/snippets.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld