Adapt gen_snapshot to the recent changes in bootstrap sources loading scheme.

runtime/bin/gen_snapshot.cc

 // Copyright (c) 2013, 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.
 
 // Generate a snapshot file after loading all the scripts specified on the
 // command line.
 
 #include <stdlib.h>
 #include <string.h>
 #include <stdio.h>
@@ skipping 274 matching lines @@
     library_url_string = DartUtils::GetStringValue(library_url);
   }
 
   if (!Dart_IsString(url)) {
     return Dart_Error("url is not a string");
   }
   const char* url_string = DartUtils::GetStringValue(url);
   const char* mapped_url_string = DartUtils::MapLibraryUrl(url_mapping,
                                                            url_string);
 
+  Builtin::BuiltinLibraryId builtinLibraryId = Builtin::kInvalidLibrary;
+  if (DartUtils::IsDartBuiltinLibURL(library_url_string)) {
+    builtinLibraryId = Builtin::kBuiltinLibrary;
+  } else if (DartUtils::IsDartIOLibURL(library_url_string)) {
+    builtinLibraryId = Builtin::kIOLibrary;
+  }
+
   if (tag == kCanonicalizeUrl) {
     // Keep original names for libraries that have a mapping (e.g. dart:io).

[siva, 2013/05/21 16:45:37]

The comment (e.g: dart:io) seems wrong because you are handling that in the next
check.

[podivilov, 2013/05/22 08:54:44]

Done.

     if (mapped_url_string) {
       return url;
     }
+    // Parts of internal libraries are handled internally.
+    if (builtinLibraryId != Builtin::kInvalidLibrary) {
+      return url;
+    }
     return ResolveUri(library_url_string, url_string);
   }
 
-  if (DartUtils::IsDartIOLibURL(url_string) && mapped_url_string == NULL) {
-    // No url mapping for dart:io. Load original version.
+  if (DartUtils::IsDartBuiltinLibURL(url_string)) {
+    return Builtin::LoadAndCheckLibrary(Builtin::kBuiltinLibrary);

[siva, 2013/05/21 16:45:37]

You probably need a
if (tag == kImportTag) {
  return ....;
}
ASSERT(tag == kSourceTag);
return Dart_Error("...");
}

or atleast an assert
ASSERT(tag == kImportTag); 
here to ensure that you do not return a library if somebody
does
source "dart:builtin";

[podivilov, 2013/05/22 08:54:44]

Done.

+  }
+  if (DartUtils::IsDartIOLibURL(url_string)) {
     return Builtin::LoadAndCheckLibrary(Builtin::kIOLibrary);

[siva, 2013/05/21 16:45:37]

Ditto comment here.

[podivilov, 2013/05/22 08:54:44]

Done.

   }
 
+  if (builtinLibraryId != Builtin::kInvalidLibrary) {
+    if (tag == kSourceTag) {
+      return Dart_LoadSource(
+          library, url, Builtin::PartSource(builtinLibraryId, url_string));
+    }
+    ASSERT(tag == kImportTag);
+    return Dart_Error("Unable to import '%s' ", url_string);
+  }
+
   Dart_Handle resolved_url = url;
   if (mapped_url_string != NULL) {
     // Mapped urls are relative to working directory.
     resolved_url = ResolveScriptUri(mapped_url_string);
     if (Dart_IsError(resolved_url)) {
       return resolved_url;
     }
   }
 
   // Get the file path out of the url.
@@ skipping 217 matching lines @@
   }
   return 0;
 }
 
 }  // namespace bin
 }  // namespace dart
 
 int main(int argc, char** argv) {
   return dart::bin::main(argc, argv);
 }