From: marja <marja@chromium.org>
Date: Fri, 13 Mar 2015 09:46:05 +0000 (-0700)
Subject: Remove duplicate script scope.
X-Git-Tag: upstream/4.7.83~3864
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0c56d7e8092e130cb1c6c11cfa18dc968f74d131;p=platform%2Fupstream%2Fv8.git

Remove duplicate script scope.

We previously had two nested SCRIPT_SCOPEs (used to be GLOBAL_SCOPES), but as
the lexical variable implementation changed, the inner one is no longer needed.
BUG=

Review URL: https://codereview.chromium.org/1002193002

Cr-Commit-Position: refs/heads/master@{#27176}
---

diff --git a/src/parser.cc b/src/parser.cc
index 255fd28..ff7d61d 100644
--- a/src/parser.cc
+++ b/src/parser.cc
@@ -994,8 +994,6 @@ FunctionLiteral* Parser::DoParseProgram(ParseInfo* info, Scope** scope,
       if (!(*scope)->is_script_scope() || is_strict(info->language_mode())) {
         *scope = NewScope(*scope, EVAL_SCOPE);
       }
-    } else if (info->is_global()) {
-      *scope = NewScope(*scope, SCRIPT_SCOPE);
     } else if (info->is_module()) {
       *scope = NewScope(*scope, MODULE_SCOPE);
     }