Use shorter names for constructor arguments.

lib/compiler/implementation/namer.dart

 // Copyright (c) 2011, 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.
 
 /**
  * Assigns JavaScript identifiers to Dart variables, class-names and members.
  */
 class Namer {
   final Compiler compiler;
 
@@ skipping 45 matching lines @@
     StringBuffer buffer = new StringBuffer();
     List<SourceString> names = selector.getOrderedNamedArguments();
     for (SourceString argumentName in names) {
       buffer.add(@'$');
       argumentName.printOn(buffer);
     }
     return '${privateName(lib, name)}\$${selector.argumentCount}$buffer';
   }
 
   String instanceFieldName(LibraryElement lib, SourceString name) {
-    return privateName(lib, name);
+    String proposedName = privateName(lib, name);
+    if (jsReserved.contains(proposedName)) {
+      return '$proposedName\$';

[ngeoffray, 2012/04/23 18:38:15]

This code very much looks like JsName.getValid, should you use that instead?

[floitsch, 2012/04/25 08:42:24]

Thanks. Addressed in CL 10223002.

+    }
+    return proposedName;
   }
 
   String setterName(LibraryElement lib, SourceString name) {
     return 'set\$${privateName(lib, name)}';
   }
 
   String getterName(LibraryElement lib, SourceString name) {
     return 'get\$${privateName(lib, name)}';
   }
 
@@ skipping 128 matching lines @@
   }
 
   String safeName(String name) {
     if (jsReserved.contains(name) || name.startsWith('\$')) {
       name = "\$$name";
       assert(!jsReserved.contains(name));
     }
     return name;
   }
 }