Fixing Sloppy Symbol.iterator setter

test/mjsunit/es6/arguments-iterator.js

Index: test/mjsunit/es6/arguments-iterator.js
diff --git a/test/mjsunit/es6/arguments-iterator.js b/test/mjsunit/es6/arguments-iterator.js
index cf1e1f97ca32242770c0d915fde276aca85b6920..a5aede64664a1d07dbcaca58c240006b9c57ea8a 100644
--- a/test/mjsunit/es6/arguments-iterator.js
+++ b/test/mjsunit/es6/arguments-iterator.js
@@ -160,6 +160,20 @@ function TestAssignmentToIterator() {
 TestAssignmentToIterator(1, 2, 3, 4, 5);
 
 
+function TestAssignmentToIterator2() {

[Camillo Bruni, 2015/09/10 13:59:48]

Do you add a regression/?? comment here?

[Jakob Kummerow, 2015/09/10 14:29:20]

You certainly can. There's no formal syntax, I'd just write:

// Regression test for crbug.com/521484.

+  var i = 0;
+  arguments.__defineGetter__('callee', function(){});
+  arguments.__defineGetter__('length', function(){ return 1 });
+  arguments[Symbol.iterator] = [].entries;
+  for (var entry of arguments) {
+    assertEquals([i, arguments[i]], entry);
+    i++;
+  }
+
+  assertEquals(arguments.length, i);
+}
+TestAssignmentToIterator2(1, 2, 3, 4, 5);
+

[Jakob Kummerow, 2015/09/10 14:29:20]

nit: two empty lines between top-level things

 function TestArgumentsMutation() {
   var i = 0;
   for (var x of arguments) {