Properly rename type variables.

tests/compiler/dart2js/unparser_test.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.
 
 #import('dart:uri');
 #import('parser_helper.dart');
 #import('mock_compiler.dart');
 #import("../../../lib/compiler/compiler.dart");
 #import("../../../lib/compiler/implementation/dart_backend/dart_backend.dart");
 #import("../../../lib/compiler/implementation/elements/elements.dart");
@@ skipping 495 matching lines @@
     'class A implements I{A(){}}'
     'class B{factory I(){}}'
     'interface p_I default p_B{p_I();}'
     'class p_A implements p_I{p_A(){}}'
     'class p_B{factory p_I(){}}'
     'main(){new p_I(); new p_A(); new I(); new A();}';
   testDart2DartWithLibrary(mainSrc, librarySrc,
       (String result) { Expect.equals(expectedResult, result); });
 }
 
+testTypeVariablesAreRenamed() {
+  // Somewhat a hack: we require all the references of the identifier
+  // to be renamed in the same way for the whole library. Hence
+  // if we have a class and type variable with the same name, they
+  // both should be renamed.
+  var librarySrc = '''
+#library('mylib');
+class T {}
+class B<T> {}
+class A<T> extends B<T> { T f; }
+''';
+  var mainSrc = '''
+#import('mylib.dart', prefix: 'mylib');
+class T {}
+class B<T> {}
+class A<T> extends B<T> { T f; }
+
+main() {
+  new A<int>().f;
+  new T();
+
+  new mylib.A<int>().f;
+  new mylib.T();
+}
+''';
+  var expectedResult =
+    'class T{}class B<T>{}class A<T> extends B<T>{T f;}'
+    'class p_T{}class p_B<p_T>{}class p_A<p_T> extends p_B<p_T>{p_T f;}'
+    'main(){new p_A<int>().f; new p_T(); new A<int>().f; new T();}';
+  testDart2DartWithLibrary(mainSrc, librarySrc,
+      (String result) { Expect.equals(expectedResult, result); });
+}
+
 main() {
   testSignedConstants();
   testGenericTypes();
   testForLoop();
   testEmptyList();
   testClosure();
   testIndexedOperatorDecl();
   testNativeMethods();
   testPrefixIncrements();
   testConstModifier();
@@ skipping 11 matching lines @@
   //testConflictSendsRename();
   testNoConflictSendsRename();
   testConflictLibraryClassRename();
   testDefaultClassWithArgs();
   testClassExtendsWithArgs();
   testStaticInvocation();
   testLibraryGetSet();
   testFieldTypeOutput();
   testDefaultClassNamePlaceholder();
   testFactoryRename();
+  testTypeVariablesAreRenamed();
 }