Include named parameters in reflection names.

dart/sdk/lib/_internal/compiler/implementation/js_backend/emitter.dart

 // Copyright (c) 2012, 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 js_backend;
 
 /**
  * A function element that represents a closure call. The signature is copied
  * from the given element.
  */
@@ skipping 1210 matching lines @@
 
   String getReflectionNameInternal(elementOrSelector) {
     String name = elementOrSelector.name.slowToString();
     if (elementOrSelector.isGetter()) return name;
     if (elementOrSelector.isSetter()) return '$name=';
     if (elementOrSelector is Selector
         || elementOrSelector.isFunction()
         || elementOrSelector.isConstructor()) {
       int requiredParameterCount;
       int optionalParameterCount;
+      String namedArguments = '';
       bool isConstructor;
       if (elementOrSelector is Selector) {
-        requiredParameterCount = elementOrSelector.argumentCount;
+        Selector selector = elementOrSelector;
+        requiredParameterCount = selector.argumentCount;
         optionalParameterCount = 0;
         isConstructor = false;
+        namedArguments = namedParametersAsReflectionNames(selector);
       } else {
         FunctionElement function = elementOrSelector;
         requiredParameterCount = function.requiredParameterCount(compiler);
         optionalParameterCount = function.optionalParameterCount(compiler);
         isConstructor = function.isConstructor();
+        FunctionSignature signature = function.computeSignature(compiler);
+        if (signature.optionalParametersAreNamed) {
+          Selector selector = new Selector.call(
+              function.name,
+              function.getLibrary(),
+              requiredParameterCount,
+              optionalParameters.map((e) => e.name).toList());
+          namedArguments = namedParametersAsReflectionNames(selector);
+        }
       }
-      String suffix = '$name:$requiredParameterCount:$optionalParameterCount';
+      String suffix =
+          '$name:$requiredParameterCount:$optionalParameterCount'
+          '$namedArguments';
       return (isConstructor) ? 'new $suffix' : suffix;
     }
     Element element = elementOrSelector;
     if (element.isGenerativeConstructorBody()) {
       return null;
     }
     throw compiler.internalErrorOnElement(
         element, 'Do not know how to reflect on this $element');
   }
 
+  String namedParametersAsReflectionNames(Selector selector) {
+    if (selector.orderedNamedArguments.isEmpty) return '';
+    String names =
+        selector.orderedNamedArguments.map((x) => x.slowToString()).join(':');
+    return ':$names';
+  }
+
   /**
    * Documentation wanted -- johnniwinther
    *
    * Invariant: [classElement] must be a declaration element.
    */
   void emitInstanceMembers(ClassElement classElement,
                            ClassBuilder builder) {
     assert(invariant(classElement, classElement.isDeclaration));
 
     void visitMember(ClassElement enclosing, Element member) {
@@ skipping 2189 matching lines @@
 
 const String HOOKS_API_USAGE = """
 // The code supports the following hooks:
 // dartPrint(message)   - if this function is defined it is called
 //                        instead of the Dart [print] method.
 // dartMainRunner(main) - if this function is defined, the Dart [main]
 //                        method will not be invoked directly.
 //                        Instead, a closure that will invoke [main] is
 //                        passed to [dartMainRunner].
 """;