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

Unified Diff: background.js

Issue 29567743: Issue 5835 - Add some notification related message handlers (Closed)
Patch Set: Addressed Sebastian's feedback Created Oct. 8, 2017, 10:07 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
diff --git a/background.js b/background.js
index f7af936b90bafd166bcb789a04df024132701578..fb48e4e6aa19427713ed7cbb8a9fe302b9b7ec75 100644
--- a/background.js
+++ b/background.js
@@ -140,6 +140,11 @@
}
};
+ modules.notificationHelper = {
+ getActiveNotification() { },
+ shouldDisplay() { return true; }
Thomas Greiner 2017/10/09 10:52:59 Coding style: "Opening braces always go on their o
kzar 2017/10/09 11:13:51 Done.
+ };
+
let subscriptionServer = "https://easylist-downloads.adblockplus.org";
let subscriptionDetails = {
[`${subscriptionServer}/easylistgermany+easylist.txt`]: {
« no previous file with comments | « no previous file | messageResponder.js » ('j') | messageResponder.js » ('J')

Powered by Google App Engine
This is Rietveld