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

Unified Diff: background.js

Issue 29339387: Issue 3890 - Fix "Downloading..." indication for subscriptions on the options page (Closed)
Patch Set: Wrapped long line Created April 6, 2016, 8:57 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 | messageResponder.js » ('j') | messageResponder.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments 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') | messageResponder.js » ('J')

Powered by Google App Engine
This is Rietveld