Reviewers: Dmitry Lomov (chromium), wingo, rossberg, Igor Sheludko,
l446240525_gmail.com,
Message:
On 2014/05/09 13:59:16, Igor Sheludko wrote:
lgtm
I am not committer yet, could you (or someone else) please land the patch?
Thanks in advance!
Description:
Fix typos in unit test for Array.prototype.fill()
BUG=
LOG=y
Please review this at https://codereview.chromium.org/277953002/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+2, -2 lines):
M test/mjsunit/harmony/array-fill.js
Index: test/mjsunit/harmony/array-fill.js
diff --git a/test/mjsunit/harmony/array-fill.js
b/test/mjsunit/harmony/array-fill.js
index
571233f6fa11bf10ac1b75a5b85f79355b0a74b5..eae18d113bf7e3d4fbbc2ecba48940009fc08f5b
100644
--- a/test/mjsunit/harmony/array-fill.js
+++ b/test/mjsunit/harmony/array-fill.js
@@ -4,7 +4,7 @@
// Flags: --harmony-arrays
-assertEquals(1, Array.prototype.find.length);
+assertEquals(1, Array.prototype.fill.length);
assertArrayEquals([].fill(8), []);
assertArrayEquals([0, 0, 0, 0, 0].fill(), [undefined, undefined,
undefined, undefined, undefined]);
@@ -22,7 +22,7 @@ assertArrayEquals([0, 0, 0, 0, 0].fill(8, -1, -3), [0, 0,
0, 0, 0]);
assertArrayEquals([0, 0, 0, 0, 0].fill(8, undefined, 4), [8, 8, 8, 8, 0]);
assertArrayEquals([ , , , , 0].fill(8, 1, 3), [, 8, 8, , 0]);
-// If the range if empty, the array is not actually modified and
+// If the range is empty, the array is not actually modified and
// should not throw, even when applied to a frozen object.
assertArrayEquals(Object.freeze([1, 2, 3]).fill(0, 0, 0), [1, 2, 3]);
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
