Adding proper handling for MethodMirror.owner

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.
 
 // These values are allowed to be passed directly over the wire.
 bool _isSimpleValue(var value) {
   return (value == null || value is num || value is String || value is bool);
 }
@@ skipping 880 matching lines @@
   Symbol _qualifiedName = null;
   Symbol get qualifiedName {
     if (_qualifiedName == null) {
       _qualifiedName = _computeQualifiedName(owner, simpleName);
     }
     return _qualifiedName;
   }
 
   var _owner;
   DeclarationMirror get owner {
+    // For nested closures it is possible, that the mirror for the owner has not
+    // been created yet.
+    if (_owner == null) {
+      _owner = _MethodMirror_owner(_reflectee);
+    }
+    // TODO(mlippautz): This will go away, as soon as lazy mirrors go away.

[siva, 2013/07/18 20:32:34]

Do you have an issue ID for this?

[Michael Lippautz (Google), 2013/07/18 22:01:23]

Created www.dartbug.com/11897

     if (_owner is! Mirror) {
       _owner = _owner.resolve(mirrors);
     }
     return _owner;
   }
 
   bool get isPrivate {
     return _n(simpleName).startsWith('_') ||
         _n(constructorName).startsWith('_');
   }
@@ skipping 59 matching lines @@
     owner; // ensure owner is computed
     // get the metadata objects, convert them into InstanceMirrors using
     // reflect() and then make them into a Dart list
     return _metadata(this).map(reflect).toList();
   }
 
   String toString() => "MethodMirror on '${_n(simpleName)}'";
 
   static String _MethodMirror_name(reflectee)
       native "MethodMirror_name";
+
+  static dynamic _MethodMirror_owner(reflectee)
+      native "MethodMirror_owner";
 }
 
 class _LocalVariableMirrorImpl extends _LocalMirrorImpl
     implements VariableMirror {
   _LocalVariableMirrorImpl(String simpleName,
                            this._owner,
                            this._type,
                            this.isStatic,
                            this.isFinal)
       : this.simpleName = _s(simpleName);
@@ skipping 136 matching lines @@
   }
 
   // Creates a new local ClassMirror.
   static ClassMirror makeLocalClassMirror(Type key)
       native "Mirrors_makeLocalClassMirror";
 
   static ClassMirror reflectClass(Type key) {
     return makeLocalClassMirror(key);
   }
 }