Properly handle exceptions that arise when reflectively invoking a constructor/factory.

runtime/lib/mirrors.cc

 // 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.
 
 #include "lib/invocation_mirror.h"
 #include "vm/bootstrap_natives.h"
 #include "vm/class_finalizer.h"
 #include "vm/dart_entry.h"
 #include "vm/exceptions.h"
 #include "vm/object_store.h"
@@ skipping 1148 matching lines @@
     // Factories get type arguments.
     args.SetAt(0, type_arguments);
   }
 
   // Invoke the constructor and return the new object.
   const Object& result =
       Object::Handle(DartEntry::InvokeFunction(redirected_constructor,
                                                args,
                                                args_descriptor_array));
   if (result.IsError()) {
-    return result.raw();
+    ThrowInvokeError(Error::Cast(result));
+    UNREACHABLE();
   }
 
   // Factories may return null.
   ASSERT(result.IsInstance() || result.IsNull());
 
   if (redirected_constructor.IsConstructor()) {
     return new_object.raw();
   } else {
     return result.raw();
   }
@@ skipping 225 matching lines @@
 }
 
 
 DEFINE_NATIVE_ENTRY(VariableMirror_type, 1) {
   GET_NON_NULL_NATIVE_ARGUMENT(MirrorReference, ref, arguments->NativeArgAt(0));
   const Field& field = Field::Handle(ref.GetFieldReferent());
   return field.type();
 }
 
 }  // namespace dart