ForIn/OfBodyEvaluation: missing return if IteratorComplete

[woess]

#1066 changed 13.7.5.13 Runtime Semantics: ForIn/OfBodyEvaluation:

-            1. Let _nextResult_ be ? IteratorStep(_iteratorRecord_).
-            1. If _nextResult_ is *false*, return NormalCompletion(_V_).
+            1. Let _nextResult_ be ? Call(_iteratorRecord_.[[NextMethod]], _iteratorRecord_.[[Iterator]], « »).
+            1. If _iteratorKind_ is ~async~, then set _nextResult_ to ? Await(_nextResult_).
+            1. If Type(_nextResult_) is not Object, throw a *TypeError* exception.
             1. Let _nextValue_ be ? IteratorValue(_nextResult_).

Note how the step If _nextResult_ is *false*, return NormalCompletion(_V_). is gone without replacement.

Unless I'm missing something, I believe the following steps should be inserted before Let _nextValue_ be ? IteratorValue(_nextResult_):

1. Let _done_ be ? IteratorComplete(_nextResult_).
1. If _done_ is *true*, then
  1. Return ? IteratorValue(_nextResult_).

[littledan]

Cc @domenic

[littledan]

Cc @domenic