dart2js: Refactor how parameter stubs are handled.

pkg/compiler/lib/src/js_emitter/model.dart

 // Copyright (c) 2014, 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.
 
 library dart2js.new_js_emitter.model;
 
 import '../js/js.dart' as js show Expression;
 import '../constants/values.dart' show ConstantValue;
 
 import '../deferred_load.dart' show OutputUnit;
@@ skipping 309 matching lines @@
 abstract class Method {
   /// The element should only be used during the transition to the new model.
   /// Uses indicate missing information in the model.
   final Element element;
   final String name;
   final js.Expression code;
 
   Method(this.element, this.name, this.code);
 }
 
-/**
- * A method that corresponds to a method in the original Dart program.
- */
+/// A method that corresponds to a method in the original Dart program.
 class DartMethod extends Method {
   final bool needsTearOff;
   final String tearOffName;
   final List<ParameterStubMethod> parameterStubs;
   // TODO(herhut): Directly store aliases instead.
   final bool canBeApplied;
   final bool canBeReflected;
 
+  // If this method can be torn off, contains the name of the corresponding
+  // call method. For example, for the member `foo$1$name` it would be
+  // `call$1$name` (in unminified mode).
+  final String callName;
+
   DartMethod(Element element, String name, js.Expression code,
-             this.parameterStubs,
+             this.parameterStubs, this.callName,
              {this.needsTearOff, this.tearOffName, this.canBeApplied,
              this.canBeReflected})
       : super(element, name, code) {
     assert(needsTearOff != null);
     assert(!needsTearOff || tearOffName != null);
     assert(canBeApplied != null);
-    assert(parameterStubs != null);
     assert(canBeReflected != null);
   }
 }
 
 class InstanceMethod extends DartMethod {
   // TODO(herhut): Directly store aliases instead.
   final bool hasSuperAlias;
   final bool isClosure;
 
-  InstanceMethod(element, name, code, List<ParameterStubMethod> parameterStubs,
-      {bool needsTearOff,
-       String tearOffName,
-       this.hasSuperAlias,
-       bool canBeApplied,
-       bool canBeReflected,
+  InstanceMethod(Element element, String name, js.Expression code,
+                 List<ParameterStubMethod> parameterStubs,
+                 String callName,
+                 {bool needsTearOff,
+                  String tearOffName,
+                  this.hasSuperAlias,
+                  bool canBeApplied,
+                  bool canBeReflected,
        this.isClosure})
-      : super(element, name, code, parameterStubs,
+      : super(element, name, code, parameterStubs, callName,
               needsTearOff: needsTearOff,
               tearOffName: tearOffName,
               canBeApplied: canBeApplied,
               canBeReflected: canBeReflected) {
     assert(hasSuperAlias != null);
     assert(isClosure != null);
   }
 }
 
-/**
- * A method that is generated by the backend and has not direct correspondence
- * to a method in the original Dart program. Examples are getter and setter
- * stubs and stubs to dispatch calls to methods with optional parameters.
- */
+/// A method that is generated by the backend and has not direct correspondence
+/// to a method in the original Dart program. Examples are getter and setter
+/// stubs and stubs to dispatch calls to methods with optional parameters.
 class StubMethod extends Method {
   StubMethod(String name, js.Expression code,
              {Element element})
       : super(element, name, code);
 }
 
- /// A method that is generated for the different versions of method calls of
- /// methods with named parameters,
- ///
- /// For example, for a method foo(a, b, {c, d}) that is called as
- /// foo(1, 2, c: 3), we have the stub
- /// foo$3$c(a, b, c) => foo$4$c$d(a, b, c, null);
+/// A stub that adapts and redirects to the main method (the one containing)
+/// the actual code.
+///
+/// For example, given a method `foo$2(x, [y: 499])` a possible parameter
+/// stub-method could be `foo$1(x) => foo$2(x, 499)`.
+///
+/// ParameterStubMethods are always attached to (static or instance) methods.
 class ParameterStubMethod extends StubMethod {
-  final Selector selector;
-  ParameterStubMethod(String name, js.Expression code, this.selector,
-                     {Element element})
-       : super(name, code, element: element);
+  /// The `call` name of this stub.
+  ///
+  /// When an instance method is torn off, it is invoked as a `call` member and
+  /// not it's original name anymore. The [callName] provides the stub's
+  /// name when it is used this way.
+  ///
+  /// If a stub's member can not be torn off, the [callName] is `null`.
+  String callName;
+
+  ParameterStubMethod(String name, this.callName, js.Expression code)
+      : super(name, code);
 }
 
 abstract class StaticMethod implements Method {
   Holder get holder;
 }
 
 class StaticDartMethod extends DartMethod implements StaticMethod {
   final Holder holder;
 
   StaticDartMethod(Element element, String name, this.holder,
-                   js.Expression code, parameterStubs,
+                   js.Expression code, List<ParameterStubMethod> parameterStubs,
+                   String callName,
                    {bool needsTearOff, String tearOffName, bool canBeApplied,
                     bool canBeReflected})
-      : super(element, name, code, parameterStubs,
+      : super(element, name, code, parameterStubs, callName,
               needsTearOff: needsTearOff,
               tearOffName : tearOffName,
               canBeApplied : canBeApplied,
               canBeReflected : canBeReflected);
 }
 
 class StaticStubMethod extends StubMethod implements StaticMethod {
   Holder holder;
   StaticStubMethod(String name, this.holder, js.Expression code)
       : super(name, code);
 }