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

Unified Diff: background.js

Issue 29338009: Issue 3763 - Got rid of redundant "add-filters" message type (Closed)
Patch Set: Update adblockplusui dependency Created March 9, 2016, 3:38 p.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 | block.js » ('j') | dependencies » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: background.js
===================================================================
--- a/background.js
+++ b/background.js
@@ -377,20 +377,6 @@
return;
}
break;
- case "add-filters":
- var result = parseFilters(msg.text);
-
- if (result.errors.length > 0)
- {
- sendResponse({status: "invalid", error: result.errors.join("\n")});
- break;
- }
-
- for (var i = 0; i < result.filters.length; i++)
- FilterStorage.addFilter(result.filters[i]);
-
- sendResponse({status: "ok"});
- break;
case "add-sitekey":
processKey(msg.token, sender.page, sender.frame);
break;
« no previous file with comments | « no previous file | block.js » ('j') | dependencies » ('J')

Powered by Google App Engine
This is Rietveld