Modules: simplify logic around allocation of module internal variables

Modules: simplify logic around allocation of module internal variables

Since recursive modules are gone, only the top-level scope can have
module inner scopes. Rename Scope::AllocateModulesRecursively to
Scope::AllocateModules, and add test showing the module Variables
are still allocated appropriately in the top level scope.

BUG=v8:1569, v8:3940
LOG=n

Committed: https://crrev.com/9703c057c504a2dcc202e07d027919a566630ae6
Cr-Commit-Position: refs/heads/master@{#27143}

[adamk, 2015-03-11 22:16:48]

adamk@chromium.org changed reviewers:
+ arv@chromium.org, rossberg@chromium.org

[arv (Not doing code reviews), 2015-03-11 22:47:07]

LGTM

[adamk, 2015-03-11 22:50:15]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2015-03-11 22:50:26]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/999893003/1

[commit-bot: I haz the power, 2015-03-11 22:57:06]

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

[commit-bot: I haz the power, 2015-03-11 22:57:07]

Try jobs failed on following builders:
  v8_presubmit on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/1167)

[adamk, 2015-03-11 22:58:55]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2015-03-11 22:59:13]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/999893003/1

[commit-bot: I haz the power, 2015-03-11 23:19:51]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-03-11 23:20:04]

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