Fix Array.prototype.{reduce,reduceRight} to pass undefined as receiver

test/mjsunit/regress/regress-1436.js

Index: test/mjsunit/regress/regress-1436.js
diff --git a/test/mjsunit/regress/regress-1436.js b/test/mjsunit/regress/regress-1436.js
new file mode 100644
index 0000000000000000000000000000000000000000..0dbad4ec7be7378bc687490d3139fc17965a5966
--- /dev/null
+++ b/test/mjsunit/regress/regress-1436.js
@@ -0,0 +1,56 @@
+// Copyright 2011 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
+//       with the distribution.
+//     * Neither the name of Google Inc. nor the names of its
+//       contributors may be used to endorse or promote products derived
+//       from this software without specific prior written permission.
+//
+// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+// "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+// LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+// A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+// OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+// SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+// LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+// DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+// THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+// (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+// OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+// Check that reduce and reduceRight call the callback function with
+// undefined as the receiver.
+//
+// In addition, check that none of the array functions that take
+// callbacks use Function.prototype.apply or Function.prototype.call
+// internally.
+
+Function.prototype.apply = function() {
+  throw "Array operation used Function.prototype.apply";
+};
+
+
+Function.prototype.call = function() {
+  throw "Array operation used Function.prototype.call";
+};
+
+
+// Check receiver for reduce and reduceRight.
+assertEquals(void 0, [2, 3].reduce(function(){ return this; }));

[Lasse Reichstein, 2011/06/08 18:41:42]

That's incorrect. The "this" value should be the global object for a non-strict
function.

+assertEquals(void 0, [2, 3].reduceRight(function(){ return this; }));
+
+
+// Call each of the other array functions to verify that the overwritten
+// Function.prototype.{call,apply} are not called.
+[2, 3].filter(function() {});
+[2, 3].forEach(function() {});
+[2, 3].some(function() {});
+[2, 3].every(function() {});
+[2, 3].map(function() {});

[Lasse Reichstein, 2011/06/08 18:41:42]

Try testing with both strict and non-strict functions, and check that they get
undefined and global object respectively.