Merge pull request #46 from jbyers/master

Fix iframe bug as a side-effect of security policy changes in FF 49
cmlenz · Oct 29, 2016 · f9a5446 · f9a5446
2 parents 814c484 + ba8a3da
commit f9a5446
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/jquery.iframe-transport.js b/jquery.iframe-transport.js
@@ -122,7 +122,7 @@
       });
       form.remove();
       iframe.one("load", function() { iframe.remove(); });
-      iframe.attr("src", "javascript:false;");
+      iframe.attr("src", "about:blank");
     }
 
     // Remove "iframe" from the data types list so that further processing is
@@ -190,7 +190,7 @@
         // The `send` function is called by jQuery when the request should be
         // sent.
         send: function(headers, completeCallback) {
-          iframe = $("<iframe src='javascript:false;' name='" + name +
+          iframe = $("<iframe src='about:blank' name='" + name +
             "' id='" + name + "' style='display:none'></iframe>");
 
           // The first load event gets fired after the iframe has been injected
@@ -235,7 +235,7 @@
         // aborted.
         abort: function() {
           if (iframe !== null) {
-            iframe.unbind("load").attr("src", "javascript:false;");
+            iframe.unbind("load").attr("src", "about:blank");
             cleanUp();
           }
         }