[modules] Never do lazy parsing in modules.

[modules] Never do lazy parsing in modules.

If we want to allow this in the future, we must force context-allocation for all
variables that are declared at the module level but not MODULE-allocated.

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

[adamk, 2016-09-26 22:08:53]

lgtm

[neis, 2016-09-26 22:09:08]

The CQ bit was checked by neis@chromium.org

[commit-bot: I haz the power, 2016-09-26 22:09:11]

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-26 22:26:01]

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

[commit-bot: I haz the power, 2016-09-26 22:26:02]

Try jobs failed on following builders:
  v8_mac_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/9673)
  v8_mac_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng_triggered/bui...)

[neis, 2016-09-26 22:53:49]

Description was changed from

==========
[modules] Never allow lazy parsing for modules.

If we want to allow this in the future, we must force context-allocation for all
variables that are declared at the module level but not MODULE-allocated.

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

to

==========
[modules] Never do lazy parsing in modules.

If we want to allow this in the future, we must force context-allocation for all
variables that are declared at the module level but not MODULE-allocated.

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

[neis, 2016-09-26 22:53:54]

The CQ bit was checked by neis@chromium.org

[neis, 2016-09-26 22:53:55]

The patchset sent to the CQ was uploaded after l-g-t-m from adamk@chromium.org
Link to the patchset: https://codereview.chromium.org/2373573003/#ps20001
(title: "Different approach.")

[commit-bot: I haz the power, 2016-09-26 22:54:03]

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

[adamk, 2016-09-26 22:58:18]

still lgtm :)

[commit-bot: I haz the power, 2016-09-26 23:19:43]

Description was changed from

==========
[modules] Never do lazy parsing in modules.

If we want to allow this in the future, we must force context-allocation for all
variables that are declared at the module level but not MODULE-allocated.

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

to

==========
[modules] Never do lazy parsing in modules.

If we want to allow this in the future, we must force context-allocation for all
variables that are declared at the module level but not MODULE-allocated.

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

[commit-bot: I haz the power, 2016-09-26 23:19:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-26 23:20:08]

Description was changed from

==========
[modules] Never do lazy parsing in modules.

If we want to allow this in the future, we must force context-allocation for all
variables that are declared at the module level but not MODULE-allocated.

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

to

==========
[modules] Never do lazy parsing in modules.

If we want to allow this in the future, we must force context-allocation for all
variables that are declared at the module level but not MODULE-allocated.

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

Committed: https://crrev.com/9f5ef0a453521a39b707c3ca6c73ca056ff7a457
Cr-Commit-Position: refs/heads/master@{#39745}
==========

[commit-bot: I haz the power, 2016-09-26 23:20:09]

Patchset 2 (id:??) landed as
https://crrev.com/9f5ef0a453521a39b707c3ca6c73ca056ff7a457
Cr-Commit-Position: refs/heads/master@{#39745}