Move global code for builtins into setup functions.

src/collection.js

Index: src/collection.js
diff --git a/src/collection.js b/src/collection.js
index c872efbb302c3d3d4a5417bb47534b2160ec1e50..c1b328c89367919f49529cb9dde1af8d4351ecc9 100644
--- a/src/collection.js
+++ b/src/collection.js
@@ -27,6 +27,10 @@
 
 "use strict";
 
+// This file relies on the fact that the following declaration has been made
+// in runtime.js:
+// var $Array = global.Array;
+
 var $Set = global.Set;
 var $Map = global.Map;
 var $WeakMap = global.WeakMap;
@@ -239,9 +243,10 @@ function WeakMapDelete(key) {
   return %WeakMapDelete(this, key);
 }
 
+
 // -------------------------------------------------------------------
 
-(function () {
+function SetUpCollections() {

[rossberg, 2013/04/11 12:02:29]

Actually, I think this should be split up into SetUpSet, SetUpMap, and
SetUpWeakMap, and moved closer to the respective methods, with separators like
in v8natives.

[Michael Starzinger, 2013/04/11 12:08:44]

Done.

   %CheckIsBootstrapping();
 
   // Set up the Set and Map constructor function.
@@ -284,4 +289,6 @@ function WeakMapDelete(key) {
     "has", WeakMapHas,
     "delete", WeakMapDelete
   ));
-})();
+}
+
+SetUpCollections();