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

test/mjsunit/harmony/promises.js

 // Copyright 2013 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 541 matching lines @@
   var p = deferred.promise
   deferred.resolve(p)
   p.then(
     assertUnreachable,
     function(r) { assertAsync(r instanceof TypeError, "cyclic/deferred/then") }
   )
   assertAsyncRan()
 })();
 
 (function() {
   Promise.all({get length() { throw 666 }}).chain(

[rossberg, 2014/02/27 08:45:28]

Nit: I would remove the poisoned length method from the argument, since it is
immaterial now.

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

Done.

     assertUnreachable,
-    function(r) { assertAsync(r === 666, "all/no-array") }
+    function(r) { assertAsync(r instanceof TypeError, "all/no-array") }
   )
   assertAsyncRan()
 })();
 
 (function() {
   Promise.all([]).chain(
     function(x) { assertAsync(x.length === 0, "all/resolve/empty") },
     assertUnreachable
   )
   assertAsyncRan()
@@ skipping 76 matching lines @@
   var p2 = Promise.resolve(2)
   var p3 = Promise.resolve(3)
   Promise.race([0, p1, p2, p3]).chain(
     function(x) { assertAsync(x === 0, "resolved-const/one") },
     assertUnreachable
   )
   assertAsyncRan()
 })();
 
 (function() {
   Promise.race({get length() { throw 666 }}).chain(

[rossberg, 2014/02/27 08:45:28]

Same here.

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

Done.

     assertUnreachable,
-    function(r) { assertAsync(r === 666, "one/no-array") }
+    function(r) { assertAsync(r instanceof TypeError, "one/no-array") }
   )
   assertAsyncRan()
 })();
 
 (function() {
   var deferred1 = Promise.defer()
   var p1 = deferred1.promise
   var deferred2 = Promise.defer()
   var p2 = deferred2.promise
   var deferred3 = Promise.defer()
@@ skipping 108 matching lines @@
   Promise.all([11, Promise.resolve(12), 13, MyPromise.resolve(14), 15, 16])
   assertTrue(log === "nx14cn", "subclass/all/arg")
 
   log = ""
   MyPromise.all([21, Promise.resolve(22), 23, MyPromise.resolve(24), 25, 26])
   assertTrue(log === "nx24nnx21cnnx23cncnnx25cnnx26cn", "subclass/all/self")
 })();
 
 
 assertAsyncDone()