[es6] Fix invalid ToObject in String/Array iterator next.

[es6] Fix invalid ToObject in String/Array iterator next.

The spec says that the "this value" has to be an Object.

R=jarin@chromium.org

Committed: https://crrev.com/9427d3255b2eba784a6a61ea998b566ff7d07c4b
Cr-Commit-Position: refs/heads/master@{#30533}

[Benedikt Meurer, 2015-09-02 06:36:51]

Hey Jaro,

This is the fix we talked about. Pretty straight forward.
Please take a look.

Thanks,
Benedikt

[Jarin, 2015-09-02 06:41:02]

lgtm with a nit.

https://codereview.chromium.org/1325023003/diff/1/src/array-iterator.js
File src/array-iterator.js (right):

https://codereview.chromium.org/1325023003/diff/1/src/array-iterator.js#newco...
src/array-iterator.js:82: !HAS_DEFINED_PRIVATE(iterator,
arrayIteratorNextIndexSymbol)) {
Technically, you should check for presence of the other internal fields, but I
suppose that if arrayIteratorNextIndexSymbol is there, the others have to be
there as well. Could you add a comment that explains this?

[Benedikt Meurer, 2015-09-02 06:42:05]

https://codereview.chromium.org/1325023003/diff/1/src/array-iterator.js
File src/array-iterator.js (right):

https://codereview.chromium.org/1325023003/diff/1/src/array-iterator.js#newco...
src/array-iterator.js:82: !HAS_DEFINED_PRIVATE(iterator,
arrayIteratorNextIndexSymbol)) {
Yeah, I'll remove all of this soonish and replace with a dedicated instance
type. So this problem will disappear.

[Benedikt Meurer, 2015-09-02 06:42:09]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2015-09-02 06:42:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1325023003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1325023003/1

[commit-bot: I haz the power, 2015-09-02 06:59:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-09-02 06:59:23]

Patchset 1 (id:??) landed as
https://crrev.com/9427d3255b2eba784a6a61ea998b566ff7d07c4b
Cr-Commit-Position: refs/heads/master@{#30533}