ES6: Add support for Array.prototype.fill()

ES6: Add support for Array.prototype.fill()

BUG=v8:3273
LOG=Y
R=dslomov@chromium.org

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

[rossberg, 2014-04-23 11:44:18]

Added Dmitry as reviewer

[Dmitry Lomov (no reviews), 2014-04-28 11:18:25]

looking good, just needs some more tests

https://codereview.chromium.org/240873002/diff/1/test/mjsunit/harmony/array-f...
File test/mjsunit/harmony/array-fill.js (right):

https://codereview.chromium.org/240873002/diff/1/test/mjsunit/harmony/array-f...
test/mjsunit/harmony/array-fill.js:10: assertArrayEquals([0, 0, 0, 0,
0].fill(8), [8, 8, 8, 8, 8]);
Add test for empty array. 
Add test for negative start and no end.
Add test for start > length.

[aperez, 2014-04-29 08:38:44]

On 2014/04/28 11:18:25, Dmitry Lomov (chromium) wrote:
> looking good, just needs some more tests

=)
 
>
https://codereview.chromium.org/240873002/diff/1/test/mjsunit/harmony/array-f...
> test/mjsunit/harmony/array-fill.js:10: assertArrayEquals([0, 0, 0, 0,
> 0].fill(8), [8, 8, 8, 8, 8]);
> Add test for empty array. 
> Add test for negative start and no end.
> Add test for start > length.

I have added those test cases and updated the CL. Everything should be fine now
for landing.

[Dmitry Lomov (no reviews), 2014-04-29 08:58:51]

lgtm, I'll land shortly (CLA has been signed by Igalia)

[aperez, 2014-04-29 09:21:40]

On 2014/04/29 08:58:51, Dmitry Lomov (chromium) wrote:
> lgtm, I'll land shortly (CLA has been signed by Igalia)

Great, thanks you for taking care of landing.

[Dmitry Lomov (no reviews), 2014-04-30 08:28:39]

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

[ziyunfei, 2014-05-01 02:16:04]

On 2014/04/30 08:28:39, Dmitry Lomov (chromium) wrote:
> Committed patchset #2 manually as r21074 (presubmit successful).

In test/mjsunit/harmony/array-fill.js#7

assertEquals(1, Array.prototype.find.length);

should be

assertEquals(1, Array.prototype.fill.length);

[aperez, 2014-05-09 13:45:11]

On 2014/05/01 02:16:04, L446240525 wrote:
> On 2014/04/30 08:28:39, Dmitry Lomov (chromium) wrote:
> > Committed patchset #2 manually as r21074 (presubmit successful).
> 
> In test/mjsunit/harmony/array-fill.js#7
> 
> assertEquals(1, Array.prototype.find.length);
> 
> should be
> 
> assertEquals(1, Array.prototype.fill.length);

Certainly, I have filed another CL to fix this and another small typo:
https://codereview.chromium.org/277953002