ElementsAccessor::CopyElements() and its callers handlified.

ElementsAccessor::CopyElements() and its callers handlified.

R=yangguo@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=20312

[Igor Sheludko, 2014-03-26 15:29:25]

PTAL

[Yang, 2014-03-27 15:39:22]

lgtm.

https://codereview.chromium.org/212573007/diff/1/src/builtins.cc
File src/builtins.cc (right):

https://codereview.chromium.org/212573007/diff/1/src/builtins.cc#newcode778
src/builtins.cc:778: handle(result_array->elements()), 0, result_len, elms);
CopyElements cannot throw, correct?

[Igor Sheludko, 2014-03-27 16:00:09]

https://codereview.chromium.org/212573007/diff/1/src/builtins.cc
File src/builtins.cc (right):

https://codereview.chromium.org/212573007/diff/1/src/builtins.cc#newcode778
src/builtins.cc:778: handle(result_array->elements()), 0, result_len, elms);
On 2014/03/27 15:39:22, Yang wrote:
> CopyElements cannot throw, correct?

Yes, it can't.

[Yang, 2014-03-27 16:01:20]

On 2014/03/27 16:00:09, Igor Sheludko wrote:
> https://codereview.chromium.org/212573007/diff/1/src/builtins.cc
> File src/builtins.cc (right):
> 
> https://codereview.chromium.org/212573007/diff/1/src/builtins.cc#newcode778
> src/builtins.cc:778: handle(result_array->elements()), 0, result_len, elms);
> On 2014/03/27 15:39:22, Yang wrote:
> > CopyElements cannot throw, correct?
> 
> Yes, it can't.

lgtm.

[Igor Sheludko, 2014-03-27 16:41:25]

Committed patchset #1 manually as r20312 (presubmit successful).