Open new handle scope when deep copying.

src/objects.cc

Index: src/objects.cc
diff --git a/src/objects.cc b/src/objects.cc
index 316b5839f4443c6e6122c5f1a875e39905d135be..513a821a6d2629051622839b3b08f6503acd7bd0 100644
--- a/src/objects.cc
+++ b/src/objects.cc
@@ -5623,6 +5623,7 @@ Handle<JSObject> JSObject::Copy(Handle<JSObject> object) {
 
 Handle<JSObject> JSObject::DeepCopy(Handle<JSObject> object) {
   Isolate* isolate = object->GetIsolate();
+  HandleScope handle_scope(isolate);

[Michael Starzinger, 2013/09/17 12:21:01]

nit: You could move the handle scope down to right before "copy" is allocated in
line 5637. Also s/handle_scope/scope/ would be more consistent with the rest of
the codebase.

[Yang, 2013/09/17 12:28:19]

Done.

   StackLimitCheck check(isolate);
   if (check.HasOverflowed()) {
     isolate->StackOverflow();
@@ -5744,7 +5745,7 @@ Handle<JSObject> JSObject::DeepCopy(Handle<JSObject> object) {
       // No contained objects, nothing to do.
       break;
   }
-  return copy;
+  return handle_scope.CloseAndEscape(copy);
 }