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

pkg/compiler/lib/src/js_emitter/old_emitter/container_builder.dart

 // Copyright (c) 2013, 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.
 
 part of dart2js.js_emitter;
 
 /// This class should morph into something that makes it easy to build
 /// JavaScript representations of libraries, class-sides, and instance-sides.
 /// Initially, it is just a placeholder for code that is moved from
 /// [CodeEmitterTask].
@@ skipping 220 matching lines @@
         }
       }
     }
   }
 
   void addMemberMethod(DartMethod method, ClassBuilder builder) {
     final FunctionElement member = method.element;
     String name = method.name;
     final FunctionSignature parameters = member.functionSignature;
     jsAst.Expression code = method.code;
-    final bool needsStubs = method.needsStubs;
+    final bool needsStubs = !method.parameterStubs.isEmpty;
     final bool canTearOff = method.needsTearOff;
     final String tearOffName = method.tearOffName;
     final bool canBeReflected = method.canBeReflected;
     final bool canBeApplied = method.canBeApplied;
     final bool isClosure = method is InstanceMethod && method.isClosure;
     final bool hasSuperAlias = method is InstanceMethod && method.hasSuperAlias;
 
     final bool needStructuredInfo =
         canTearOff || canBeReflected || canBeApplied || hasSuperAlias;
 
     emitter.interceptorEmitter.recordMangledNameOfMemberMethod(member, name);
 
     if (!needStructuredInfo) {
       compiler.dumpInfoTask.registerElementAst(member,
           builder.addProperty(name, code));
-      if (needsStubs) {
-        addParameterStubs(
-            member,
-            (Selector selector, jsAst.Fun function) {
-              String invocationName = namer.invocationName(selector);
-              emitter.interceptorEmitter
-                  .recordMangledNameOfMemberMethod(member, invocationName);
-              compiler.dumpInfoTask.registerElementAst(member,
-                  builder.addProperty(invocationName, function));
-            });
+
+      for (ParameterStubMethod method in method.parameterStubs) {
+        jsAst.Property property = builder.addProperty(method.name, method.code);
+        compiler.dumpInfoTask.registerElementAst(member, property);
+        emitter.interceptorEmitter
+            .recordMangledNameOfMemberMethod(member, method.name);
       }
       return;
     }
     emitter.needsStructuredMemberInfo = true;
 
     // This element is needed for reflection or needs additional stubs or has a
     // super alias. So we need to retain additional information.
 
     // The information is stored in an array with this format:
     //
@@ skipping 52 matching lines @@
     int requiredParameterCount = parameters.requiredParameterCount << 1;
     if (member.isAccessor) requiredParameterCount++;
 
     int optionalParameterCount = parameters.optionalParameterCount << 1;
     if (parameters.optionalParametersAreNamed) optionalParameterCount++;
 
     // TODO(sra): Don't use LiteralString for non-strings.
     List tearOffInfo = [new jsAst.LiteralString(callSelectorString)];
 
     if (needsStubs || canTearOff) {
-      addParameterStubs(member, (Selector selector, jsAst.Fun function) {
 
-        String invocationName = namer.invocationName(selector);
+      for (ParameterStubMethod method in method.parameterStubs) {
+        String invocationName = method.name;
         emitter.interceptorEmitter.
-            recordMangledNameOfMemberMethod(member, invocationName);
-        expressions.add(function);
+                    recordMangledNameOfMemberMethod(member, invocationName);
+        expressions.add(method.code);
         if (member.isInstanceMember) {
-          Set invokedSelectors =
-              compiler.codegenWorld.invokedNames[member.name];
-            expressions.add(js.string(invocationName));
+          expressions.add(js.string(invocationName));
         } else {
           expressions.add(js('null'));
           // TOOD(ahe): Since we know when reading static data versus instance
           // data, we can eliminate this element.
         }
+
         Set<Selector> callSelectors = compiler.codegenWorld.invokedNames[
-            namer.closureInvocationSelectorName];
-        Selector callSelector = selector.toCallSelector();
+                   namer.closureInvocationSelectorName];
+        Selector callSelector = method.selector.toCallSelector();
         String callSelectorString = 'null';
         if (canTearOff && callSelectors != null &&
             callSelectors.contains(callSelector)) {
           callSelectorString = '"${namer.invocationName(callSelector)}"';
         }
         tearOffInfo.add(new jsAst.LiteralString(callSelectorString));
-      }, canTearOff);
+      }
     }
 
     jsAst.Expression memberTypeExpression;
     if (canTearOff || canBeReflected) {
       DartType memberType;
       if (member.isGenerativeConstructorBody) {
         var body = member;
         memberType = body.constructor.type;
       } else {
         memberType = member.type;
@@ skipping 60 matching lines @@
     jsAst.ArrayInitializer arrayInit =
       new jsAst.ArrayInitializer(expressions.toList());
     compiler.dumpInfoTask.registerElementAst(member,
         builder.addProperty(name, arrayInit));
   }
 
   void addMemberField(Field field, ClassBuilder builder) {
     // For now, do nothing.
   }
 }