fix for static methods and names banned in strict mode

lib/src/utils.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.
 
 /// Holds a couple utility functions used at various places in the system.
 library dev_compiler.src.utils;
 
 import 'dart:io';
 
 import 'package:path/path.dart' as path;
 import 'package:analyzer/src/generated/ast.dart'
     show
         ImportDirective,
         ExportDirective,
         PartDirective,
         CompilationUnit,
         Identifier,
         AnnotatedNode,
         AstNode;
 import 'package:analyzer/src/generated/engine.dart'
     show ParseDartTask, AnalysisContext;
 import 'package:analyzer/src/generated/source.dart' show Source;
 import 'package:analyzer/src/generated/element.dart';
 import 'package:analyzer/analyzer.dart' show parseDirectives;
 import 'package:crypto/crypto.dart' show CryptoUtils, MD5;
 import 'package:source_span/source_span.dart';
 import 'package:yaml/yaml.dart';
 
-import 'js/keywords.dart';
+import 'codegen/js_names.dart' show invalidJSVariableName;
 
 bool isDartPrivateLibrary(LibraryElement library) {
   var uri = library.source.uri;
   if (uri.scheme != "dart") return false;
   return Identifier.isPrivateName(uri.path);
 }
 
 /// Choose a canonical name from the library element. This is safe to use as a
 /// namespace in JS and Dart code generation.  This never uses the library's
 /// name (the identifier in the `library` declaration) as it doesn't have any
@@ skipping 15 matching lines @@
     var needsEscape = ch == r'$' || _invalidCharInIdentifier.hasMatch(ch);
     if (needsEscape && buffer == null) {
       buffer = new StringBuffer(name.substring(0, i));
     }
     if (buffer != null) {
       buffer.write(needsEscape ? '\$${ch.codeUnits.join("")}' : ch);
     }
   }
 
   var result = buffer != null ? '$buffer' : name;
-  // Ensure the idenifier first character is not numeric and that the whole
+  // Ensure the identifier first character is not numeric and that the whole
   // identifier is not a keyword.

[Jacob, 2015/03/27 21:18:25]

tweak comment to 
is a valid JS variable name
instead of Keyword.

-  if (result.startsWith(new RegExp('[0-9]')) || isJsKeyword(result)) {
+  if (result.startsWith(new RegExp('[0-9]')) || invalidJSVariableName(result)) {
     return '\$$result';
   }
   return result;
 }
 
 // Invalid characters for identifiers, which would need to be escaped.
 final _invalidCharInIdentifier = new RegExp(r'[^A-Za-z_$0-9]');
 
 /// Returns all libraries transitively imported or exported from [start].
 Iterable<LibraryElement> reachableLibraries(LibraryElement start) {
@@ skipping 251 matching lines @@
   // TODO(jmesserly): can we implement this without repeatedly reading pubspec?
   // It seems like we should know our package's root directory without needing
   // to search like this.
   var pubspec =
       loadYaml(new File(path.join(dir, 'pubspec.yaml')).readAsStringSync());
 
   // Ensure this is loaded from the dev_compiler package.
   if (pubspec['name'] != 'dev_compiler') return null;
   return path.join('dev_compiler', 'runtime', filename);
 }