[parsing] Never marks script scopes as sloppy eval

This prevents the bytecode generator generating loads that look for
extensions in the global context, which can never succeed and means
that lazy and eager bytecode compilation will match.

Bug: v8:8510
Change-Id: I51dca62b5d1ee34f8dea82260cf27295ddf427d9
Reviewed-on: https://chromium-review.googlesource.com/c/1425520Reviewed-by: Toon Verwaest <verwaest@chromium.org>
Commit-Queue: Dan Elphick <delphick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#58983}

src/ast/scopes.h

@@ -751,7 +751,10 @@ class V8_EXPORT_PRIVATE DeclarationScope : public Scope {
   }
 
   bool calls_sloppy_eval() const {
-    return scope_calls_eval_ && is_sloppy(language_mode());
+    // TODO(delphick): Calculate this when setting and change the name of
+    // scope_calls_eval_.
+    return !is_script_scope() && scope_calls_eval_ &&
+           is_sloppy(language_mode());
   }
 
   bool was_lazily_parsed() const { return was_lazily_parsed_; }