Unified Diff: background.js
Issue 29339387:
Issue 3890 - Fix "Downloading..." indication for subscriptions on the options page (Closed)
Patch Set: Pass URL to Synchronizer.isExecuting()
Created April 6, 2016, 5:20 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
messageResponder.js » ('j')
|
options.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: background.js |
| =================================================================== |
| --- a/background.js |
| +++ b/background.js |
| @@ -287,8 +287,10 @@ |
| _downloading: false, |
| execute: function(subscription, manual) |
| { |
| - subscription.lastDownload = 0; |
| modules.synchronizer.Synchronizer._downloading = true; |
| + modules.filterNotifier.FilterNotifier.emit( |
| + "subscription.downloading", subscription |
| + ); |
| setTimeout(function() |
| { |
| modules.synchronizer.Synchronizer._downloading = false; |
« no previous file with comments
|
« no previous file
|
messageResponder.js » ('j')
|
options.js » ('J')
