Fix runtime ordering and other minor fixes

Fix runtime ordering and other minor fixes

Other than mirrors (coming next :-)), this gets angular todo running.

R=leafp@google.com

Committed: https://github.com/dart-lang/dev_compiler/commit/cd399b17c24444fcaaf161b75aac9b24e67bae3b

[vsm, 2015-06-11 17:23:16]

Refactor

[vsm, 2015-06-11 17:27:32]

vsm@google.com changed reviewers:
+ jmesserly@google.com

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

https://codereview.chromium.org/1177253002/diff/20001/lib/runtime/dart_runtim...
File lib/runtime/dart_runtime.js (right):

https://codereview.chromium.org/1177253002/diff/20001/lib/runtime/dart_runtim...
lib/runtime/dart_runtime.js:1448: import_('dart/core');
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.

https://codereview.chromium.org/1177253002/diff/20001/lib/src/codegen/html_co...
File lib/src/codegen/html_codegen.dart (right):

https://codereview.chromium.org/1177253002/diff/20001/lib/src/codegen/html_co...
lib/src/codegen/html_codegen.dart:32: return '${document.outerHtml}\n';
Note, todo requires --resources="todo.html".  That file gets processed here, but
it only has angular templates, no dart code.

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

leafp@google.com changed reviewers:
+ leafp@google.com

[Leaf, 2015-06-11 18:05:04]

lgtm

https://codereview.chromium.org/1177253002/diff/20001/lib/runtime/dart_runtim...
File lib/runtime/dart_runtime.js (right):

https://codereview.chromium.org/1177253002/diff/20001/lib/runtime/dart_runtim...
lib/runtime/dart_runtime.js:1448: import_('dart/core');
On 2015/06/11 17:27:40, vsm wrote:
> 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.

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... :}

[vsm, 2015-06-11 18:07:31]

Committed patchset #2 (id:20001) manually as
cd399b17c24444fcaaf161b75aac9b24e67bae3b (presubmit successful).

[Jennifer Messerly, 2015-06-11 18:28:25]

lgtm too!

https://codereview.chromium.org/1177253002/diff/20001/lib/src/codegen/html_co...
File lib/src/codegen/html_codegen.dart (right):

https://codereview.chromium.org/1177253002/diff/20001/lib/src/codegen/html_co...
lib/src/codegen/html_codegen.dart:32: return '${document.outerHtml}\n';
On 2015/06/11 17:27:40, vsm wrote:
> Note, todo requires --resources="todo.html".  That file gets processed here,
but
> it only has angular templates, no dart code.

yeah, I'd be totally fine removing this warning. An HTML file is an HTML file
... if we need to serve it, we serve it. Not having a Dart script is not an
error in an HTML file :)