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

Issue 5730260331003904: Issue 2419 - Show newly downloaded notifications immediately (Closed)

Created:
June 8, 2015, 8 p.m. by Felix Dahlke
Modified:
June 15, 2015, 10:40 a.m.
Reviewers:
Wladimir Palant
Visibility:
Public.

Description

Issue 2419 - Show newly downloaded notifications immediately

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -7 lines) Patch
M lib/notification.js View 3 chunks +2 lines, -7 lines 1 comment Download

Messages

Total messages: 2
Felix Dahlke
http://codereview.adblockplus.org/5730260331003904/diff/5629499534213120/lib/notification.js File lib/notification.js (left): http://codereview.adblockplus.org/5730260331003904/diff/5629499534213120/lib/notification.js#oldcode91 lib/notification.js:91: notificationTimer = Cc["@mozilla.org/timer;1"].createInstance(Ci.nsITimer); While we had IIRC discussed that ...
June 8, 2015, 8:06 p.m. (2015-06-08 20:06:23 UTC) #1
Wladimir Palant
June 8, 2015, 8:48 p.m. (2015-06-08 20:48:46 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld