Removed a bunch of GetExistingThreadLocal calls by threading the Isolate.

src/factory.cc

Index: src/factory.cc
diff --git a/src/factory.cc b/src/factory.cc
index 279bd8f7ba3464eaf5b9ac901a75893bcdfa2893..71d412e13c6058e1504a88289cf2a4e6a833bd8b 100644
--- a/src/factory.cc
+++ b/src/factory.cc
@@ -1367,7 +1367,7 @@ Handle<Map> Factory::ObjectLiteralMapFromCache(Handle<Context> context,
   // Check to see whether there is a matching element in the cache.
   Handle<MapCache> cache =
       Handle<MapCache>(MapCache::cast(context->map_cache()));
-  Handle<Object> result = Handle<Object>(cache->Lookup(*keys));
+  Handle<Object> result = Handle<Object>(cache->Lookup(*keys), isolate());
   if (result->IsMap()) return Handle<Map>::cast(result);
   // Create a new map and add it to the cache.
   Handle<Map> map =
@@ -1419,7 +1419,8 @@ void Factory::ConfigureInstance(Handle<FunctionTemplateInfo> desc,
                                 bool* pending_exception) {
   // Configure the instance by adding the properties specified by the
   // instance template.
-  Handle<Object> instance_template = Handle<Object>(desc->instance_template());
+  Handle<Object> instance_template = Handle<Object>(desc->instance_template(),

[Michael Starzinger, 2012/11/15 21:24:25]

Use the handle constructor, that's easier to read I think.

Handle<Object> instance_template(desc->instance_template(), isolate());

[Sven Panne, 2012/11/16 08:24:37]

Done. In general I like the proposed direct initialization much more, too,
because it is conceptually simpler: It is similar to a normal function call with
the usual overloading resolution, just for constructors. The previous code uses
copy initialization, which is much more tricky: Conceptually it tries to create
a temporary of the type of the LHS (possibly involving conversion
functions/operators) and uses a copy constructor to move the values into the
LHS. Luckily enough, GCC is clever enough to avoid the intermediate steps, but
why should we make things more complicated than necessary? :-)

+                                                    isolate());
   if (!instance_template->IsUndefined()) {
     Execution::ConfigureInstance(instance,
                                  instance_template,