RUN-2934 try/catch the call the getAllWindows (#116)

HadoukenIO · May 2, 2017 · 3d85eda · 3d85eda
1 parent e7bb57f
commit 3d85eda
Showing 1 changed file with 15 additions and 11 deletions.
diff --git a/src/browser/api/notifications/subscriptions.ts b/src/browser/api/notifications/subscriptions.ts
@@ -166,22 +166,26 @@ ofEvents.on('application/window-end-load/*', (e: any) => {
 
 seqs.requestNoteClose
     .subscribe((req: NotificationMessage) => {
-        const noteIsOpen = windowIsValid(req.id);
+        try {
+            const noteIsOpen = windowIsValid(req.id);
 
-        if (noteIsOpen) {
-            const ns = getCurrNotes();
-            const mousePos = System.getMousePosition();
-            const monitorInfo = getPrimaryMonitorAvailableRect();
-            const mouseOver = mouseisOverNotes(mousePos, monitorInfo, ns.length);
+            if (noteIsOpen) {
+                const ns = getCurrNotes();
+                const mousePos = System.getMousePosition();
+                const monitorInfo = getPrimaryMonitorAvailableRect();
+                const mouseOver = mouseisOverNotes(mousePos, monitorInfo, ns.length);
 
-            if (!mouseOver || req.data.force) {
-                closeNotification(req);
+                if (!mouseOver || req.data.force) {
+                    closeNotification(req);
 
+                } else {
+                    scheduleNoteClose(req, 1000);
+                }
             } else {
-                scheduleNoteClose(req, 1000);
+                removePendingNote(req.id);
             }
-        } else {
-            removePendingNote(req.id);
+        } catch (e) {
+            writeToLog('info', e);
         }
     });