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

Unified Diff: background.js

Issue 29339314: Issue 3870 - Rewrite legacy options page to use async messages (Closed)
Patch Set: Fix Downloading... indicators Created April 3, 2016, 4:10 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 | dependencies » ('j') | options.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: background.js
diff --git a/background.js b/background.js
index fbfb7166751946e15b0dee1268fedc9fbf3551da..4d37cdceb5fa5dd62e9ed8e1edfd02daa21cb7ff 100644
--- a/background.js
+++ b/background.js
@@ -28,33 +28,6 @@ var extractHostFromFrame = require("url").extractHostFromFrame;
var port = require("messaging").port;
var devtools = require("devtools");
-// This is a hack to speedup loading of the options page on Safari.
-// Once we replaced the background page proxy with message passing
-// this global function should removed.
-function getUserFilters()
-{
- var filters = [];
- var exceptions = [];
-
- for (var i = 0; i < FilterStorage.subscriptions.length; i++)
Sebastian Noack 2016/04/03 18:19:59 Please remove the imports for names no longer used
kzar 2016/04/05 11:10:15 Done.
- {
- var subscription = FilterStorage.subscriptions[i];
- if (!(subscription instanceof SpecialSubscription))
- continue;
-
- for (var j = 0; j < subscription.filters.length; j++)
- {
- var filter = subscription.filters[j];
- if (filter instanceof WhitelistFilter && /^@@\|\|([^\/:]+)\^\$document$/.test(filter.text))
- exceptions.push(RegExp.$1);
- else
- filters.push(filter.text);
- }
- }
-
- return {filters: filters, exceptions: exceptions};
-}
-
port.on("get-selectors", function(msg, sender)
{
var selectors;
« no previous file with comments | « no previous file | dependencies » ('j') | options.js » ('J')

Powered by Google App Engine
This is Rietveld