increase MessageChannel priority in $.task, close #95

zloirock · Aug 6, 2015 · a6bee17 · a6bee17
1 parent 24ff8bb
commit a6bee17
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 18 deletions.
diff --git a/library/modules/$.task.js b/library/modules/$.task.js
@@ -31,7 +31,7 @@ if(!setTask || !clearTask){
     queue[++counter] = function(){
       invoke(typeof fn == 'function' ? fn : Function(fn), args);
     };
-    defer(counter);
+    defer(counter + '');
     return counter;
   };
   clearTask = function clearImmediate(id){
@@ -42,19 +42,19 @@ if(!setTask || !clearTask){
     defer = function(id){
       process.nextTick(ctx(run, id, 1));
     };
-  // Modern browsers, skip implementation for WebWorkers
-  // IE8 has postMessage, but it's sync & typeof its postMessage is 'object'
-  } else if(global.addEventListener && typeof postMessage == 'function' && !global.importScripts){
-    defer = function(id){
-      global.postMessage(id, '*');
-    };
-    global.addEventListener('message', listner, false);
-  // WebWorkers
+  // Browsers with MessageChannel, includes WebWorkers
   } else if(MessageChannel){
     channel = new MessageChannel;
     port    = channel.port2;
     channel.port1.onmessage = listner;
     defer = ctx(port.postMessage, port, 1);
+  // Browsers with postMessage, skip WebWorkers
+  // IE8 has postMessage, but it's sync & typeof its postMessage is 'object'
+  } else if(global.addEventListener && typeof postMessage == 'function' && !global.importScript){
+    defer = function(id){
+      global.postMessage(id, '*');
+    };
+    global.addEventListener('message', listner, false);
   // IE8-
   } else if(ONREADYSTATECHANGE in cel('script')){
     defer = function(id){

diff --git a/modules/$.task.js b/modules/$.task.js
@@ -31,7 +31,7 @@ if(!setTask || !clearTask){
     queue[++counter] = function(){
       invoke(typeof fn == 'function' ? fn : Function(fn), args);
     };
-    defer(counter);
+    defer(counter + '');
     return counter;
   };
   clearTask = function clearImmediate(id){
@@ -42,19 +42,19 @@ if(!setTask || !clearTask){
     defer = function(id){
       process.nextTick(ctx(run, id, 1));
     };
-  // Modern browsers, skip implementation for WebWorkers
-  // IE8 has postMessage, but it's sync & typeof its postMessage is 'object'
-  } else if(global.addEventListener && typeof postMessage == 'function' && !global.importScripts){
-    defer = function(id){
-      global.postMessage(id, '*');
-    };
-    global.addEventListener('message', listner, false);
-  // WebWorkers
+  // Browsers with MessageChannel, includes WebWorkers
   } else if(MessageChannel){
     channel = new MessageChannel;
     port    = channel.port2;
     channel.port1.onmessage = listner;
     defer = ctx(port.postMessage, port, 1);
+  // Browsers with postMessage, skip WebWorkers
+  // IE8 has postMessage, but it's sync & typeof its postMessage is 'object'
+  } else if(global.addEventListener && typeof postMessage == 'function' && !global.importScript){
+    defer = function(id){
+      global.postMessage(id, '*');
+    };
+    global.addEventListener('message', listner, false);
   // IE8-
   } else if(ONREADYSTATECHANGE in cel('script')){
     defer = function(id){