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

Unified Diff: background.js

Issue 5118635873927168: Added missing import for filterClasses.ActiveFilter to the background page (Closed)
Patch Set: Added comment to remove that line again Created April 9, 2014, 10:23 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: background.js
===================================================================
--- a/background.js
+++ b/background.js
@@ -21,6 +21,13 @@
this.RegExpFilter = RegExpFilter;
this.BlockingFilter = BlockingFilter;
this.WhitelistFilter = WhitelistFilter;
+
+ // TODO: The import below isn't necessary anymore with
+ // https://hg.adblockplus.org/adblockplus/rev/7eb29729a72c
+ // However due to https://issues.adblockplus.org/ticket/189
+ // we can't update the subrepo yet. So please remove this
+ // line when the issue was fixed and the subrepo was updated.
+ this.ActiveFilter = ActiveFilter;
}
with(require("subscriptionClasses"))
{
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld