When generating snapshots for a specified script don't include the builtin library by default. Th...

bin/process_script.cc

 // Copyright (c) 2011, 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.
 
 // Handle dart scripts.
 
 #include <stdlib.h>
 #include <string.h>
 #include <stdio.h>
 
@@ skipping 68 matching lines @@
     return Dart_Error("Unable to fully read contents");
   }
   text_buffer[len] = '\0';
   delete file;
   Dart_Handle str = Dart_NewString(text_buffer);
   free(text_buffer);
   return str;
 }
 
 
-static Dart_Handle LibraryTagHandler(Dart_LibraryTag tag,
-                                     Dart_Handle library,
-                                     Dart_Handle url) {
+enum ImportBuiltin {

[Anton Muhin, 2011/10/31 17:48:34]

nit: I would rather have two bool constants for it, but that's pretty personal.

[siva, 2011/10/31 21:09:32]

Done.

+  kImportBuiltinLib = 0,  // Import builtin library into all added libs.
+  kDontImportBuiltinLib,  // Do not import builtin lib into the added libs.
+};
+
+
+static Dart_Handle LibraryTagHandlerHelper(Dart_LibraryTag tag,
+                                           Dart_Handle library,
+                                           Dart_Handle url,
+                                           ImportBuiltin import_builtin_lib) {
   if (!Dart_IsLibrary(library)) {
     return Dart_Error("not a library");
   }
   if (!Dart_IsString8(url)) {
     return Dart_Error("url is not a string");
   }
   const char* url_chars = NULL;
   Dart_Handle result = Dart_StringToCString(url, &url_chars);
   if (!Dart_IsValid(result)) {
     return Dart_Error("accessing url characters failed");
@@ skipping 23 matching lines @@
 
     return canon_url;
   }
 
   // The tag is either an import or a source tag. Read the file based on the
   // url chars.
   Dart_Handle source = ReadStringFromFile(url_chars);
   if (!Dart_IsValid(source)) {
     return result;
   }
-
   if (tag == kImportTag) {
     Dart_Handle new_lib = Dart_LoadLibrary(url, source);
-    if (Dart_IsValid(new_lib)) {
+    if ((import_builtin_lib == kImportBuiltinLib) && Dart_IsValid(new_lib)) {
       // TODO(iposva): Should the builtin library be added to all libraries?

[Anton Muhin, 2011/10/31 17:48:34]

may TODO go away or get moved into MainLibraryTagHandler?

[siva, 2011/10/31 21:09:32]

Done.

       Builtin_ImportLibrary(new_lib);
     }
     return result;
   } else if (tag == kSourceTag) {
     return Dart_LoadSource(library, url, source);
   }
   return Dart_Error("wrong tag");
 }
 
+static Dart_Handle MainLibraryTagHandler(Dart_LibraryTag tag,
+                                         Dart_Handle library,
+                                         Dart_Handle url) {
+  return LibraryTagHandlerHelper(tag, library, url, kImportBuiltinLib);
+}
+
+
+static Dart_Handle CreateSnapshotLibraryTagHandler(Dart_LibraryTag tag,
+                                                   Dart_Handle library,
+                                                   Dart_Handle url) {
+  return LibraryTagHandlerHelper(tag, library, url, kDontImportBuiltinLib);
+}
+
 
 Dart_Handle LoadScript(const char* script_name) {
   Dart_Handle source = ReadStringFromFile(script_name);
   if (!Dart_IsValid(source)) {
     return source;
   }
   Dart_Handle url = Dart_NewString(script_name);
 
-  return Dart_LoadScript(url, source, LibraryTagHandler);
+  return Dart_LoadScript(url, source, MainLibraryTagHandler);
 }
+
+
+Dart_Handle LoadSnapshotCreationScript(const char* script_name) {
+  Dart_Handle source = ReadStringFromFile(script_name);
+  if (!Dart_IsValid(source)) {
+    return source;
+  }
+  Dart_Handle url = Dart_NewString(script_name);
+
+  return Dart_LoadScript(url, source, CreateSnapshotLibraryTagHandler);
+}