Reland 4068 which wrapped Array methods into native builtins (reverted in 4075).

Reland 4068 which wrapped Array methods into native builtins (reverted in 4075).

Committed: http://code.google.com/p/v8/source/detail?r=4104

[antonm, 2010-03-11 06:57:05]

Mads,

may you have a look?

It's almost revert of 4075.  One thing I changed:

diff --git a/src/builtins.cc b/src/builtins.cc
index f5132c6..1e95bc2 100644
--- a/src/builtins.cc
+++ b/src/builtins.cc
@@ -349,7 +349,7 @@ static Object* CallJsBuiltin(const char* name,
   Vector<Object**> argv(Vector<Object**>::New(args.length() - 1));
   int n_args = args.length() - 1;
   for (int i = 0; i < n_args; i++) {
-    argv[i] = &args[i + 1];
+    argv[i] = args.at<Object>(i + 1).location();
   }
   bool pending_exception = false;
   Handle<Object> result = Execution::Call(function,

[Mads Ager (chromium), 2010-03-11 07:18:33]

LGTM, thanks for finding the unrelated issue that this triggered!