Ensure a class is finalized before attempting any reflective invocation on it.

runtime/lib/mirrors.cc

Index: runtime/lib/mirrors.cc
diff --git a/runtime/lib/mirrors.cc b/runtime/lib/mirrors.cc
index 3d228cdf4bb0eb05627855a8ff2437639644284d..486c33ed862453f77fa8d0b6b690b98913fca35a 100644
--- a/runtime/lib/mirrors.cc
+++ b/runtime/lib/mirrors.cc
@@ -332,6 +332,11 @@ static RawInstance* CreateClassMirror(const Class& cls,
     }
   }
 
+  const Error& error = Error::Handle(cls.EnsureIsFinalized(Isolate::Current()));
+  if (!error.IsNull()) {
+    ThrowInvokeError(error);

[zra, 2014/01/08 23:02:54]

After many other calls to ThrowInvokeError() there is an UNREACHABLE() assert.
Maybe add one for consistency's sake.

[rmacnak, 2014/01/09 00:17:58]

Done.

+  }
+
   const Bool& is_generic = Bool::Get(cls.NumTypeParameters() != 0);
   const Bool& is_mixin_app_alias = Bool::Get(cls.is_mixin_app_alias());