- Move the Timer interface to the dart:isolate library so that it

runtime/bin/main.cc

Index: runtime/bin/main.cc
===================================================================
--- runtime/bin/main.cc	(revision 8601)
+++ runtime/bin/main.cc	(working copy)
@@ -505,8 +505,10 @@
 
   if (snapshot_buffer != NULL) {
     // Setup the native resolver as the snapshot does not carry it.
-    Builtin::SetNativeResolver(Builtin::kBuiltinLibrary);
-    Builtin::SetNativeResolver(Builtin::kIOLibrary);
+    Builtin::SetupLibrary(Builtin::LoadLibrary(Builtin::kBuiltinLibrary),
+                          Builtin::kBuiltinLibrary);
+    Builtin::SetupLibrary(Builtin::LoadLibrary(Builtin::kIOLibrary),
+                          Builtin::kIOLibrary);

[siva, 2012/06/13 17:05:23]

I would prefer if we did a Dart_LookupLibrary here instead
of Builtin::LoadLibrary. We are expected to find the library here. If for some
reason there is a bug in the snapshot generation we want to assert out when the
library is not found instead of silently loading it.

[Ivan Posva, 2012/06/21 16:17:06]

Agreed, but this needs a whole restructuring of the Builtin loading code which
is not part of this change. Also you currently get an assert within
Builtin::LoadLibrary if the library is not found, even if this is not
necessarily the way you wanted to.

If we want to make this change then it should come together with a change to
reduce the standalone snapshot size and making some of the non-critical
libraries loadable from source.

   }
 
   // Set up the library tag handler for this isolate.
@@ -543,6 +545,10 @@
     return false;
   }
 
+  // Setup the IO library.
+  Dart_Handle io_lib = Builtin::LoadLibrary(Builtin::kIOLibrary);
+  Builtin::SetupIOLibrary(io_lib);
+
   if (package_root != NULL) {
     const int kNumArgs = 1;
     Dart_Handle dart_args[kNumArgs];