notification fixes

background.js

Index: background.js
===================================================================
--- a/background.js
+++ b/background.js
@@ -347,7 +347,8 @@
     var message = texts.message ? texts.message.replace(/<\/?(a|strong)>/g, "") : "";
     var iconUrl = ext.getURL("icons/abp-128.png");
     var hasLinks = activeNotification.links && activeNotification.links.length > 0;
-    if ("browserNotifications" in ext) 
+    // Chrome on Linux does not fully support chrome.notifications yet ( https://code.google.com/p/chromium/issues/detail?id=291485 )
+    if (typeof chrome != "undefined" && "notifications" in chrome && navigator.platform.indexOf("Linux") == -1)

[Thomas Greiner, 2014/03/04 09:48:48]

You can find the platform information in the info module. Use that instead of
checking for the global "chrome" variable.

[saroyanm, 2014/03/04 10:53:26]

Thanks for pointing to the module.

     {
       var opts = {
         type: "basic",
@@ -362,7 +363,7 @@
       while (match = regex.exec(plainMessage))
         opts.buttons.push({title: match[1]});
       
-      var notification = ext.browserNotifications;
+      var notification = chrome.notifications;
       notification.create("", opts, function() {});
       notification.onClosed.addListener(prepareNotificationIconAndPopup);
       notification.onButtonClicked.addListener(notificationButtonClick);