Defer marking libraries as ‘loaded’

runtime/vm/dart_api_impl.cc

Index: runtime/vm/dart_api_impl.cc
===================================================================
--- runtime/vm/dart_api_impl.cc	(revision 37822)
+++ runtime/vm/dart_api_impl.cc	(working copy)
@@ -4748,14 +4748,9 @@
   const Error& error = Error::Handle(isolate, Compiler::Compile(lib, script));
   if (error.IsNull()) {
     *result = Api::NewHandle(isolate, lib.raw());
-    if (update_lib_status) {
-      lib.SetLoaded();
-    }
   } else {
     *result = Api::NewHandle(isolate, error.raw());
-    if (update_lib_status) {
-      lib.SetLoadError();
-    }
+    lib.SetLoadError();
   }
 }
 
@@ -5151,6 +5146,11 @@
   DARTSCOPE(isolate);
   CHECK_CALLBACK_STATE(isolate);
 
+  isolate->DoneLoading();
+
+  // TODO(hausner): move the remaining code below (finalization and
+  // invoing of _completeDeferredLoads) into Isolate::DoneLoading().
+
   // Finalize all classes if needed.
   Dart_Handle state = Api::CheckIsolateState(isolate);
   if (::Dart_IsError(state)) {