Async transitions

This is the Ember side of
tildeio/router.js#19

All set, just keep in mind that there is probably more discussion to be
had over the `beforeModel` and `afterModel` hook names, which might
change before RC6.

Author: machty

ember-dev.yml

@@ -19,7 +19,6 @@ testing_suites:
     - "package=all&jquery=1.8.3&nojshint=true"
     - "package=all&jquery=git&nojshint=true"
     - "package=all&jquery=git2&nojshint=true"
-    - "package=all&enablerouteto&nojshint=true"
     - "package=all&extendprototypes=true&nojshint=true"
     - "package=all&extendprototypes=true&jquery=git&nojshint=true"
     - "package=all&extendprototypes=true&jquery=git2&nojshint=true"

packages/ember-application/lib/system/application.js

@@ -136,11 +136,14 @@ DeprecatedContainer.prototype = {
 
   In addition to creating your application's router, `Ember.Application` is
   also responsible for telling the router when to start routing. Transitions
-  between routes can be logged with the LOG_TRANSITIONS flag:
+  between routes can be logged with the LOG_TRANSITIONS flag, and more
+  detailed intra-transition logging can be logged with
+  the LOG_TRANSITIONS_INTERNAL flag:
 
   ```javascript
   window.App = Ember.Application.create({
-    LOG_TRANSITIONS: true
+    LOG_TRANSITIONS: true, // basic logging of successful transitions
+    LOG_TRANSITIONS_INTERNAL: true // detailed logging of all routing steps
   });
   ```
 

packages/ember-application/tests/system/reset_test.js

@@ -155,7 +155,10 @@ test("When an application is reset, the router URL is reset to `/`", function()
   router = application.__container__.lookup('router:main');
 
   location = router.get('location');
-  location.handleURL('/one');
+
+  Ember.run(function() {
+    location.handleURL('/one');
+  });
 
   application.reset();
 
@@ -168,7 +171,9 @@ test("When an application is reset, the router URL is reset to `/`", function()
   equal(get(applicationController, 'currentPath'), "index");
 
   location = application.__container__.lookup('router:main').get('location');
-  location.handleURL('/one');
+  Ember.run(function() {
+    location.handleURL('/one');
+  });
 
   equal(get(applicationController, 'currentPath'), "one");
 });

packages/ember-routing/lib/ext.js

@@ -1,2 +1,16 @@
 require('ember-routing/ext/controller');
 require('ember-routing/ext/view');
+
+/**
+@module ember
+@submodule ember-views
+*/
+
+// Add a new named queue after the 'actions' queue (where RSVP promises
+// resolve), which is used in router transitions to prevent unnecessary
+// loading state entry if all context promises resolve on the 
+// 'actions' queue first.
+
+var queues = Ember.run.queues,
+    indexOf = Ember.ArrayPolyfills.indexOf;
+queues.splice(indexOf.call(queues, 'actions') + 1, 0, 'routerTransitions');

packages/ember-routing/lib/helpers/link_to.js

@@ -118,23 +118,13 @@ Ember.onLoad('Ember.Handlebars', function(Handlebars) {
 
       if (get(this, '_isDisabled')) { return false; }
 
-      var router = this.get('router');
-
-      if (Ember.ENV.ENABLE_ROUTE_TO) {
-
-        var routeArgs = args(this, router);
+      var router = this.get('router'),
+          routeArgs = args(this, router);
 
-        router.routeTo(Ember.TransitionEvent.create({
-          transitionMethod: this.get('replace') ? 'replaceWith' : 'transitionTo',
-          destinationRouteName: routeArgs[0],
-          contexts: routeArgs.slice(1)
-        }));
+      if (this.get('replace')) {
+        router.replaceWith.apply(router, routeArgs);
       } else {
-        if (this.get('replace')) {
-          router.replaceWith.apply(router, args(this, router));
-        } else {
-          router.transitionTo.apply(router, args(this, router));
-        }
+        router.transitionTo.apply(router, routeArgs);
       }
     },
 

packages/ember-routing/lib/system.js

@@ -2,4 +2,3 @@ require('ember-routing/system/dsl');
 require('ember-routing/system/controller_for');
 require('ember-routing/system/router');
 require('ember-routing/system/route');
-require('ember-routing/system/transition_event');