Add basic API support for Map & Set

src/bootstrapper.cc

Index: src/bootstrapper.cc
diff --git a/src/bootstrapper.cc b/src/bootstrapper.cc
index d93c465b47dd20b7e290bf8865b747d4dcb88469..597c1d1a53565f2c4316fb26dc0142d15c3bebdf 100644
--- a/src/bootstrapper.cc
+++ b/src/bootstrapper.cc
@@ -1214,13 +1214,19 @@ void Genesis::InitializeGlobal(Handle<GlobalObject> global_object,
     native_context()->set_data_view_fun(*data_view_fun);
   }
 
-  // -- M a p
-  InstallFunction(global, "Map", JS_MAP_TYPE, JSMap::kSize,
-                  isolate->initial_object_prototype(), Builtins::kIllegal);
+  {  // -- M a p
+    Handle<JSFunction> js_map_fun = InstallFunction(
+        global, "Map", JS_MAP_TYPE, JSMap::kSize,
+        isolate->initial_object_prototype(), Builtins::kIllegal);
+    native_context()->set_js_map_map(js_map_fun->initial_map());

[jochen (gone - plz use gerrit), 2015/06/01 13:32:27]

this doesn't work :-/

the prototype of the Map() function is overwritten in collection.js which runs
after this code.

if you built without a snapshot and create a Map via the C++ API, the resulting
object's __proto__ will be Object.prototype

you should store the function in the native context, and read the initial map in
ConfigureGlobalObjects

[adamk, 2015/06/01 21:30:02]

Good eye, fix up for review at https://codereview.chromium.org/1161363002

+  }
 
-  // -- S e t
-  InstallFunction(global, "Set", JS_SET_TYPE, JSSet::kSize,
-                  isolate->initial_object_prototype(), Builtins::kIllegal);
+  {  // -- S e t
+    Handle<JSFunction> js_set_fun = InstallFunction(
+        global, "Set", JS_SET_TYPE, JSSet::kSize,
+        isolate->initial_object_prototype(), Builtins::kIllegal);
+    native_context()->set_js_set_map(js_set_fun->initial_map());
+  }
 
   {  // Set up the iterator result object
     STATIC_ASSERT(JSGeneratorObject::kResultPropertyCount == 2);