fuse List and js Array together and a few other misc fixes.

lib/src/codegen/js_codegen.dart

Index: lib/src/codegen/js_codegen.dart
diff --git a/lib/src/codegen/js_codegen.dart b/lib/src/codegen/js_codegen.dart
index e99a23327fc4946cd455bafcd0f9a4b07fbbf9df..b90d850b68439399e6a0e6b49f43ad424b5b00d0 100644
--- a/lib/src/codegen/js_codegen.dart
+++ b/lib/src/codegen/js_codegen.dart
@@ -274,7 +274,7 @@ class JSCodegenVisitor extends GeneralizingAstVisitor with ConversionVisitor {
       _emitTypeName(node.element.type, lowerTypedef: true)
     ]);
 
-    return _finishClassDef(type, result);
+    return _finishClassDef(type, result, null);
   }
 
   @override
@@ -292,7 +292,7 @@ class JSCodegenVisitor extends GeneralizingAstVisitor with ConversionVisitor {
     var classDecl = new JS.ClassDeclaration(
         new JS.ClassExpression(new JS.Identifier(name), heritage, []));
 
-    return _finishClassDef(type, classDecl);
+    return _finishClassDef(type, classDecl, null);
   }
 
   JS.Statement _emitJsType(String dartClassName, DartObjectImpl jsName) {
@@ -317,6 +317,7 @@ class JSCodegenVisitor extends GeneralizingAstVisitor with ConversionVisitor {
     if (_pendingClasses.remove(classElem) == null) return null;
 
     var jsName = getAnnotationValue(node, _isJsNameAnnotation);
+
     if (jsName != null) return _emitJsType(node.name.name, jsName);
 
     var ctors = <ConstructorDeclaration>[];
@@ -336,7 +337,13 @@ class JSCodegenVisitor extends GeneralizingAstVisitor with ConversionVisitor {
     var body =
         _finishClassMembers(classElem, classExpr, ctors, fields, staticFields);
 
-    return _finishClassDef(type, body);
+    var jsPeer = getAnnotationValue(node, _isJsPeerInterface);
+    String jsPeerName = null;
+    if (jsPeer != null ) {
+      jsPeerName = getConstantField(jsPeer, 'name', types.stringType);
+    }
+
+    return _finishClassDef(type, body, jsPeerName);
   }
 
   @override
@@ -346,7 +353,7 @@ class JSCodegenVisitor extends GeneralizingAstVisitor with ConversionVisitor {
   /// Given a class element and body, complete the class declaration.
   /// This handles generic type parameters, laziness (in library-cycle cases),
   /// and ensuring dependencies are loaded first.
-  JS.Statement _finishClassDef(ParameterizedType type, JS.Statement body) {
+  JS.Statement _finishClassDef(ParameterizedType type, JS.Statement body, String jsPeerName) {

[Jennifer Messerly, 2015/04/22 21:14:18]

long line?

[Jacob, 2015/04/22 22:58:32]

Done.

     var name = type.name;
     var genericName = '$name\$';
 
@@ -405,6 +412,11 @@ class JSCodegenVisitor extends GeneralizingAstVisitor with ConversionVisitor {
       _emitClassIfNeeded(classDefs, types.functionType);
     }
     classDefs.add(body);
+    if (jsPeerName != null) {
+      classDefs.add(js.statement('dart.copyProperties(window.#.prototype, #.prototype);', [

[Jennifer Messerly, 2015/04/22 21:14:18]

`window` seems dubious :)

a few other ideas:
1. capture up front into temp, e.g. `let JsString = String;`
2. rename our type locally: `class DartString { ... }`
3. save the JS type in dart_runtime.js and use: `dart.JsString` ... we already
do this for Symbol

[Jacob, 2015/04/22 22:58:32]

done.
added
dart.global = window || global;
to work with node and web.

+      _propertyName(jsPeerName),
+      name])); // XXX Array-->VAL;

[Jennifer Messerly, 2015/04/22 21:14:18]

remove?

[Jacob, 2015/04/22 22:58:32]

removed. any XXX comments will be removed before commit.

+    }
     return _statement(classDefs);
   }
 
@@ -2333,6 +2345,7 @@ class JSCodegenVisitor extends GeneralizingAstVisitor with ConversionVisitor {
         _pendingSymbols.add(new JS.Identifier(extName));
         _addExport(extName);
       }
+      return new JS.Identifier(extName);

[Jennifer Messerly, 2015/04/22 21:14:18]

ah, nice fix.

[Jacob, 2015/04/22 22:58:32]

Acknowledged.

     }
     return js.call('#.#', [_libraryName(library), _propertyName(extName)]);
   }
@@ -2425,6 +2438,8 @@ String jsOutputPath(LibraryInfo info, Uri root) {
 // TODO(jmesserly): validate the library. See issue #135.
 bool _isJsNameAnnotation(DartObjectImpl value) => value.type.name == 'JsName';
 
+bool _isJsPeerInterface(DartObjectImpl value) => value.type.name == 'JsPeerInterface';
+
 // TODO(jacobr): we would like to do something like the following
 // but we don't have summary support yet.
 // bool _supportJsExtensionMethod(AnnotatedNode node) =>