[modules] Don't force context allocation of variables declared in module scope.

[modules] Don't force context allocation of variables declared in module scope.

There's no need to do so.

Also simplify NeedsContext(), which doesn't need to check for with-scopes.

R=adamk@chromium.org
BUG=

Committed: https://crrev.com/33f3262f557c36fdc6c2eca41660774b419c8fbc
Cr-Commit-Position: refs/heads/master@{#38451}

[adamk, 2016-08-08 15:35:59]

lgtm

[neis, 2016-08-08 15:42:41]

The CQ bit was checked by neis@chromium.org

[commit-bot: I haz the power, 2016-08-08 15:42:57]

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-08-08 16:16:35]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-08 16:19:34]

Description was changed from

==========
[modules] Don't force context allocation of variables declared in module scope.

There's no need to do so.

Also simplify NeedsContext(), which doesn't need to check for with-scopes.

R=adamk@chromium.org
BUG=
==========

to

==========
[modules] Don't force context allocation of variables declared in module scope.

There's no need to do so.

Also simplify NeedsContext(), which doesn't need to check for with-scopes.

R=adamk@chromium.org
BUG=

Committed: https://crrev.com/33f3262f557c36fdc6c2eca41660774b419c8fbc
Cr-Commit-Position: refs/heads/master@{#38451}
==========

[commit-bot: I haz the power, 2016-08-08 16:19:35]

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