Unified Diff: lib/notificationHelper.js
Issue 29371739:
Issue 4722 - Remove old check for Chrome notification API (Closed)
Patch Set: Check for chrome directly instead of platform
Created Jan. 16, 2017, 4:14 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/notificationHelper.js |
| diff --git a/lib/notificationHelper.js b/lib/notificationHelper.js |
| index 84f34b3dcf966543a918bfae43efc78bd6565fbe..8c5485835e926436c7c93e7fac0f18d79b2166cd 100644 |
| --- a/lib/notificationHelper.js |
| +++ b/lib/notificationHelper.js |
| @@ -33,20 +33,7 @@ displayMethods.question = ["notification"]; |
| displayMethods.normal = ["notification"]; |
| displayMethods.information = ["icon", "popup"]; |
| -// Chrome on Linux does not fully support chrome.notifications until version 35 |
| -// https://code.google.com/p/chromium/issues/detail?id=291485 |
| -let canUseChromeNotifications = (function() |
| -{ |
| - let info = require("info"); |
| - if (info.platform == "chromium" && "notifications" in chrome) |
| - { |
| - if (navigator.platform.indexOf("Linux") == -1) |
| - return true; |
| - if (Services.vc.compare(info.applicationVersion, "35") >= 0) |
| - return true; |
| - } |
| - return false; |
| -})(); |
| +let canUseChromeNotifications = typeof chrome != "undefined" && "notifications" in chrome; |
| function prepareNotificationIconAndPopup() |
| { |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
