[modules] Teach JSObject::ReferencesObject about module contexts.

[modules] Teach JSObject::ReferencesObject about module contexts.

A module context holds the module object in its extension slot, which should not be recursed on (it's not even a JSObject).

(I don't know how this code path can be triggered, though.)

R=adamk@chromium.org
BUG=v8:1569
Review-Url: https://codereview.chromium.org/2919063002
Cr-Commit-Position: refs/heads/master@{#45695}
Committed: https://chromium.googlesource.com/v8/v8/+/90b68baabc5ba545de2a73047f9beaac8b5bc0b5

[neis, 2017-06-02 14:58:16]

Description was changed from

==========
[modules] Teach JSObject::ReferencesObject about module contexts.

A module context holds the module object in its extension slot, which is
not a JSObject, so don't recurse on it.

(I don't know how/if this code path can be triggered, though.)

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

to

==========
[modules] Teach JSObject::ReferencesObject about module contexts.

A module context holds the module object in its extension slot, which should not
be recursed on (it's not even a JSObject).

(I don't know how this code path can be triggered, though.)

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

[neis, 2017-06-02 14:58:21]

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

[commit-bot: I haz the power, 2017-06-02 14:58:28]

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, 2017-06-02 15:34:09]

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

[commit-bot: I haz the power, 2017-06-02 15:34:10]

Dry run: This issue passed the CQ dry run.

[adamk, 2017-06-02 17:06:00]

lgtm

[neis, 2017-06-02 17:34:11]

The CQ bit was checked by neis@chromium.org

[commit-bot: I haz the power, 2017-06-02 17:34:19]

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, 2017-06-02 17:36:01]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496424851011100, "parent_rev":
"b015229c091cb064c101b28b8da477ef1a356bd6", "commit_rev":
"90b68baabc5ba545de2a73047f9beaac8b5bc0b5"}

[commit-bot: I haz the power, 2017-06-02 17:36:08]

Description was changed from

==========
[modules] Teach JSObject::ReferencesObject about module contexts.

A module context holds the module object in its extension slot, which should not
be recursed on (it's not even a JSObject).

(I don't know how this code path can be triggered, though.)

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

to

==========
[modules] Teach JSObject::ReferencesObject about module contexts.

A module context holds the module object in its extension slot, which should not
be recursed on (it's not even a JSObject).

(I don't know how this code path can be triggered, though.)

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

Review-Url: https://codereview.chromium.org/2919063002
Cr-Commit-Position: refs/heads/master@{#45695}
Committed:
https://chromium.googlesource.com/v8/v8/+/90b68baabc5ba545de2a73047f9beaac8b5...
==========

[commit-bot: I haz the power, 2017-06-02 17:36:09]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/90b68baabc5ba545de2a73047f9beaac8b5...