Support typeVariables in ClassMirror.

sdk/lib/_internal/compiler/implementation/js_backend/emitter.dart

 // Copyright (c) 2012, 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.
 
 part of js_backend;
 
 /// Enables debugging of fast/slow objects using V8-specific primitives.
 const DEBUG_FAST_OBJECTS = false;
 
 /**
@@ skipping 1992 matching lines @@
 
   void emitClassBuilderWithReflectionData(String className,
                                           ClassElement classElement,
                                           ClassBuilder builder,
                                           CodeBuffer buffer) {
     var metadata = buildMetadataFunction(classElement);
     if (metadata != null) {
       builder.addProperty("@", metadata);
     }
 
+    if (backend.isNeededForReflection(classElement)) {
+      List typeVars = [];
+      for (TypeVariableType typeVar in classElement.typeVariables) {
+        typeVars.add(js.string(typeVar.name.slowToString()));
+        typeVars.add(js.toExpression(reifyType(typeVar.element.bound)));
+      }
+
+      var superclass = classElement.superclass;

[karlklose, 2013/09/23 08:50:34]

Please add a type annotation.

[zarah, 2013/09/23 11:17:02]

Done.

+      if ((!typeVars.isEmpty && superclass == null)

[karlklose, 2013/09/23 08:50:34]

You could save the result of this check in a local to make the tests shorter.

[zarah, 2013/09/23 11:17:02]

Done.

+          || (superclass != null
+              && superclass.typeVariables != classElement.typeVariables)) {

[karlklose, 2013/09/23 08:50:34]

Consider storing classElement.typeVariables in a local and shere it with line
2015. Perhaps that would allow to write this boolean expression in one line.

[zarah, 2013/09/23 11:17:02]

Done.

+        builder.addProperty('<>', new jsAst.ArrayInitializer.from(typeVars));

[karlklose, 2013/09/23 08:50:34]

We should move the '<>' into the namer (not this CL).

[zarah, 2013/09/23 11:17:02]

OK.

+      }
+    }
     List<CodeBuffer> classBuffers = elementBuffers[classElement];
     if (classBuffers == null) {
       classBuffers = [];
     } else {
       elementBuffers.remove(classElement);
     }
     CodeBuffer statics = new CodeBuffer();
     statics.write('{$n');
     bool hasStatics = false;
     ClassBuilder staticsBuilder = new ClassBuilder();
@@ skipping 2284 matching lines @@
 
 const String HOOKS_API_USAGE = """
 // The code supports the following hooks:
 // dartPrint(message)   - if this function is defined it is called
 //                        instead of the Dart [print] method.
 // dartMainRunner(main) - if this function is defined, the Dart [main]
 //                        method will not be invoked directly.
 //                        Instead, a closure that will invoke [main] is
 //                        passed to [dartMainRunner].
 """;