Make selector registration in the resolver and code generator more explicit.

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 17 matching lines @@
         usedGlobals = new Map<String, int>(),
         usedPrivateNames = new Map<String, Set<LibraryElement>>(),
         shortPrivateNameOwners = new Map<String, LibraryElement>();
 
   final String CURRENT_ISOLATE = @'$';
   final String ISOLATE = 'Isolate';
   final String ISOLATE_PROPERTIES = @"$isolateProperties";
   /** Some closures must contain their name. The name is stored in
     * [STATIC_CLOSURE_NAME_NAME]. */
   final String STATIC_CLOSURE_NAME_NAME = @'$name';
-  final SourceString CLOSURE_INVOCATION_NAME = Compiler.CALL_OPERATOR_NAME;
+  static final SourceString CLOSURE_INVOCATION_NAME =

[floitsch, 2012/08/15 14:14:04]

we should remove the CLOSURE_INVOCATION_NAME from the Namer and use the
CALL_OPERATOR_NAME directly.
Having statics in the namer is imho a bad thing. We want to be able to switch
it.

+      Compiler.CALL_OPERATOR_NAME;
 
 
   String closureInvocationName(Selector selector) {
     // TODO(floitsch): mangle, while not conflicting with instance names.
     return instanceMethodInvocationName(null, CLOSURE_INVOCATION_NAME,
                                         selector);
   }
 
   String breakLabelName(LabelElement label) {
     return '\$${label.labelName}\$${label.target.nestingLevel}';
@@ skipping 252 matching lines @@
   }
 
   String safeName(String name) {
     if (jsReserved.contains(name) || name.startsWith('\$')) {
       name = "\$$name";
       assert(!jsReserved.contains(name));
     }
     return name;
   }
 }