Add --modules=node support

lib/src/codegen/js_names.dart

 // 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.
 
 library dev_compiler.src.codegen.js_names;
 
 import 'dart:collection';
 
 import '../js/js_ast.dart';
 
@@ skipping 263 matching lines @@
     case "implements":
     case "interface":
     case "let":
     case "package":
     case "private":
     case "protected":
     case "public":
     case "static":
     case "yield":
       return strictMode;
+
+    // Node refuses these:

[Jennifer Messerly, 2016/01/29 00:32:31]

Any more info on these? What's the error message say, and under what conditions
do they not work?

it's a little scary, usually JS is careful to not take away names that worked,
it breaks backwards compat. Hence a lot of names are only blacklisted in "strict
mode"

(we try and be careful to only blacklist names when we absolutely need to.)

[ochafik, 2016/01/29 09:38:17]

Dropped these, not needed now...

+    case "async":

[Jennifer Messerly, 2016/01/29 00:32:31]

this could be related to https://tc39.github.io/ecmascript-asyncawait/, but I'd
be surprised if it isn't backwards compatible.

[ochafik, 2016/01/29 09:38:17]

My bad, now fixed (I had a weird collision of async lib import alias and async
function declaration, sometime before the exportName -> name fix)

+    case "dynamic":
+    case "void":

[Jennifer Messerly, 2016/01/29 00:32:31]

it looks like "void" is already covered, above? So this is a duplicate case

[ochafik, 2016/01/29 09:38:17]

Done.

+      return true;
   }
   return false;
 }
 
 /// Returns true for invalid static field names in strict mode.
 /// In particular, "caller" "callee" and "arguments" cannot be used.
 bool invalidStaticFieldName(String name) {
   switch (name) {
     case "arguments":
     case "caller":
     case "callee":
       return true;
   }
   return false;
 }