Change the return value of setField to be in line with non-reflective assignments. Complete unwrapp…

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 176 matching lines @@
                                 _n(memberName),
                                 positionalArguments.toList(growable:false))); 
   }
 
   InstanceMirror getField(Symbol memberName) {
     return reflect(this._invokeGetter(_reflectee,
                                       _n(memberName))); 
   }
 
   InstanceMirror setField(Symbol memberName, Object value) {
-    return reflect(this._invokeSetter(_reflectee,
-                                      _n(memberName),
-                                      value)); 
+    this._invokeSetter(_reflectee,
+                       _n(memberName),
+                       value);
+    return reflect(value);
   }
 
   Future<InstanceMirror> invokeAsync(Symbol memberName,
                                      List positionalArguments,
                                      [Map<Symbol, dynamic> namedArguments]) {
     if (namedArguments != null) {
       throw new UnimplementedError(
           'named argument support is not implemented');
     }
 
@@ skipping 22 matching lines @@
       var unwrappedValue;
       if(_isSimpleValue(value)) {
         unwrappedValue = value;
       } else if(value is InstanceMirror) {
         unwrappedValue = value._reflectee;
       } else { 
         throw "setter argument ($value) must be"
               "a simple value or InstanceMirror";
       }
 
-      var result = this._invokeSetter(_reflectee,
-                                      _n(memberName),
-                                      unwrappedValue);
-      return new Future.value(reflect(result)); 
+      this._invokeSetter(_reflectee,
+                         _n(memberName),
+                         unwrappedValue);
+      return new Future.value(reflect(unwrappedValue)); 
     } catch(e) {
       return new Future.error(e);
     }
   }
 
   static _validateArgument(int i, Object arg)
   {
     if (arg is Mirror) {
         if (arg is! InstanceMirror) {
           throw new MirrorException(
@@ skipping 946 matching lines @@
   static Expando<ClassMirror> _classMirrorCache = new Expando("ClassMirror");
   static ClassMirror reflectClass(Type key) {
     var classMirror = _classMirrorCache[key];
     if (classMirror == null) {
       classMirror = makeLocalClassMirror(key);
       _classMirrorCache[key] = classMirror;
     }
     return classMirror;
   }
 }