ember-source 4.4.0-beta.1 → 4.5.0-alpha.1

package/dist/packages/@ember/application/lib/application.js

@@ -167,156 +167,63 @@ import { RouterService } from '@ember/-internals/routing';
 
   @class Application
   @extends Engine
-  @uses RegistryProxyMixin
   @public
 */
 
-const Application = Engine.extend({
-  /**
-    The root DOM element of the Application. This can be specified as an
-    element or a [selector string](https://developer.mozilla.org/en-US/docs/Learn/CSS/Building_blocks/Selectors#reference_table_of_selectors).
-     This is the element that will be passed to the Application's,
-    `eventDispatcher`, which sets up the listeners for event delegation. Every
-    view in your application should be a child of the element you specify here.
-     @property rootElement
-    @type DOMElement
-    @default 'body'
-    @public
-  */
-  rootElement: 'body',
-
+class Application extends Engine {
+  constructor() {
+    super(...arguments);
+    this._bootPromise = null;
+    this._bootResolver = null;
+  }
   /**
-     @property _document
-    @type Document | null
-    @default 'window.document'
+    This creates a registry with the default Ember naming conventions.
+       It also configures the registry:
+       * registered views are created every time they are looked up (they are
+      not singletons)
+    * registered templates are not factories; the registered value is
+      returned directly.
+    * the router receives the application as its `namespace` property
+    * all controllers receive the router as their `target` and `controllers`
+      properties
+    * all controllers receive the application as their `namespace` property
+    * the application view receives the application controller as its
+      `controller` property
+    * the application view receives the application template as its
+      `defaultTemplate` property
+       @method buildRegistry
+    @static
+    @param {Application} namespace the application for which to
+      build the registry
+    @return {Ember.Registry} the built registry
     @private
   */
-  _document: hasDOM ? window.document : null,
-
-  /**
-    The `Ember.EventDispatcher` responsible for delegating events to this
-    application's views.
-     The event dispatcher is created by the application at initialization time
-    and sets up event listeners on the DOM element described by the
-    application's `rootElement` property.
-     See the documentation for `Ember.EventDispatcher` for more information.
-     @property eventDispatcher
-    @type Ember.EventDispatcher
-    @default null
-    @public
-  */
-  eventDispatcher: null,
-
-  /**
-    The DOM events for which the event dispatcher should listen.
-     By default, the application's `Ember.EventDispatcher` listens
-    for a set of standard DOM events, such as `mousedown` and
-    `keyup`, and delegates them to your application's `Ember.View`
-    instances.
-     If you would like additional bubbling events to be delegated to your
-    views, set your `Application`'s `customEvents` property
-    to a hash containing the DOM event name as the key and the
-    corresponding view method name as the value. Setting an event to
-    a value of `null` will prevent a default event listener from being
-    added for that event.
-     To add new events to be listened to:
-     ```app/app.js
-    import Application from '@ember/application';
-     let App = Application.extend({
-      customEvents: {
-        // add support for the paste event
-        paste: 'paste'
-      }
-    });
-    ```
-     To prevent default events from being listened to:
-     ```app/app.js
-    import Application from '@ember/application';
-     let App = Application.extend({
-      customEvents: {
-        // remove support for mouseenter / mouseleave events
-        mouseenter: null,
-        mouseleave: null
-      }
-    });
-    ```
-    @property customEvents
-    @type Object
-    @default null
-    @public
-  */
-  customEvents: null,
 
-  /**
-    Whether the application should automatically start routing and render
-    templates to the `rootElement` on DOM ready. While default by true,
-    other environments such as FastBoot or a testing harness can set this
-    property to `false` and control the precise timing and behavior of the boot
-    process.
-     @property autoboot
-    @type Boolean
-    @default true
-    @private
-  */
-  autoboot: true,
 
-  /**
-    Whether the application should be configured for the legacy "globals mode".
-    Under this mode, the Application object serves as a global namespace for all
-    classes.
-     ```javascript
-    import Application from '@ember/application';
-    import Component from '@ember/component';
-     let App = Application.create({
-      ...
-    });
-     App.Router.reopen({
-      location: 'none'
-    });
-     App.Router.map({
-      ...
-    });
-     App.MyComponent = Component.extend({
-      ...
-    });
-    ```
-     This flag also exposes other internal APIs that assumes the existence of
-    a special "default instance", like `App.__container__.lookup(...)`.
-     This option is currently not configurable, its value is derived from
-    the `autoboot` flag – disabling `autoboot` also implies opting-out of
-    globals mode support, although they are ultimately orthogonal concerns.
-     Some of the global modes features are already deprecated in 1.x. The
-    existence of this flag is to untangle the globals mode code paths from
-    the autoboot code paths, so that these legacy features can be reviewed
-    for deprecation/removal separately.
-     Forcing the (autoboot=true, _globalsMode=false) here and running the tests
-    would reveal all the places where we are still relying on these legacy
-    behavior internally (mostly just tests).
-     @property _globalsMode
-    @type Boolean
-    @default true
-    @private
-  */
-  _globalsMode: true,
+  static buildRegistry(namespace) {
+    let registry = super.buildRegistry(namespace);
+    commonSetupRegistry(registry);
+    setupApplicationRegistry(registry);
+    return registry;
+  }
 
-  /**
-    An array of application instances created by `buildInstance()`. Used
-    internally to ensure that all instances get destroyed.
-     @property _applicationInstances
-    @type Array
-    @default null
-    @private
-  */
-  _applicationInstances: null,
+  init(properties) {
+    var _a, _b, _c, _d, _e, _f, _g, _h;
 
-  init() {
-    this._super(...arguments);
+    super.init(properties);
+    (_a = this.rootElement) !== null && _a !== void 0 ? _a : this.rootElement = 'body';
+    (_b = this._document) !== null && _b !== void 0 ? _b : this._document = null;
+    (_c = this.eventDispatcher) !== null && _c !== void 0 ? _c : this.eventDispatcher = null;
+    (_d = this.customEvents) !== null && _d !== void 0 ? _d : this.customEvents = null;
+    (_e = this.autoboot) !== null && _e !== void 0 ? _e : this.autoboot = true;
+    (_f = this._document) !== null && _f !== void 0 ? _f : this._document = hasDOM ? window.document : null;
+    (_g = this._globalsMode) !== null && _g !== void 0 ? _g : this._globalsMode = true;
 
     if (DEBUG) {
       if (ENV.LOG_VERSION) {
         // we only need to see this once per Application#init
         ENV.LOG_VERSION = false;
-        libraries.logVersions();
+        (_h = libraries.logVersions) === null || _h === void 0 ? void 0 : _h.call(libraries);
       }
     } // Start off the number of deferrals at 1. This will be decremented by
     // the Application's own `boot` method.
@@ -334,51 +241,56 @@ const Application = Engine.extend({
     if (this.autoboot) {
       this.waitForDOMReady();
     }
-  },
-
+  }
   /**
     Create an ApplicationInstance for this application.
-     @public
+       @public
     @method buildInstance
     @return {ApplicationInstance} the application instance
   */
+
+
   buildInstance(options = {}) {
     assert('You cannot build new instances of this application since it has already been destroyed', !this.isDestroyed);
     assert('You cannot build new instances of this application since it is being destroyed', !this.isDestroying);
-    options.base = this;
-    options.application = this;
-    return ApplicationInstance.create(options);
-  },
-
+    return ApplicationInstance.create(Object.assign(Object.assign({}, options), {
+      base: this,
+      application: this
+    }));
+  }
   /**
     Start tracking an ApplicationInstance for this application.
     Used when the ApplicationInstance is created.
-     @private
+       @private
     @method _watchInstance
   */
+
+
   _watchInstance(instance) {
     this._applicationInstances.add(instance);
-  },
-
+  }
   /**
     Stop tracking an ApplicationInstance for this application.
     Used when the ApplicationInstance is about to be destroyed.
-     @private
+       @private
     @method _unwatchInstance
   */
+
+
   _unwatchInstance(instance) {
     return this._applicationInstances.delete(instance);
-  },
-
+  }
   /**
     Enable the legacy globals mode by allowing this application to act
     as a global namespace. See the docs on the `_globalsMode` property
     for details.
-     Most of these features are already deprecated in 1.x, so we can
+       Most of these features are already deprecated in 1.x, so we can
     stop using them internally and try to remove them.
-     @private
+       @private
     @method _prepareForGlobalsMode
   */
+
+
   _prepareForGlobalsMode() {
     // Create subclass of Router for this Application instance.
     // This is to ensure that someone reopening `App.Router` does not
@@ -386,20 +298,21 @@ const Application = Engine.extend({
     this.Router = (this.Router || Router).extend();
 
     this._buildDeprecatedInstance();
-  },
-
+  }
   /*
     Build the deprecated instance for legacy globals mode support.
     Called when creating and resetting the application.
-     This is orthogonal to autoboot: the deprecated instance needs to
+       This is orthogonal to autoboot: the deprecated instance needs to
     be created at Application construction (not boot) time to expose
     App.__container__. If autoboot sees that this instance exists,
     it will continue booting it to avoid doing unncessary work (as
     opposed to building a new instance at boot time), but they are
     otherwise unrelated.
-     @private
+       @private
     @method _buildDeprecatedInstance
   */
+
+
   _buildDeprecatedInstance() {
     // Build a default instance
     let instance = this.buildInstance(); // Legacy support for App.__container__ and other global methods
@@ -407,42 +320,43 @@ const Application = Engine.extend({
 
     this.__deprecatedInstance__ = instance;
     this.__container__ = instance.__container__;
-  },
-
+  }
   /**
     Automatically kick-off the boot process for the application once the
     DOM has become ready.
-     The initialization itself is scheduled on the actions queue which
+       The initialization itself is scheduled on the actions queue which
     ensures that code-loading finishes before booting.
-     If you are asynchronously loading code, you should call `deferReadiness()`
+       If you are asynchronously loading code, you should call `deferReadiness()`
     to defer booting, and then call `advanceReadiness()` once all of your code
     has finished loading.
-     @private
+       @private
     @method waitForDOMReady
   */
+
+
   waitForDOMReady() {
-    if (this._document === null || this._document.readyState !== 'loading') {
-      schedule('actions', this, 'domReady');
+    const document = this._document;
+
+    if (document === null || document.readyState !== 'loading') {
+      schedule('actions', this, this.domReady);
     } else {
       let callback = () => {
-        this._document.removeEventListener('DOMContentLoaded', callback);
-
-        run(this, 'domReady');
+        document.removeEventListener('DOMContentLoaded', callback);
+        run(this, this.domReady);
       };
 
-      this._document.addEventListener('DOMContentLoaded', callback);
+      document.addEventListener('DOMContentLoaded', callback);
     }
-  },
-
+  }
   /**
     This is the autoboot flow:
-     1. Boot the app by calling `this.boot()`
+       1. Boot the app by calling `this.boot()`
     2. Create an instance (or use the `__deprecatedInstance__` in globals mode)
     3. Boot the instance by calling `instance.boot()`
     4. Invoke the `App.ready()` callback
     5. Kick-off routing on the instance
-     Ideally, this is all we would need to do:
-     ```javascript
+       Ideally, this is all we would need to do:
+       ```javascript
     _autoBoot() {
       this.boot().then(() => {
         let instance = (this._globalsMode) ? this.__deprecatedInstance__ : this.buildInstance();
@@ -453,16 +367,18 @@ const Application = Engine.extend({
       });
     }
     ```
-     Unfortunately, we cannot actually write this because we need to participate
+       Unfortunately, we cannot actually write this because we need to participate
     in the "synchronous" boot process. While the code above would work fine on
     the initial boot (i.e. DOM ready), when `App.reset()` is called, we need to
     boot a new instance synchronously (see the documentation on `_bootSync()`
     for details).
-     Because of this restriction, the actual logic of this method is located
+       Because of this restriction, the actual logic of this method is located
     inside `didBecomeReady()`.
-     @private
+       @private
     @method domReady
   */
+
+
   domReady() {
     if (this.isDestroying || this.isDestroyed) {
       return;
@@ -470,45 +386,47 @@ const Application = Engine.extend({
 
     this._bootSync(); // Continues to `didBecomeReady`
 
-  },
-
+  }
   /**
     Use this to defer readiness until some condition is true.
-     Example:
-     ```javascript
+       Example:
+       ```javascript
     import Application from '@ember/application';
-     let App = Application.create();
-     App.deferReadiness();
-     fetch('/auth-token')
+       let App = Application.create();
+       App.deferReadiness();
+       fetch('/auth-token')
     .then(response => response.json())
     .then(data => {
       App.token = data.token;
       App.advanceReadiness();
     });
     ```
-     This allows you to perform asynchronous setup logic and defer
+       This allows you to perform asynchronous setup logic and defer
     booting your application until the setup has finished.
-     However, if the setup requires a loading UI, it might be better
+       However, if the setup requires a loading UI, it might be better
     to use the router for this purpose.
-     @method deferReadiness
+       @method deferReadiness
     @public
   */
+
+
   deferReadiness() {
     assert('You must call deferReadiness on an instance of Application', this instanceof Application);
     assert('You cannot defer readiness since application has already destroyed', !this.isDestroyed);
     assert('You cannot defer readiness since the application is being destroyed', !this.isDestroying);
     assert('You cannot defer readiness since the `ready()` hook has already been called', this._readinessDeferrals > 0);
     this._readinessDeferrals++;
-  },
-
+  }
   /**
     Call `advanceReadiness` after any asynchronous setup logic has completed.
     Each call to `deferReadiness` must be matched by a call to `advanceReadiness`
     or the application will never become ready and routing will not begin.
-     @method advanceReadiness
+       @method advanceReadiness
     @see {Application#deferReadiness}
     @public
   */
+
+
   advanceReadiness() {
     assert('You must call advanceReadiness on an instance of Application', this instanceof Application);
     assert('You cannot advance readiness since application has already destroyed', !this.isDestroyed);
@@ -519,21 +437,22 @@ const Application = Engine.extend({
     if (this._readinessDeferrals === 0) {
       once(this, this.didBecomeReady);
     }
-  },
-
+  }
   /**
     Initialize the application and return a promise that resolves with the `Application`
     object when the boot process is complete.
-     Run any application initializers and run the application load hook. These hooks may
+       Run any application initializers and run the application load hook. These hooks may
     choose to defer readiness. For example, an authentication hook might want to defer
     readiness until the auth token has been retrieved.
-     By default, this method is called automatically on "DOM ready"; however, if autoboot
+       By default, this method is called automatically on "DOM ready"; however, if autoboot
     is disabled, this is automatically called when the first application instance is
     created via `visit`.
-     @public
+       @public
     @method boot
     @return {Promise<Application,Error>}
   */
+
+
   boot() {
     assert('You cannot boot this application since it has already been destroyed', !this.isDestroyed);
     assert('You cannot boot this application since it is being destroyed', !this.isDestroying);
@@ -548,20 +467,22 @@ const Application = Engine.extend({
       // in the promise rejection
     }
 
+    assert('has boot promise', this._bootPromise);
     return this._bootPromise;
-  },
-
+  }
   /**
     Unfortunately, a lot of existing code assumes the booting process is
     "synchronous". Specifically, a lot of tests assumes the last call to
     `app.advanceReadiness()` or `app.reset()` will result in the app being
     fully-booted when the current runloop completes.
-     We would like new code (like the `visit` API) to stop making this assumption,
+       We would like new code (like the `visit` API) to stop making this assumption,
     so we created the asynchronous version above that returns a promise. But until
     we have migrated all the code, we would have to expose this method for use
     *internally* in places where we need to boot an app "synchronously".
-     @private
+       @private
   */
+
+
   _bootSync() {
     if (this._booted || this.isDestroying || this.isDestroyed) {
       return;
@@ -584,45 +505,44 @@ const Application = Engine.extend({
 
       throw error;
     }
-  },
-
+  }
   /**
     Reset the application. This is typically used only in tests. It cleans up
     the application in the following order:
-     1. Deactivate existing routes
+       1. Deactivate existing routes
     2. Destroy all objects in the container
     3. Create a new application container
     4. Re-route to the existing url
-     Typical Example:
-     ```javascript
+       Typical Example:
+       ```javascript
     import Application from '@ember/application';
     let App;
-     run(function() {
+       run(function() {
       App = Application.create();
     });
-     module('acceptance test', {
+       module('acceptance test', {
       setup: function() {
         App.reset();
       }
     });
-     test('first test', function() {
+       test('first test', function() {
       // App is freshly reset
     });
-     test('second test', function() {
+       test('second test', function() {
       // App is again freshly reset
     });
     ```
-     Advanced Example:
-     Occasionally you may want to prevent the app from initializing during
+       Advanced Example:
+       Occasionally you may want to prevent the app from initializing during
     setup. This could enable extra configuration, or enable asserting prior
     to the app becoming ready.
-     ```javascript
+       ```javascript
     import Application from '@ember/application';
     let App;
-     run(function() {
+       run(function() {
       App = Application.create();
     });
-     module('acceptance test', {
+       module('acceptance test', {
       setup: function() {
         run(function() {
           App.reset();
@@ -630,17 +550,19 @@ const Application = Engine.extend({
         });
       }
     });
-     test('first test', function() {
+       test('first test', function() {
       ok(true, 'something before app is initialized');
-       run(function() {
+         run(function() {
         App.advanceReadiness();
       });
-       ok(true, 'something after app is initialized');
+         ok(true, 'something after app is initialized');
     });
     ```
-     @method reset
+       @method reset
     @public
   */
+
+
   reset() {
     assert('You cannot reset this application since it has already been destroyed', !this.isDestroyed);
     assert('You cannot reset this application since it is being destroyed', !this.isDestroying);
@@ -655,6 +577,7 @@ const Application = Engine.extend({
     this._booted = false;
 
     function handleReset() {
+      assert('expected instance', instance);
       run(instance, 'destroy');
 
       this._buildDeprecatedInstance();
@@ -663,17 +586,20 @@ const Application = Engine.extend({
     }
 
     join(this, handleReset);
-  },
-
+  }
   /**
     @private
     @method didBecomeReady
   */
+
+
   didBecomeReady() {
     if (this.isDestroying || this.isDestroyed) {
       return;
     }
 
+    assert('expected _bootResolver', this._bootResolver);
+
     try {
       // TODO: Is this still needed for _globalsMode = false?
       // See documentation on `_autoboot()` for details
@@ -684,6 +610,7 @@ const Application = Engine.extend({
           // If we already have the __deprecatedInstance__ lying around, boot it to
           // avoid unnecessary work
           instance = this.__deprecatedInstance__;
+          assert('expected instance', instance);
         } else {
           // Otherwise, build an instance and boot it. This is currently unreachable,
           // because we forced _globalsMode to === autoboot; but having this branch
@@ -711,24 +638,25 @@ const Application = Engine.extend({
 
       throw error;
     }
-  },
-
+  }
   /**
     Called when the Application has become ready, immediately before routing
     begins. The call will be delayed until the DOM has become ready.
-     @event ready
+       @event ready
     @public
   */
+
+
   ready() {
     return this;
-  },
+  } // This method must be moved to the application instance object
+
 
-  // This method must be moved to the application instance object
   willDestroy() {
-    this._super(...arguments);
+    super.willDestroy();
 
-    if (_loaded.application === this) {
-      _loaded.application = undefined;
+    if (_loaded['application'] === this) {
+      _loaded['application'] = undefined;
     }
 
     if (this._applicationInstances.size) {
@@ -736,86 +664,85 @@ const Application = Engine.extend({
 
       this._applicationInstances.clear();
     }
-  },
-
+  }
   /**
     Boot a new instance of `ApplicationInstance` for the current
     application and navigate it to the given `url`. Returns a `Promise` that
     resolves with the instance when the initial routing and rendering is
     complete, or rejects with any error that occurred during the boot process.
-     When `autoboot` is disabled, calling `visit` would first cause the
+       When `autoboot` is disabled, calling `visit` would first cause the
     application to boot, which runs the application initializers.
-     This method also takes a hash of boot-time configuration options for
+       This method also takes a hash of boot-time configuration options for
     customizing the instance's behavior. See the documentation on
     `ApplicationInstance.BootOptions` for details.
-     `ApplicationInstance.BootOptions` is an interface class that exists
+       `ApplicationInstance.BootOptions` is an interface class that exists
     purely to document the available options; you do not need to construct it
     manually. Simply pass a regular JavaScript object containing of the
     desired options:
-     ```javascript
+       ```javascript
     MyApp.visit("/", { location: "none", rootElement: "#container" });
     ```
-     ### Supported Scenarios
-     While the `BootOptions` class exposes a large number of knobs, not all
+       ### Supported Scenarios
+       While the `BootOptions` class exposes a large number of knobs, not all
     combinations of them are valid; certain incompatible combinations might
     result in unexpected behavior.
-     For example, booting the instance in the full browser environment
+       For example, booting the instance in the full browser environment
     while specifying a foreign `document` object (e.g. `{ isBrowser: true,
     document: iframe.contentDocument }`) does not work correctly today,
     largely due to Ember's jQuery dependency.
-     Currently, there are three officially supported scenarios/configurations.
+       Currently, there are three officially supported scenarios/configurations.
     Usages outside of these scenarios are not guaranteed to work, but please
     feel free to file bug reports documenting your experience and any issues
     you encountered to help expand support.
-     #### Browser Applications (Manual Boot)
-     The setup is largely similar to how Ember works out-of-the-box. Normally,
+       #### Browser Applications (Manual Boot)
+       The setup is largely similar to how Ember works out-of-the-box. Normally,
     Ember will boot a default instance for your Application on "DOM ready".
     However, you can customize this behavior by disabling `autoboot`.
-     For example, this allows you to render a miniture demo of your application
+       For example, this allows you to render a miniture demo of your application
     into a specific area on your marketing website:
-     ```javascript
+       ```javascript
     import MyApp from 'my-app';
-     $(function() {
+       $(function() {
       let App = MyApp.create({ autoboot: false });
-       let options = {
+         let options = {
         // Override the router's location adapter to prevent it from updating
         // the URL in the address bar
         location: 'none',
-         // Override the default `rootElement` on the app to render into a
+           // Override the default `rootElement` on the app to render into a
         // specific `div` on the page
         rootElement: '#demo'
       };
-       // Start the app at the special demo URL
+         // Start the app at the special demo URL
       App.visit('/demo', options);
     });
     ```
-     Or perhaps you might want to boot two instances of your app on the same
+       Or perhaps you might want to boot two instances of your app on the same
     page for a split-screen multiplayer experience:
-     ```javascript
+       ```javascript
     import MyApp from 'my-app';
-     $(function() {
+       $(function() {
       let App = MyApp.create({ autoboot: false });
-       let sessionId = MyApp.generateSessionID();
-       let player1 = App.visit(`/matches/join?name=Player+1&session=${sessionId}`, { rootElement: '#left', location: 'none' });
+         let sessionId = MyApp.generateSessionID();
+         let player1 = App.visit(`/matches/join?name=Player+1&session=${sessionId}`, { rootElement: '#left', location: 'none' });
       let player2 = App.visit(`/matches/join?name=Player+2&session=${sessionId}`, { rootElement: '#right', location: 'none' });
-       Promise.all([player1, player2]).then(() => {
+         Promise.all([player1, player2]).then(() => {
         // Both apps have completed the initial render
         $('#loading').fadeOut();
       });
     });
     ```
-     Do note that each app instance maintains their own registry/container, so
+       Do note that each app instance maintains their own registry/container, so
     they will run in complete isolation by default.
-     #### Server-Side Rendering (also known as FastBoot)
-     This setup allows you to run your Ember app in a server environment using
+       #### Server-Side Rendering (also known as FastBoot)
+       This setup allows you to run your Ember app in a server environment using
     Node.js and render its content into static HTML for SEO purposes.
-     ```javascript
+       ```javascript
     const HTMLSerializer = new SimpleDOM.HTMLSerializer(SimpleDOM.voidMap);
-     function renderURL(url) {
+       function renderURL(url) {
       let dom = new SimpleDOM.Document();
       let rootElement = dom.body;
       let options = { isBrowser: false, document: dom, rootElement: rootElement };
-       return MyApp.visit(options).then(instance => {
+         return MyApp.visit(options).then(instance => {
         try {
           return HTMLSerializer.serialize(rootElement.firstChild);
         } finally {
@@ -824,50 +751,50 @@ const Application = Engine.extend({
       });
     }
     ```
-     In this scenario, because Ember does not have access to a global `document`
+       In this scenario, because Ember does not have access to a global `document`
     object in the Node.js environment, you must provide one explicitly. In practice,
     in the non-browser environment, the stand-in `document` object only needs to
     implement a limited subset of the full DOM API. The `SimpleDOM` library is known
     to work.
-     Since there is no DOM access in the non-browser environment, you must also
+       Since there is no DOM access in the non-browser environment, you must also
     specify a DOM `Element` object in the same `document` for the `rootElement` option
     (as opposed to a selector string like `"body"`).
-     See the documentation on the `isBrowser`, `document` and `rootElement` properties
+       See the documentation on the `isBrowser`, `document` and `rootElement` properties
     on `ApplicationInstance.BootOptions` for details.
-     #### Server-Side Resource Discovery
-     This setup allows you to run the routing layer of your Ember app in a server
+       #### Server-Side Resource Discovery
+       This setup allows you to run the routing layer of your Ember app in a server
     environment using Node.js and completely disable rendering. This allows you
     to simulate and discover the resources (i.e. AJAX requests) needed to fulfill
     a given request and eagerly "push" these resources to the client.
-     ```app/initializers/network-service.js
+       ```app/initializers/network-service.js
     import BrowserNetworkService from 'app/services/network/browser';
     import NodeNetworkService from 'app/services/network/node';
-     // Inject a (hypothetical) service for abstracting all AJAX calls and use
+       // Inject a (hypothetical) service for abstracting all AJAX calls and use
     // the appropriate implementation on the client/server. This also allows the
     // server to log all the AJAX calls made during a particular request and use
     // that for resource-discovery purpose.
-     export function initialize(application) {
+       export function initialize(application) {
       if (window) { // browser
         application.register('service:network', BrowserNetworkService);
       } else { // node
         application.register('service:network', NodeNetworkService);
       }
     };
-     export default {
+       export default {
       name: 'network-service',
       initialize: initialize
     };
     ```
-     ```app/routes/post.js
+       ```app/routes/post.js
     import Route from '@ember/routing/route';
     import { service } from '@ember/service';
-     // An example of how the (hypothetical) service is used in routes.
-     export default class IndexRoute extends Route {
+       // An example of how the (hypothetical) service is used in routes.
+       export default class IndexRoute extends Route {
       @service network;
-       model(params) {
+         model(params) {
         return this.network.fetch(`/api/posts/${params.post_id}.json`);
       }
-       afterModel(post) {
+         afterModel(post) {
         if (post.isExternalContent) {
           return this.network.fetch(`/api/external/?url=${post.externalURL}`);
         } else {
@@ -876,21 +803,23 @@ const Application = Engine.extend({
       }
     }
     ```
-     ```javascript
+       ```javascript
     // Finally, put all the pieces together
-     function discoverResourcesFor(url) {
+       function discoverResourcesFor(url) {
       return MyApp.visit(url, { isBrowser: false, shouldRender: false }).then(instance => {
         let networkService = instance.lookup('service:network');
         return networkService.requests; // => { "/api/posts/123.json": "..." }
       });
     }
     ```
-     @public
+       @public
     @method visit
     @param url {String} The initial URL to navigate to
     @param options {ApplicationInstance.BootOptions}
     @return {Promise<ApplicationInstance, Error>}
   */
+
+
   visit(url, options) {
     assert('You cannot visit this application since it has already been destroyed', !this.isDestroyed);
     assert('You cannot visit this application since it is being destroyed', !this.isDestroying);
@@ -903,39 +832,7 @@ const Application = Engine.extend({
     });
   }
 
-});
-Application.reopenClass({
-  /**
-    This creates a registry with the default Ember naming conventions.
-     It also configures the registry:
-     * registered views are created every time they are looked up (they are
-      not singletons)
-    * registered templates are not factories; the registered value is
-      returned directly.
-    * the router receives the application as its `namespace` property
-    * all controllers receive the router as their `target` and `controllers`
-      properties
-    * all controllers receive the application as their `namespace` property
-    * the application view receives the application controller as its
-      `controller` property
-    * the application view receives the application template as its
-      `defaultTemplate` property
-     @method buildRegistry
-    @static
-    @param {Application} namespace the application for which to
-      build the registry
-    @return {Ember.Registry} the built registry
-    @private
-  */
-  buildRegistry() {
-    let registry = this._super(...arguments);
-
-    commonSetupRegistry(registry);
-    setupApplicationRegistry(registry);
-    return registry;
-  }
-
-});
+}
 
 function commonSetupRegistry(registry) {
   registry.register('router:main', Router);