[parser] Disallow Expression in for..of statements

src/parsing/parser.cc

Index: src/parsing/parser.cc
diff --git a/src/parsing/parser.cc b/src/parsing/parser.cc
index 0048b8da698b718cc01671f52ab659b905427c2e..4d98fe95e87c2c735f4d25138cd305cb35eac108 100644
--- a/src/parsing/parser.cc
+++ b/src/parsing/parser.cc
@@ -3676,7 +3676,13 @@ Statement* Parser::ParseForStatement(ZoneList<const AstRawString*>* labels,
             factory()->NewForEachStatement(mode, labels, stmt_pos);
         Target target(&this->target_stack_, loop);
 
-        Expression* enumerable = ParseExpression(true, CHECK_OK);
+        Expression* enumerable;
+        if (mode == ForEachStatement::ITERATE) {
+          ExpressionClassifier classifier;
+          enumerable = ParseAssignmentExpression(true, &classifier, CHECK_OK);

[adamk, 2016/01/19 22:21:47]

Why no ValidateExpression() call here?

+        } else {
+          enumerable = ParseExpression(true, CHECK_OK);
+        }
 
         Expect(Token::RPAREN, CHECK_OK);
 
@@ -3791,7 +3797,14 @@ Statement* Parser::ParseForStatement(ZoneList<const AstRawString*>* labels,
             factory()->NewForEachStatement(mode, labels, stmt_pos);
         Target target(&this->target_stack_, loop);
 
-        Expression* enumerable = ParseExpression(true, CHECK_OK);
+        Expression* enumerable;
+        if (mode == ForEachStatement::ITERATE) {
+          ExpressionClassifier classifier;
+          enumerable = ParseAssignmentExpression(true, &classifier, CHECK_OK);

[adamk, 2016/01/19 22:21:47]

Same question down here?

+        } else {
+          enumerable = ParseExpression(true, CHECK_OK);
+        }
+
         Expect(Token::RPAREN, CHECK_OK);
 
         // Make a block around the statement in case a lexical binding