Restructured the set up for the compiling function. Also added room for list of additional summarie…

web/main.dart

 #!/usr/bin/env dart
 // Copyright (c) 2016, 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.
 
 /// Command line entry point for Dart Development Compiler (dartdevc).
 ///
 /// Supported commands are
 ///   * compile: builds a collection of dart libraries into a single JS module
 ///
@@ skipping 28 matching lines @@
 library dev_compiler.web.main;
 
 import 'dart:async';
 
 import 'package:args/command_runner.dart';
 import 'package:js/js.dart';
 
 import 'web_command.dart';
 
 @JS()
-external set compileDartExpression(Function function);
-
-typedef String CompileFn(String dart);
-typedef void OnLoadFn(CompileFn compile);
+external set setUp(Function function);
 
 Future main() async {
   var args = ['compile'];

[Jacob, 2016/08/03 23:58:31]

passing args and using command runner doesn't seem like it is doing anything
useful. am I missing something?

[priscillalee, 2016/08/04 16:01:56]

I looked into this myself earlier because I was wondering the same thing. I
think something happens behind the scenes with ArgResults or ArgParsers, and I
wasn't able to remove the CommandRunner or the args after a couple tries. I'm
also not quite sure how this works. Do you think it's worth it to continue
investigating?

[Jacob, 2016/08/04 16:27:48]

Just add a TODO indicating just that so the next person to look at this code
knows it is a known issue but fixing may be harder than it seems.

[priscillalee, 2016/08/04 17:56:33]

Done.

-  _runCommand((result) {
-    compileDartExpression = allowInterop(result);
-  }, args);
+  _runCommand(args);
 }
 
 /// Runs a single compile command, and returns an exit code.
-Future<int> _runCommand(OnLoadFn onload, List<String> args,
+Future<int> _runCommand(List<String> args,
     {MessageHandler messageHandler}) async {
   try {
     var runner = new CommandRunner('dartdevc', 'Dart Development Compiler');
-    runner
-        .addCommand(new WebCompileCommand(onload, messageHandler: messageHandler));
-    await runner.run(args);
+    runner.addCommand(new WebCompileCommand(messageHandler: messageHandler));
+    setUp = allowInterop(await runner.run(args));
   } catch (e, s) {
     return _handleError(e, s, args, messageHandler: messageHandler);
   }
   return 1;
 }
 
 /// Handles [error] in a uniform fashion. Returns the proper exit code and calls
 /// [messageHandler] with messages.
 int _handleError(dynamic error, dynamic stackTrace, List<String> args,
     {MessageHandler messageHandler}) {
@@ skipping 26 matching lines @@
     messageHandler("");
     messageHandler("    dartdevc arguments: " + args.join(' '));
     messageHandler("");
     messageHandler("```");
     messageHandler(error);
     messageHandler(stackTrace);
     messageHandler("```");
     return 70;
   }
 }