Revert "Emit named parameter information in declaration order."

sdk/lib/_internal/compiler/implementation/js_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 407 matching lines @@
     expressions.add(code);
 
     List tearOffInfo = [new jsAst.LiteralString(callSelectorString)];
 
     if (needsStubs || canTearOff) {
       addParameterStubs(member, (Selector selector, jsAst.Fun function) {
         expressions.add(function);
         if (member.isInstanceMember()) {
           Set invokedSelectors =
               compiler.codegenWorld.invokedNames[member.name];
+          //if (invokedSelectors != null && invokedSelectors.contains(selector)) {
             expressions.add(js.string(namer.invocationName(selector)));
+          //} else {
+          //  // Don't add a stub for calling this as a regular instance method,
+          //  // we only need the "call" stub for implicit closures of this
+          //  // method.
+          //  expressions.add("null");
+          //}
         } else {
+          // Static methods don't need "named" stubs as the default arguments
+          // are inlined at call sites. But static methods might need "call"
+          // stubs for implicit closures.
           expressions.add("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();
         String callSelectorString = 'null';
         if (canTearOff && callSelectors != null &&
             callSelectors.contains(callSelector)) {
@@ skipping 27 matching lines @@
     expressions
         ..addAll(tearOffInfo)
         ..add((tearOffName == null || member.isAccessor())
               ? js("null") : js.string(tearOffName))
         ..add(requiredParameterCount)
         ..add(optionalParameterCount)
         ..add(memberTypeExpression)
         ..addAll(task.metadataEmitter.reifyDefaultArguments(member));
 
     if (canBeReflected || canBeApplied) {
-      parameters.forEachParameter((Element parameter) {
+      parameters.orderedForEachParameter((Element parameter) {
         expressions.add(task.metadataEmitter.reifyName(parameter.name));
         List<MetadataAnnotation> annotations = parameter.metadata.toList();
         Iterable<int> metadataIndices = annotations.map((MetadataAnnotation a) {
           compiler.constantHandler.addCompileTimeConstantForEmission(a.value);
           return task.metadataEmitter.reifyMetadata(a);
         });
         expressions.add(metadataIndices.isNotEmpty ? metadataIndices.toList()
                                                    : js('[]'));
       });
     }
@@ skipping 14 matching lines @@
       expressions.add(js.string(member.name));
     }
 
     builder.addProperty(name, js.toExpression(expressions));
   }
 
   void addMemberField(VariableElement member, ClassBuilder builder) {
     // For now, do nothing.
   }
 }