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

Unified Diff: background.js

Issue 29333819: Issue 2375 - Implement "Blocking lists" section in new options page (Closed)
Patch Set: Created Jan. 18, 2016, 9:50 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 | locale/en-US/options.json » ('j') | locale/en-US/options.json » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: background.js
===================================================================
--- a/background.js
+++ b/background.js
@@ -72,6 +72,7 @@
this.title = "Subscription " + url;
this.disabled = false;
this.lastDownload = 1234;
+ this.downloadStatus = "synchronize_ok";
},
SpecialSubscription: function(url)
@@ -83,6 +84,9 @@
};
modules.subscriptionClasses.Subscription.fromURL = function(url)
{
+ if (url in knownSubscriptions)
+ return knownSubscriptions[url];
+
if (/^https?:\/\//.test(url))
return new modules.subscriptionClasses.Subscription(url);
else
@@ -184,7 +188,13 @@
};
modules.synchronizer = {
- Synchronizer: {}
+ Synchronizer: {
+ execute: function(subscription, manual)
+ {
+ subscription.lastDownload = Date.now() / 1000;
+ modules.filterNotifier.FilterNotifier.triggerListeners("subscription.updated", subscription);
Thomas Greiner 2016/01/19 11:27:26 Since you added error messages as part of this rev
saroyanm 2016/01/22 09:55:03 Done.
+ }
+ }
};
modules.matcher = {
« no previous file with comments | « no previous file | locale/en-US/options.json » ('j') | locale/en-US/options.json » ('J')

Powered by Google App Engine
This is Rietveld