[modules] Propagate module status when creating ParseInfo from SFI.

[modules] Propagate module status when creating ParseInfo from SFI.

In the ParseInfo constructor that takes a SharedFunctionInfo, we must
set the module flag when the function represents a module.

BUG=v8:1569
Committed: https://crrev.com/cde9f7bfedbff818fc05e5c7f723b108de4c6a4a
Cr-Commit-Position: refs/heads/master@{#41257}

[neis, 2016-11-23 16:17:10]

Description was changed from

==========
[modules] Propagate module status when creating ParseInfo from SFI.

In the ParseInfo constructor that takes a SharedFunctionInfo, we must set the
module flag when the function represents a module.

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

to

==========
[modules] Propagate module status when creating ParseInfo from SFI.

In the ParseInfo constructor that takes a SharedFunctionInfo, we must
set the module flag when the function represents a module.

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

[neis, 2016-11-23 16:17:10]

neis@chromium.org changed reviewers:
+ adamk@chromium.org

[neis, 2016-11-23 16:17:18]

ptal

[neis, 2016-11-24 09:58:14]

neis@chromium.org changed reviewers:
+ marja@chromium.org
- adamk@chromium.org

[neis, 2016-11-24 09:58:14]

+marja: ptal

[neis, 2016-11-24 10:09:16]

Description was changed from

==========
[modules] Propagate module status when creating ParseInfo from SFI.

In the ParseInfo constructor that takes a SharedFunctionInfo, we must
set the module flag when the function represents a module.

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

to

==========
[modules] Propagate module status when creating ParseInfo from SFI.

In the ParseInfo constructor that takes a SharedFunctionInfo, we must
set the module flag when the function represents a module.

BUG=v8:1569
==========

[Michael Starzinger, 2016-11-24 10:21:26]

LGTM. Thanks!

[marja, 2016-11-24 10:22:27]

lgtm

[neis, 2016-11-24 10:23:02]

The CQ bit was checked by neis@chromium.org

[commit-bot: I haz the power, 2016-11-24 10:23:06]

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-11-24 10:48:39]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479982982553550, "parent_rev":
"c4c98fdec82ea1f5eb8f6d7880cd65f49508ac10", "commit_rev":
"beafaa14b4347b6ce2dca2070e9af445528b67af"}

[commit-bot: I haz the power, 2016-11-24 10:48:45]

Description was changed from

==========
[modules] Propagate module status when creating ParseInfo from SFI.

In the ParseInfo constructor that takes a SharedFunctionInfo, we must
set the module flag when the function represents a module.

BUG=v8:1569
==========

to

==========
[modules] Propagate module status when creating ParseInfo from SFI.

In the ParseInfo constructor that takes a SharedFunctionInfo, we must
set the module flag when the function represents a module.

BUG=v8:1569
==========

[commit-bot: I haz the power, 2016-11-24 10:48:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-24 10:49:06]

Description was changed from

==========
[modules] Propagate module status when creating ParseInfo from SFI.

In the ParseInfo constructor that takes a SharedFunctionInfo, we must
set the module flag when the function represents a module.

BUG=v8:1569
==========

to

==========
[modules] Propagate module status when creating ParseInfo from SFI.

In the ParseInfo constructor that takes a SharedFunctionInfo, we must
set the module flag when the function represents a module.

BUG=v8:1569

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

[commit-bot: I haz the power, 2016-11-24 10:49:07]

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