Fix ParameterMirror.toString and FunctionTypeMirror.toString.

runtime/lib/mirrors_impl.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.
 
 // VM-specific implementation of the dart:mirrors library.
 
 import "dart:collection";
 
 // These values are allowed to be passed directly over the wire.
 bool _isSimpleValue(var value) {
@@ skipping 31 matching lines @@
 Map<Symbol, dynamic> _convertStringToSymbolMap(Map<String, dynamic> map) {
   if (map == null) return null;
   Map<Symbol, dynamic> result = new Map<Symbol, dynamic>();
   map.forEach((name, value) => result[_s(name)] = value);
   return result;
 }
 
 String _makeSignatureString(TypeMirror returnType,
                             List<ParameterMirror> parameters) {
   StringBuffer buf = new StringBuffer();
-  buf.write(_n(returnType.qualifiedName));
-  buf.write(' (');
-  bool found_optional_param = false;
+  buf.write('(');
+  bool found_optional_positional = false;
+  bool found_optional_named = false;
+
   for (int i = 0; i < parameters.length; i++) {
     var param = parameters[i];
-    if (param.isOptional && !found_optional_param) {
+    if (param.isOptional && param.isNamed && !found_optional_named) {
+      buf.write('{');
+      found_optional_named = true;
+    }
+    if (param.isOptional && !param.isNamed && !found_optional_positional) {
       buf.write('[');
-      found_optional_param = true;
+      found_optional_positional = true;
+    }
+    if (param.isNamed) {
+      buf.write(_n(param.simpleName));
+      buf.write(': ');
     }
     buf.write(_n(param.type.qualifiedName));
     if (i < (parameters.length - 1)) {
       buf.write(', ');
     }
   }
-  if (found_optional_param) {
+  if (found_optional_named) {
+    buf.write('}');
+  }
+  if (found_optional_positional) {
     buf.write(']');
   }
-  buf.write(')');
+  buf.write(') -> ');
+  buf.write(_n(returnType.qualifiedName));
   return buf.toString();
 }
 
 Map<Uri, LibraryMirror> _createLibrariesMap(List<LibraryMirror> list) {
   var map = new Map<Uri, LibraryMirror>();
   list.forEach((LibraryMirror mirror) => map[mirror.uri] = mirror);
   return map;
 }
 
 List _metadata(reflectee)
@@ skipping 1179 matching lines @@
 
   TypeMirror _type = null;
   TypeMirror get type {
     if (_type == null) {
       _type =
           _Mirrors._reflectType(_ParameterMirror_type(_reflectee, _position));
     }
     return _type;
   }
 
+  String toString() => "ParameterMirror on '${_n(simpleName)}'";
+
   static Type _ParameterMirror_type(_reflectee, _position)
       native "ParameterMirror_type";
 }
 
 class _SpecialTypeMirrorImpl extends _LocalMirrorImpl
     implements TypeMirror, DeclarationMirror {
   _SpecialTypeMirrorImpl(String name) : simpleName = _s(name);
 
   final bool isPrivate = false;
   final bool isTopLevel = true;
@@ skipping 92 matching lines @@
     if (typeMirror == null) {
       typeMirror = makeLocalTypeMirror(key);
       _instanitationCache[key] = typeMirror;
       if (typeMirror is ClassMirror && !typeMirror._isGeneric) {
         _declarationCache[key] = typeMirror;
       }
     }
     return typeMirror;
   }
 }