[parser] Enforce module-specific identifier restriction

src/ast/scopes.h

Index: src/ast/scopes.h
diff --git a/src/ast/scopes.h b/src/ast/scopes.h
index 76f761dba3ceaf8cd0681982be7541dec60b3772..b1a789597f3d70acf57a7cead4d0e9304a5b3775 100644
--- a/src/ast/scopes.h
+++ b/src/ast/scopes.h
@@ -352,6 +352,9 @@ class Scope: public ZoneObject {
   // Is this scope inside a with statement.
   bool inside_with() const { return scope_inside_with_; }
 
+  // Is this scope inside module code.
+  bool inside_module() const { return scope_inside_module_; }

[adamk, 2016/02/23 22:37:15]

Scope isn't the right place to put this, since it's not used at all in any Scope
analysis logic.

Can't this just be on Parser? There's no way to parse a module inside a module,
so for any given parser invocation, the options are:

1. We're parsing a module
2. We're parsing some other top-level program
3. We're parsing a function lazily, in which case it must have parsed
successfully the first time.

[mike3, 2016/02/24 16:20:54]

Acknowledged.

+
   // Does this scope access "arguments".
   bool uses_arguments() const { return scope_uses_arguments_; }
   // Does this scope access "super" property (super.foo).
@@ -653,6 +656,8 @@ class Scope: public ZoneObject {
   //
   // This scope is inside a 'with' of some outer scope.
   bool scope_inside_with_;
+  // This scpope is inside module code
+  bool scope_inside_module_;
   // This scope contains a 'with' statement.
   bool scope_contains_with_;
   // This scope or a nested catch scope or with scope contain an 'eval' call. At