[modules] Implement @@iterator on namespace objects.

test/mjsunit/modules-namespace1.js

 // Copyright 2016 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // MODULE
 
 let ja = 42;
 export {ja as yo};
 export const bla = "blaa";
 export {foo as foo_again};
 // See further below for the actual star import that declares "foo".
 
 // The object itself.
 assertEquals("object", typeof foo);
 assertThrows(() => foo = 666, TypeError);
 assertFalse(Reflect.isExtensible(foo));
 assertTrue(Reflect.preventExtensions(foo));
 assertThrows(() => Reflect.apply(foo, {}, []));
 assertThrows(() => Reflect.construct(foo, {}, []));
 assertSame(null, Reflect.getPrototypeOf(foo));
 // TODO(neis): The next one should be False.
 assertTrue(Reflect.setPrototypeOf(foo, null));
 assertFalse(Reflect.setPrototypeOf(foo, {}));
 assertSame(null, Reflect.getPrototypeOf(foo));
-// TODO(neis): The next one should include @@iterator at the end.
 assertEquals(
-    ["bla", "foo_again", "yo", Symbol.toStringTag],
+    ["bla", "foo_again", "yo", Symbol.toStringTag, Symbol.iterator],
     Reflect.ownKeys(foo));
 
 // Its "yo" property.
 assertEquals(
     {value: 42, enumerable: true, configurable: false, writable: true},
     Reflect.getOwnPropertyDescriptor(foo, "yo"));
 assertFalse(Reflect.deleteProperty(foo, "yo"));
 assertTrue(Reflect.has(foo, "yo"));
 // TODO(neis): The next three should be False.
 assertTrue(Reflect.set(foo, "yo", true));
 assertTrue(Reflect.defineProperty(foo, "yo",
     Reflect.getOwnPropertyDescriptor(foo, "yo")));
 assertTrue(Reflect.defineProperty(foo, "yo", {}));
 assertFalse(Reflect.defineProperty(foo, "yo", {get() {return 1}}));
 assertEquals(42, Reflect.get(foo, "yo"));
 assertEquals(43, (ja++, foo.yo));
 
 // Its "foo_again" property.
 assertSame(foo, foo.foo_again);
 
 // Its @@toStringTag property.
 assertTrue(Reflect.has(foo, Symbol.toStringTag));
 assertEquals("string", typeof Reflect.get(foo, Symbol.toStringTag));
 assertEquals(
     {value: "Module", configurable: true, writable: false, enumerable: false},
     Reflect.getOwnPropertyDescriptor(foo, Symbol.toStringTag));
 
-// TODO(neis): Its @@iterator property.
-// assertTrue(Reflect.has(foo, Symbol.iterator));
-// assertEquals("function", typeof Reflect.get(foo, Symbol.iterator));
-// assertEquals(["bla", "yo"], [...foo]);
-// assertThrows(() => (42, foo[Symbol.iterator])(), TypeError);
-// assertSame(foo[Symbol.iterator]().__proto__,
-//     ([][Symbol.iterator]()).__proto__.__proto__);
+// Its @@iterator property.
+assertTrue(Reflect.has(foo, Symbol.iterator));
+assertEquals("function", typeof Reflect.get(foo, Symbol.iterator));
+assertEquals("[Symbol.iterator]", foo[Symbol.iterator].name);
+assertEquals(0, foo[Symbol.iterator].length);
+assertSame(Function.prototype, foo[Symbol.iterator].__proto__);
+assertEquals(
+    {value: foo[Symbol.iterator],
+     configurable: true, writable: true, enumerable: false},
+    Reflect.getOwnPropertyDescriptor(foo, Symbol.iterator));
+assertEquals(["bla", "foo_again", "yo"], [...foo]);
+assertThrows(() => (42, foo[Symbol.iterator])(), TypeError);
+{
+  let it = foo[Symbol.iterator]();
+  assertSame(it.__proto__, ([][Symbol.iterator]()).__proto__.__proto__);
+  assertEquals(["next"], Reflect.ownKeys(it));
+  assertEquals(
+      {value: it.next, configurable: true, writable: true, enumerable: false},
+      Reflect.getOwnPropertyDescriptor(it, "next"));
+  assertEquals("function", typeof it.next);
+  assertEquals("next", it.next.name);
+  assertEquals(0, it.next.length);
+  assertSame(Function.prototype, it.next.__proto__);
+  assertFalse(it === foo[Symbol.iterator]());
+  assertFalse(it.next === foo[Symbol.iterator]().next);
+  assertThrows(() => (42, it.next)(), TypeError);
+  assertThrows(() => it.next.call(foo[Symbol.iterator]()), TypeError);
+  let next = it.next;
+  assertEquals(42, (it.next = 42, it.next));
+  assertEquals(43, (it.bla = 43, it.bla));
+  assertTrue(delete it.next);
+  assertThrows(() => next.call(foo[Symbol.iterator]()), TypeError);
+}
 
 // TODO(neis): Clarify spec w.r.t. other symbols.
 
 // Nonexistant properties.
 let nonexistant = ["gaga", 123, Symbol('')];
 for (let key of nonexistant) {
   assertSame(undefined, Reflect.getOwnPropertyDescriptor(foo, key));
   assertTrue(Reflect.deleteProperty(foo, key));
   assertFalse(Reflect.set(foo, key, true));
   assertSame(undefined, Reflect.get(foo, key));
   assertFalse(Reflect.defineProperty(foo, key, {get() {return 1}}));
   assertFalse(Reflect.has(foo, key));
 }
 
 // The actual star import that we are testing. Namespace imports are
-// initialized before evaluation
+// initialized before evaluation.
 import * as foo from "modules-namespace1.js";
 
 // There can be only one namespace object.
 import * as bar from "modules-namespace1.js";
 assertSame(foo, bar);