Use alternative means to check if using browser vs cordova

LetterboxDev · Nov 12, 2015 · ccc42e0 · ccc42e0
1 parent d52c481
commit ccc42e0
Showing 1 changed file with 23 additions and 23 deletions.
diff --git a/www/js/services/chromeNotifService.js b/www/js/services/chromeNotifService.js
@@ -3,39 +3,39 @@ angular.module('letterbox.services')
 .service('ChromeNotifService', function($timeout, eventbus, ChatService) {
   var ChromeNotif = this;
 
-  if (!window.cordova) {
+  if (document.URL.indexOf( 'http://' ) !== -1 || document.URL.indexOf( 'https://' ) !== -1) {
     var Notification = window.Notification ||
                        window.mozNotification ||
                        window.webkitNotification;
 
     if (Notification.permission !== 'granted') {
       Notification.requestPermission();
     }
-  }
-
-  eventbus.registerListener('roomMessage', function(roomMessage) {
-    var message = roomMessage.message;
-    var senderName = roomMessage.senderName;
-    var formattedMessage = ChatService.formatMessage(message);
 
-    if (!window.cordova && !formattedMessage.isOwner) {
-      show(senderName, formattedMessage.content);
-    }
-  });
+    eventbus.registerListener('roomMessage', function(roomMessage) {
+      var message = roomMessage.message;
+      var senderName = roomMessage.senderName;
+      var formattedMessage = ChatService.formatMessage(message);
 
-  function show(title, message) {
-    var instance = new Notification(
-      title, {
-        body: message,
-        icon: "img/android-icon-48x48.png"
+      if (!formattedMessage.isOwner) {
+        show(senderName, formattedMessage.content);
       }
-    );
-
-    instance.onshow = function () {
-      $timeout(function(){
-        instance.close();
-      }, 10000);
-    };
+    });
+
+    function show(title, message) {
+      var instance = new Notification(
+        title, {
+          body: message,
+          icon: "img/android-icon-48x48.png"
+        }
+      );
+
+      instance.onshow = function () {
+        $timeout(function(){
+          instance.close();
+        }, 10000);
+      };
+    }
   }
 });