[modules] Don't throw when detecting cycle while processing star exports.

[modules] Don't throw when detecting cycle while processing star exports.

We must not throw when seeing a cycle while trying to resolve a name through
star exports.  (It may be surprising that we do have to throw when seeing an
ambiguity, but this is what the spec says.)

R=adamk@chromium.org
BUG=v8:1569
Committed: https://crrev.com/b7913f33a3d979d656016403a3c5d33e01e5734a
Cr-Commit-Position: refs/heads/master@{#39787}

[neis, 2016-09-27 01:08:20]

The CQ bit was checked by neis@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-27 01:08:23]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-27 01:46:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-27 01:46:38]

Dry run: This issue passed the CQ dry run.

[adamk, 2016-09-27 17:25:36]

lgtm

[neis, 2016-09-27 17:28:27]

The CQ bit was checked by neis@chromium.org

[commit-bot: I haz the power, 2016-09-27 17:28:40]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-27 17:31:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-27 17:31:42]

Description was changed from

==========
[modules] Don't throw when detecting cycle while processing star exports.

We must not throw when seeing a cycle while trying to resolve a name through
star exports.  (It may be surprising that we do have to throw when seeing an
ambiguity, but this is what the spec says.)

R=adamk@chromium.org
BUG=v8:1569
==========

to

==========
[modules] Don't throw when detecting cycle while processing star exports.

We must not throw when seeing a cycle while trying to resolve a name through
star exports.  (It may be surprising that we do have to throw when seeing an
ambiguity, but this is what the spec says.)

R=adamk@chromium.org
BUG=v8:1569

Committed: https://crrev.com/b7913f33a3d979d656016403a3c5d33e01e5734a
Cr-Commit-Position: refs/heads/master@{#39787}
==========

[commit-bot: I haz the power, 2016-09-27 17:31:43]

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