From 1699040649717869b7ed0138463765829329afad Mon Sep 17 00:00:00 2001
From: Chris <chris@chrismatic.io>
Date: Mon, 23 Mar 2015 13:01:50 -0500
Subject: [PATCH] Safari: better popup blocking

---
 platform/safari/vapi-background.js | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/platform/safari/vapi-background.js b/platform/safari/vapi-background.js
index 8ecc02820..be51553f3 100644
--- a/platform/safari/vapi-background.js
+++ b/platform/safari/vapi-background.js
@@ -248,8 +248,8 @@
     /******************************************************************************/
 
     vAPI.tabs.registerListeners = function() {
-        safari.application.addEventListener('beforeNavigate', function(e) {
-            if(!vAPI.tabs.popupCandidate || !e.target || e.url === 'about:blank') {
+        safari.application.addEventListener("beforeNavigate", function(e) {
+            if(!vAPI.tabs.popupCandidate || !e.target || e.url === "about:blank") {
                 return;
             }
             var url = e.url,
@@ -259,7 +259,6 @@
                 targetTabId: tabId,
                 openerTabId: vAPI.tabs.popupCandidate
             };
-            vAPI.tabs.popupCandidate = false;
             if(vAPI.tabs.onPopup(details)) {
                 e.preventDefault();
                 if(vAPI.tabs.stack[details.openerTabId]) {
@@ -702,9 +701,9 @@
             }
             switch(e.message.type) {
                 case "popup":
-                    if(e.message.url === 'about:blank') {
-                        vAPI.tabs.popupCandidate = vAPI.tabs.getTabId(e.target);
-                        e.message = true;
+                    vAPI.tabs.popupCandidate = vAPI.tabs.getTabId(e.target);
+                    if(e.message.url === "about:blank") {
+                        e.message = false;
                         return;
                     }
                     else {