fix for static methods and names banned in strict mode

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 'package:dev_compiler/src/js/js_ast.dart';
-import 'package:dev_compiler/src/js/keywords.dart';
 
 /// Marker subclass for temporary variables.
 /// We treat these as being in a different scope from other identifiers, and
 /// rename them so they don't collide. See [JSNamer].
 // TODO(jmesserly): move into js_ast? add a boolean to Identifier?
 class JSTemporary extends Identifier {
   JSTemporary(String name) : super(name);
 }
 
 /// This class has two purposes:
@@ skipping 15 matching lines @@
   final renames = new Map<String, String>();
 
   JSNamer(Node node) {
     node.accept(new _NameCollector(usedNames));
   }
 
   String getName(Identifier node) {
     var name = node.name;
     if (node is JSTemporary) {
       return _rename(name, valid: true);
-    } else if (isJsKeyword(name)) {
+    } else if (invalidJSVariableName(name)) {
       return _rename(name, valid: false);
     }
     return name;
   }
 
   String _rename(String name, {bool valid}) {
     var candidate = renames[name];
     if (candidate != null) return candidate;
 
     // Try to use the temp's name, otherwise rename.
@@ skipping 17 matching lines @@
 }
 
 /// Collects all names used in the visited tree.
 class _NameCollector extends BaseVisitor {
   final Set<String> names;
   _NameCollector(this.names);
   visitIdentifier(Identifier node) {
     if (node is! JSTemporary) names.add(node.name);
   }
 }
+
+/// Returns true for invalid JS variable names, such as keywords.
+/// Also handles invalid variable names in strict mode, like "arguments".
+bool invalidJSVariableName(String keyword, {bool strictMode: true}) {
+  switch (keyword) {
+    case "break":
+    case "case":
+    case "catch":
+    case "class":
+    case "const":
+    case "continue":
+    case "debugger":
+    case "default":
+    case "delete":
+    case "do":
+    case "else":
+    case "export":
+    case "extends":
+    case "finally":
+    case "for":
+    case "function":
+    case "if":
+    case "import":
+    case "in":
+    case "instanceof":
+    case "let":
+    case "new":
+    case "return":
+    case "static":
+    case "super":
+    case "switch":
+    case "this":
+    case "throw":
+    case "try":
+    case "typeof":
+    case "var":
+    case "void":
+    case "while":
+    case "with":
+    case "yield":
+      return true;
+    case "arguments":
+    case "eval":
+      return strictMode;
+  }
+  return false;
+}
+
+/// Returns true for invalid static method names in strict mode.
+/// In particular, "caller" "callee" and "arguments" cannot be used.
+bool invalidJSStaticMethodName(String name) {
+  switch (name) {
+    case "arguments":
+    case "caller":
+    case "callee":
+      return true;
+  }
+  return false;
+}