Handlify JSObject::DeepCopy method.

src/runtime.cc

Index: src/runtime.cc
diff --git a/src/runtime.cc b/src/runtime.cc
index 10de6f9e5ec001c595a0b86632599f9be9a22074..e8e76f3a1390df954c7cb4af27c22f2260f15b80 100644
--- a/src/runtime.cc
+++ b/src/runtime.cc
@@ -500,7 +500,7 @@ RUNTIME_FUNCTION(MaybeObject*, Runtime_CreateObjectLiteral) {
     // Update the functions literal and return the boilerplate.
     literals->set(literals_index, *boilerplate);
   }
-  return JSObject::cast(*boilerplate)->DeepCopy(isolate);
+  return *JSObject::DeepCopy(Handle<JSObject>::cast(boilerplate));

[Toon Verwaest, 2013/08/20 09:37:13]

As discussed, here and everywhere else, we need an empty-handle check; since the
stack check in DeepCopy can return Handle<Object>::null().

[Michael Starzinger, 2013/08/29 19:18:27]

Done.

 }
 
 
@@ -565,8 +565,8 @@ RUNTIME_FUNCTION(MaybeObject*, Runtime_CreateArrayLiteral) {
       literals_index, elements);
   RETURN_IF_EMPTY_HANDLE(isolate, site);
 
-  JSObject* boilerplate = JSObject::cast(site->transition_info());
-  return boilerplate->DeepCopy(isolate);
+  Handle<JSObject> boilerplate(JSObject::cast(site->transition_info()));
+  return *JSObject::DeepCopy(boilerplate);
 }