dart2js: Move parameterStub generation to parameter_stub_generator and add parameter stubs to model.

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 315 matching lines @@
 
   Method(this.element, this.name, this.code);
 }
 
 /**
  * A method that corresponds to a method in the original Dart program.
  */
 class DartMethod extends Method {
   final bool needsTearOff;
   final String tearOffName;
-  // TODO(herhut): Directly store stubs instead/
-  final bool needsStubs;
+  final List<ParameterStubMethod> parameterStubs;
   // TODO(herhut): Directly store aliases instead.
   final bool canBeApplied;
   final bool canBeReflected;
 
   DartMethod(Element element, String name, js.Expression code,
-      {this.needsTearOff, this.tearOffName, this.needsStubs, this.canBeApplied,
-       this.canBeReflected})
+      {this.needsTearOff, this.tearOffName, this.parameterStubs,
+       this.canBeApplied, this.canBeReflected})
       : super(element, name, code) {
     assert(needsTearOff != null);
     assert(!needsTearOff || tearOffName != null);
     assert(canBeApplied != null);
+    assert(parameterStubs != null);
     assert(canBeReflected != null);
-    assert(needsStubs != null);
   }
 }
 
 class InstanceMethod extends DartMethod {
   // TODO(herhut): Directly store aliases instead.
   final bool hasSuperAlias;
   final bool isClosure;
 
   InstanceMethod(element, name, code,
       {bool needsTearOff,
        String tearOffName,
        this.hasSuperAlias,
        bool canBeApplied,
        bool canBeReflected,
        this.isClosure,
-       bool needsStubs})
+       List<ParameterStubMethod> parameterStubs})
       : super(element, name, code,
               needsTearOff: needsTearOff,
               tearOffName: tearOffName,
               canBeApplied: canBeApplied,
               canBeReflected: canBeReflected,
-              needsStubs: needsStubs) {
+              parameterStubs: parameterStubs) {

[herhut, 2015/01/30 10:02:22]

I would prefer this to be an unnamed parameter. They are mandatory and at the
call site it is clear what is being passed (other than for a boolean flag).

[zarah, 2015/01/30 12:46:38]

Done.

     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.
  */
 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

[floitsch, 2015/01/30 12:26:30]

For new code please prefer "///" comments.

[zarah, 2015/01/30 12:46:38]

Done.

+ * 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);
+ */
+class ParameterStubMethod extends StubMethod {
+  final Selector selector;
+  ParameterStubMethod(String name, js.Expression code, this.selector,
+                     {Element element})
+       : super(name, code, element: element);
+}
+
 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,
                    {bool needsTearOff, String tearOffName, bool canBeApplied,
-                    bool canBeReflected, bool needsStubs})
+                    bool canBeReflected, List<StubMethod> parameterStubs})
       : super(element, name, code,
               needsTearOff: needsTearOff,
               tearOffName : tearOffName,
               canBeApplied : canBeApplied,
               canBeReflected : canBeReflected,
-              needsStubs : needsStubs);
+              parameterStubs : parameterStubs);

[herhut, 2015/01/30 10:02:22]

As above.

[zarah, 2015/01/30 12:46:38]

Done.

 }
 
 class StaticStubMethod extends StubMethod implements StaticMethod {
   Holder holder;
   StaticStubMethod(String name, this.holder, js.Expression code)
       : super(name, code);
 }