Move runtime array constructor functions from builtins.cc to runtime.cc.

Move runtime array constructor functions from builtins.cc to runtime.cc.

Not only is runtime.cc a better location, but situations arise soon where we'll
want to make runtime calls to these functions.

BUG=
R=mstarzinger@chromium.org

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

[mvstanton, 2013-06-06 11:45:50]

Hi Michael, here is (finally) the follow-up to the TODO we discussed, moving
array constructor stub failure code to runtime.cc.

[Michael Starzinger, 2013-06-06 12:22:52]

Just a few nits for the runtime.cc file.

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

https://codereview.chromium.org/16399007/diff/1/src/runtime.cc#newcode13505
src/runtime.cc:13505: RUNTIME_FUNCTION(MaybeObject*, Runtime_ArrayConstructor) {
Each runtime function should have it's own HandleScope (or one of the assertion
scopes).

https://codereview.chromium.org/16399007/diff/1/src/runtime.cc#newcode13516
src/runtime.cc:13516: Handle<JSFunction> constructor =
args.at<JSFunction>(parameters_start);
Use the CONVERT_ARG_HANDLE_CHECKED macro to access "args".

https://codereview.chromium.org/16399007/diff/1/src/runtime.cc#newcode13526
src/runtime.cc:13526: RUNTIME_FUNCTION(MaybeObject*,
Runtime_InternalArrayConstructor) {
Likewise.

https://codereview.chromium.org/16399007/diff/1/src/runtime.cc#newcode13534
src/runtime.cc:13534: Handle<JSFunction> constructor =
args.at<JSFunction>(parameters_start);
Use the CONVERT_ARG_HANDLE_CHECKED macro to access "args".

[mvstanton, 2013-06-06 12:30:31]

Right on, thanks for the good look!
--Michael

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

https://codereview.chromium.org/16399007/diff/1/src/runtime.cc#newcode13505
src/runtime.cc:13505: RUNTIME_FUNCTION(MaybeObject*, Runtime_ArrayConstructor) {
On 2013/06/06 12:22:52, Michael Starzinger wrote:
> Each runtime function should have it's own HandleScope (or one of the
assertion
> scopes).

Done.

https://codereview.chromium.org/16399007/diff/1/src/runtime.cc#newcode13516
src/runtime.cc:13516: Handle<JSFunction> constructor =
args.at<JSFunction>(parameters_start);
On 2013/06/06 12:22:52, Michael Starzinger wrote:
> Use the CONVERT_ARG_HANDLE_CHECKED macro to access "args".

Done.

https://codereview.chromium.org/16399007/diff/1/src/runtime.cc#newcode13526
src/runtime.cc:13526: RUNTIME_FUNCTION(MaybeObject*,
Runtime_InternalArrayConstructor) {
On 2013/06/06 12:22:52, Michael Starzinger wrote:
> Likewise.

Done.

https://codereview.chromium.org/16399007/diff/1/src/runtime.cc#newcode13534
src/runtime.cc:13534: Handle<JSFunction> constructor =
args.at<JSFunction>(parameters_start);
On 2013/06/06 12:22:52, Michael Starzinger wrote:
> Use the CONVERT_ARG_HANDLE_CHECKED macro to access "args".

Done.

[Michael Starzinger, 2013-06-06 12:33:15]

LGTM.

[mvstanton, 2013-06-06 13:22:52]

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