[parser] Fix unfinalized block scope in non-lexical for each

src/parsing/parser.cc

Index: src/parsing/parser.cc
diff --git a/src/parsing/parser.cc b/src/parsing/parser.cc
index 4973857c37ea0f2cf2854c35000be559db065e3a..622e563d01444f1466291fe373ef079aa62a1dbb 100644
--- a/src/parsing/parser.cc
+++ b/src/parsing/parser.cc
@@ -2667,7 +2667,9 @@ Statement* Parser::ParseForStatement(ZoneList<const AstRawString*>* labels,
         Statement* final_loop = InitializeForEachStatement(
             loop, expression, enumerable, body, each_keyword_position);
 
-        DCHECK_NULL(for_state.FinalizedBlockScope());

[nickie, 2016/09/15 14:54:25]

Method FinalizedBlockScope() has side effects, especially in the case that its
result is NULL.

+        Scope* for_scope = for_state.FinalizedBlockScope();
+        DCHECK_NULL(for_scope);
+        USE(for_scope);
         return final_loop;
 
       } else {