Fix issue 1361 - Implement ES5 Array.prototype.toString.

Fix issue 1361 - Implement ES5 Array.prototype.toString.

BUG=v8:1361
TEST=mjsunit/array-tostring

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

[Rico, 2011-10-04 12:10:45]

http://codereview.chromium.org/8124025/diff/1/test/mjsunit/array-tostring.js
File test/mjsunit/array-tostring.js (right):

http://codereview.chromium.org/8124025/diff/1/test/mjsunit/array-tostring.js#...
test/mjsunit/array-tostring.js:78: 
Add comment on what we are testing, e.g.,
// Check that setting join on a normal object works.

http://codereview.chromium.org/8124025/diff/1/test/mjsunit/array-tostring.js#...
test/mjsunit/array-tostring.js:85: 
One more comment?:
// Check that when join is not callable we fallback to the normal tostring on
the actual object passed in

http://codereview.chromium.org/8124025/diff/1/test/mjsunit/array-tostring.js#...
test/mjsunit/array-tostring.js:104: assertFalse(called_before);
indention here seems random

http://codereview.chromium.org/8124025/diff/1/test/mjsunit/array-tostring.js#...
test/mjsunit/array-tostring.js:110: assertEquals(success, 42..arrayToString());
Please add parenthesis around 42 to increase readability

[Rico, 2011-10-04 12:10:57]

And LGTM

[Lasse Reichstein, 2011-10-04 12:56:33]

http://codereview.chromium.org/8124025/diff/1/test/mjsunit/array-tostring.js
File test/mjsunit/array-tostring.js (right):

http://codereview.chromium.org/8124025/diff/1/test/mjsunit/array-tostring.js#...
test/mjsunit/array-tostring.js:104: assertFalse(called_before);
Pretty much, yes. I blame Emacs.

http://codereview.chromium.org/8124025/diff/1/test/mjsunit/array-tostring.js#...
test/mjsunit/array-tostring.js:110: assertEquals(success, 42..arrayToString());
Done.

[Rico, 2011-10-04 13:01:49]

Still lgtm