--- a/js/src/frontend/Parser.cpp
+++ b/js/src/frontend/Parser.cpp
@@ -2093,34 +2093,31 @@ Parser<FullParseHandler, char16_t>::eval
     ParseContext evalpc(this, evalsc, /* newDirectives = */ nullptr);
     if (!evalpc.init())
         return nullptr;
 
     ParseContext::VarScope varScope(this);
     if (!varScope.init(pc))
         return nullptr;
 
-    ParseNode* body;
-    {
-        // All evals have an implicit non-extensible lexical scope.
-        ParseContext::Scope lexicalScope(this);
-        if (!lexicalScope.init(pc))
-            return nullptr;
-
-        body = statementList(YieldIsName);
-        if (!body)
-            return nullptr;
-
-        if (!checkStatementsEOF())
-            return nullptr;
-
-        body = finishLexicalScope(lexicalScope, body);
-        if (!body)
-            return nullptr;
-    }
+    // All evals have an implicit non-extensible lexical scope.
+    ParseContext::Scope lexicalScope(this);
+    if (!lexicalScope.init(pc))
+        return nullptr;
+
+    ParseNode* body = statementList(YieldIsName);
+    if (!body)
+        return nullptr;
+
+    if (!checkStatementsEOF())
+        return nullptr;
+
+    body = finishLexicalScope(lexicalScope, body);
+    if (!body)
+        return nullptr;
 
     // It's an error to use 'arguments' in a legacy generator expression.
     //
     // If 'arguments' appears free (i.e. not a declared name) or if the
     // declaration does not shadow the enclosing script's 'arguments'
     // binding (i.e. not a lexical declaration), check the enclosing
     // script.
     if (hasUsedName(context->names().arguments)) {

deleted file mode 100644
--- a/js/src/jit-test/tests/parser/bug-1363191.js
+++ /dev/null
@@ -1,2 +0,0 @@
-eval("{ function f() {} }");
-let f;