IE8 Fix, take 1

postaljs · May 12, 2014 · 6a75c60 · 6a75c60
1 parent 369af23
commit 6a75c60
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 6 deletions.
diff --git a/lib/postal.xframe.js b/lib/postal.xframe.js
@@ -30,7 +30,8 @@
     // out he didn"t know JavaScript, and our passports were stolen on the
     // return trip. We stowed away aboard a freighter headed back to the
     // US and by the time we got back, no one had heard of IE 8 or 9. True story.
-    var useEagerSerialize = /MSIE [8,9]/.test(navigator.userAgent);
+    var noPostMessageApply;
+    var useEagerSerialize = noPostMessageApply = /MSIE [8,9]/.test(navigator.userAgent);
     var _memoRemoteByInstanceId = function (memo, instanceId) {
         var proxy = _.find(this.remotes, function (x) {
             return x.instanceId === instanceId;
@@ -83,7 +84,11 @@
                     if (!this.options.isWorker && !_envIsWorker) {
                         args.push(this.options.origin);
                     }
-                    this.target.postMessage.apply(context, args);
+                    if (noPostMessageApply) {
+                        this.target.postMessage(args[0], args[1]);
+                    } else {
+                        this.target.postMessage.apply(context, args);
+                    }
                 }
             }
         }, {

diff --git a/lib/postal.xframe.min.js b/lib/postal.xframe.min.js
diff --git a/src/xframe.js b/src/xframe.js
@@ -1,5 +1,5 @@
 /* global postal */
-/*jshint -W003 */
+/*jshint -W003,-W120 */
 var _origin = location.origin || location.protocol + "//" + location.host;
 
 function listener() {
@@ -13,7 +13,8 @@ function listener() {
 // out he didn"t know JavaScript, and our passports were stolen on the
 // return trip. We stowed away aboard a freighter headed back to the
 // US and by the time we got back, no one had heard of IE 8 or 9. True story.
-var useEagerSerialize = /MSIE [8,9]/.test(navigator.userAgent);
+var noPostMessageApply;
+var useEagerSerialize = noPostMessageApply = /MSIE [8,9]/.test(navigator.userAgent);
 
 var _memoRemoteByInstanceId = function(memo, instanceId) {
     var proxy = _.find(this.remotes, function(x) {
@@ -72,7 +73,11 @@ var XFRAME = "xframe",
                 if (!this.options.isWorker && !_envIsWorker) {
                     args.push(this.options.origin);
                 }
-                this.target.postMessage.apply(context, args);
+                if (noPostMessageApply) {
+                    this.target.postMessage(args[0], args[1]);
+                } else {
+                    this.target.postMessage.apply(context, args);
+                }
             }
         }
     }, {