Changeset 7348

Timestamp:

02/18/07 21:11:34 (21 months ago)

Author:

jburke

Message:

References #2366. Dumb ordering mistake. Caused lots of DnD test failures. Hooray for unit tests.

Files:

• branches/0.4/src/loader_xd.js (modified) (2 diffs)

branches/0.4/src/loader_xd.js

@@ -109 +109 @@
         //Add the module (package) to the list of modules.
         if(this.isXDomain){
-                //Curious: is this array going to get whacked with multiple access since scripts
-                //load asynchronously and may be accessing the array at the same time?
-                //JS is single-threaded supposedly, so it should be ok. And we don't need
-                //a precise ordering.
-                this.xdOrderedReqs.push(module);
-
-                //Add to waiting packages.
                 //If this is a __package__.js file, then this must be
                 //a package.* request (since xdomain can only work with the first
@@ -124 +117 @@
                 }
 
+                this.xdOrderedReqs.push(module);
+
+                //Add to waiting packages.
                 this.xdInFlight[module] = true;