Fix runtime ordering and other minor fixes

lib/runtime/dart_runtime.js

 // Copyright (c) 2015, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 var dart, dartx;
 (function (dart) {
   'use strict';
 
   const defineProperty = Object.defineProperty;
   const getOwnPropertyDescriptor = Object.getOwnPropertyDescriptor;
@@ skipping 470 matching lines @@
       if (names.length == 0) return false;
       for (name of names) {
         if (!(Object.prototype.hasOwnProperty.call(this.named, name))) {
           return false;
         }
         if (!instanceOfOrNull(opts[name], this.named[name])) return false;
       }
       return true;
     }
 
+    toString() { return this.name; }
+
     get name() {
       if (this._stringValue) return this._stringValue;
 
       let buffer = '(';
       for (let i = 0; i < this.args.length; ++i) {
         if (i > 0) {
           buffer += ', ';
         }
         buffer += _typeName(this.args[i]);
       }
@@ skipping 787 matching lines @@
       this.dartIterator = dartIterator;
     }
     next() {
       let i = this.dartIterator;
       let done = !i.moveNext();
       return { done: done, value: done ? void 0 : i.current };
     }
   }
   dart.JsIterator = JsIterator;
 
-  // TODO(jmesserly): right now this is a sentinel. It should be a type object
-  // of some sort, assuming we keep around `dynamic` at runtime.
-  dart.dynamic = { toString() { return 'dynamic'; }, get name() {return toString();}};
-  dart.void = { toString() { return 'void'; }, get name() {return toString();}};
-  dart.bottom = { toString() { return 'bottom'; }, get name() {return toString();}};
+  // TODO(jmesserly/vsm): Right now these are sentinels. They should be type
+  // objects of some sort, assuming we keep them at runtime.
+  function _sentinel(_name) {
+    return {
+      toString() { return _name; },
+      get name() { return this.toString(); }
+    };
+  }
+  dart.dynamic = _sentinel('dynamic');
+  dart.void = _sentinel('void');
+  dart.bottom = _sentinel('bottom');
 
   dart.global = window || global;
   dart.JsSymbol = Symbol;
 
   // Module support.  This is a simplified module system for Dart.
   // Longer term, we can easily migrate to an existing JS module system:
   // ES6, AMD, RequireJS, ....
 
   class LibraryLoader {
     constructor(name, defaultValue, imports, lazyImports, loader) {
@@ skipping 69 matching lines @@
   }
   LibraryLoader.NOT_LOADED = 0;
   LibraryLoader.LOADING = 1;
   LibraryLoader.LOADED = 2;
   LibraryLoader.READY = 3;
 
   // Map from name to LibraryLoader
   let libraries = new Map();
 
   function library(name, defaultValue, imports, lazyImports, loader) {
-    libraries[name] =
+    return libraries[name] =
       new LibraryLoader(name, defaultValue, imports, lazyImports, loader);
   }
   dart.library = library;
 
   function import_(libraryName) {
     bootstrap();
     let loader = libraries[libraryName];
     return loader.load();
   }
   dart.import = import_;
 
   function start(libraryName) {
     let library = import_(libraryName);
     _isolate_helper.startRootIsolate(library.main, []);
   }
   dart.start = start;
 
   // Libraries used in this file.
   let core;
   let collection;
   let async;
   let _interceptors;
   let _isolate_helper;
   let _js_helper;
   let _js_primitives;
 
+  let _bootstrapped = false;
   function bootstrap() {
-    if (core) return;
+    if (_bootstrapped) return;
+    _bootstrapped = true;
 
+    // Setup stubs for top-level symbols.
     let lazyImport = (name) => libraries[name].stub();
-
     core = lazyImport('dart/core');
     collection = lazyImport('dart/collection');
     async = lazyImport('dart/async');
     _interceptors = lazyImport('dart/_interceptors');
     _isolate_helper = lazyImport('dart/_isolate_helper');
     _js_helper = lazyImport('dart/_js_helper');
     _js_helper.checkNum = notNull;
     _js_primitives = lazyImport('dart/_js_primitives');
     _js_primitives.printString = (s) => console.log(s);
 
+    // Create namespace for dart extension members.
+    dartx = dartx || {};
+
+    // Force import of core.
+    import_('dart/core');

[vsm, 2015/06/11 17:27:40]

Note, I need to do a lazy import above first to ensure that the 'core' variable
is defined in this file before doing the actual import.

The actual import here also pulls in _interceptors before dart.list above is
invoked.

[Leaf, 2015/06/11 18:05:03]

Just a note - I'm refactoring a lot of this following John's suggestion of using
the loader infrastructure for the pieces of the runtime.  You may want to go
ahead and land this, but I'll probably need your help understanding how to merge
it... :}

+
     // TODO(vsm): DOM facades?
     // See: https://github.com/dart-lang/dev_compiler/issues/173
     NodeList.prototype.get = function(i) { return this[i]; };
     NamedNodeMap.prototype.get = function(i) { return this[i]; };
     DOMTokenList.prototype.get = function(i) { return this[i]; };
+    HTMLCollection.prototype.get = function(i) { return this[i]; };
 
     // TODO(vsm): This is referenced (as init.globalState) from
     // isolate_helper.dart.  Where should it go?
     // See: https://github.com/dart-lang/dev_compiler/issues/164
     dart.globalState = null;
-
-    /** Dart extension members. */
-    dartx = dartx || {};
   }
 })(dart || (dart = {}));