Promise.all and Promise.race reject non-array parameter.

Promise.all and Promise.race should reject non-array parameter.

Promise.all and Promise.race should reject the returned Promise if an
invalid parameter is given.
Since they don't support iterable now, they should reject the Promise
if a non-array parameter is given.

BUG=347453
LOG=Y
R=rossberg@chromium.org

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

[rossberg, 2014-02-27 08:45:27]

https://codereview.chromium.org/182613003/diff/1/test/mjsunit/harmony/promise...
File test/mjsunit/harmony/promises.js (right):

https://codereview.chromium.org/182613003/diff/1/test/mjsunit/harmony/promise...
test/mjsunit/harmony/promises.js:562: Promise.all({get length() { throw 666
}}).chain(
Nit: I would remove the poisoned length method from the argument, since it is
immaterial now.

https://codereview.chromium.org/182613003/diff/1/test/mjsunit/harmony/promise...
test/mjsunit/harmony/promises.js:661: Promise.race({get length() { throw 666
}}).chain(
Same here.

[yhirano, 2014-02-27 08:54:51]

https://codereview.chromium.org/182613003/diff/1/test/mjsunit/harmony/promise...
File test/mjsunit/harmony/promises.js (right):

https://codereview.chromium.org/182613003/diff/1/test/mjsunit/harmony/promise...
test/mjsunit/harmony/promises.js:562: Promise.all({get length() { throw 666
}}).chain(
On 2014/02/27 08:45:28, rossberg wrote:
> Nit: I would remove the poisoned length method from the argument, since it is
> immaterial now.

Done.

https://codereview.chromium.org/182613003/diff/1/test/mjsunit/harmony/promise...
test/mjsunit/harmony/promises.js:661: Promise.race({get length() { throw 666
}}).chain(
On 2014/02/27 08:45:28, rossberg wrote:
> Same here.

Done.

[rossberg, 2014-02-27 09:30:28]

lgtm

[rossberg, 2014-03-10 12:01:13]

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