Callers of JSArray::SetContent() handlified.

Callers of JSArray::SetContent() handlified.

R=yangguo@chromium.org

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

[Igor Sheludko, 2014-03-20 12:47:54]

PTAL

[Yang, 2014-03-20 13:22:07]

LGTM with suggestion.

https://codereview.chromium.org/206383003/diff/1/src/runtime.cc
File src/runtime.cc (right):

https://codereview.chromium.org/206383003/diff/1/src/runtime.cc#newcode13101
src/runtime.cc:13101:
isolate->factory()->SetContent(Handle<JSArray>::cast(result), instances);
for the sake of consistency, could we move SetContent to a static
JSArray::SetContent?

[Igor Sheludko, 2014-03-20 13:28:13]

https://codereview.chromium.org/206383003/diff/1/src/runtime.cc
File src/runtime.cc (right):

https://codereview.chromium.org/206383003/diff/1/src/runtime.cc#newcode13101
src/runtime.cc:13101:
isolate->factory()->SetContent(Handle<JSArray>::cast(result), instances);
On 2014/03/20 13:22:07, Yang wrote:
> for the sake of consistency, could we move SetContent to a static
> JSArray::SetContent?

It is done in a next CL where I handlified JSArray::SetContent().

[Yang, 2014-03-20 13:29:09]

On 2014/03/20 13:28:13, Igor Sheludko wrote:
> https://codereview.chromium.org/206383003/diff/1/src/runtime.cc
> File src/runtime.cc (right):
> 
> https://codereview.chromium.org/206383003/diff/1/src/runtime.cc#newcode13101
> src/runtime.cc:13101:
> isolate->factory()->SetContent(Handle<JSArray>::cast(result), instances);
> On 2014/03/20 13:22:07, Yang wrote:
> > for the sake of consistency, could we move SetContent to a static
> > JSArray::SetContent?
> 
> It is done in a next CL where I handlified JSArray::SetContent().

LGTM then.

[Igor Sheludko, 2014-03-20 13:40:14]

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