ArraySplice builtin handlified.

src/factory.h

Index: src/factory.h
diff --git a/src/factory.h b/src/factory.h
index 22ec4d395ce628b04b9fc7e6e5dcb863f275c8a0..830aeee0e43aba8be1b98015aaffbd9366e636a8 100644
--- a/src/factory.h
+++ b/src/factory.h
@@ -355,8 +355,22 @@ class Factory {
 
   // JS arrays are pretenured when allocated by the parser.
   Handle<JSArray> NewJSArray(
+      ElementsKind elements_kind,
+      int length,
+      int capacity,
+      PretenureFlag pretenure = NOT_TENURED);
+
+  Handle<JSArray> NewJSArray(
       int capacity,
       ElementsKind elements_kind = TERMINAL_FAST_ELEMENTS_KIND,
+      PretenureFlag pretenure = NOT_TENURED) {
+    return NewJSArray(elements_kind, 0, capacity, pretenure);

[Yang, 2014/03/20 15:21:41]

Do we really need this wrapper?

[Igor Sheludko, 2014/03/21 09:49:39]

The NewJSArray() without length was here for some time and already has a number
of usages. But in my case I needed a more general one having both length and
capacity parameters. So I added it (see above) and defined the old one as a
special case of a new one.

+  }
+
+  Handle<JSArray> NewJSArrayWithElements(
+      Handle<FixedArrayBase> elements,
+      ElementsKind elements_kind,
+      int length,
       PretenureFlag pretenure = NOT_TENURED);

[Yang, 2014/03/20 15:21:41]

Same here.

[Igor Sheludko, 2014/03/21 09:49:39]

Same here for NewJSArrayWithElements(). To make this relationship more explicit
I'll move the implementation of the specific case to the header.

 
   Handle<JSArray> NewJSArrayWithElements(