Open new handle scope when deep copying.

Open new handle scope when deep copying.

R=mstarzinger@chromium.org
BUG=292903
Committed: https://code.google.com/p/v8/source/detail?r=16769

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

LGTM with a nit.

https://codereview.chromium.org/24201002/diff/1/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/24201002/diff/1/src/objects.cc#newcode5626
src/objects.cc:5626: HandleScope handle_scope(isolate);
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]

https://codereview.chromium.org/24201002/diff/1/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/24201002/diff/1/src/objects.cc#newcode5626
src/objects.cc:5626: HandleScope handle_scope(isolate);
On 2013/09/17 12:21:01, Michael Starzinger wrote:
> 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.

Done.

[Yang, 2013-09-17 12:28:42]

Committed patchset #2 manually as r16769 (presubmit successful).