[destructuring] More tests for object literal pattern

src/parser.cc

Index: src/parser.cc
diff --git a/src/parser.cc b/src/parser.cc
index 7d7f6a626991167af1ed5fa1f900d4db83640098..991d3c326cf60b1e19ca25694c7274b9fb582f66 100644
--- a/src/parser.cc
+++ b/src/parser.cc
@@ -4143,6 +4143,32 @@ void Parser::AddAssertIsConstruct(ZoneList<Statement*>* body, int pos) {
 }
 
 
+Statement* Parser::BuildAssertIsCoercible(Variable* var) {
+  // if (var === null || var === undefined)
+  //     return throw /* type error kNonCoercible) */;

[arv (Not doing code reviews), 2015/05/18 14:43:42]

Just throw, no return.

[Dmitry Lomov (no reviews), 2015/05/18 16:28:22]

Indeed, so much simpler! Awesomesauce. Done.

+
+  Expression* condition = factory()->NewBinaryOperation(
+      Token::OR, factory()->NewCompareOperation(
+                     Token::EQ_STRICT, factory()->NewVariableProxy(var),
+                     factory()->NewUndefinedLiteral(RelocInfo::kNoPosition),
+                     RelocInfo::kNoPosition),
+      factory()->NewCompareOperation(
+          Token::EQ_STRICT, factory()->NewVariableProxy(var),
+          factory()->NewNullLiteral(RelocInfo::kNoPosition),
+          RelocInfo::kNoPosition),
+      RelocInfo::kNoPosition);
+  Expression* throw_type_error = this->NewThrowTypeError(
+      MessageTemplate::kNonCoercible, ast_value_factory()->empty_string(),
+      RelocInfo::kNoPosition);
+  IfStatement* if_statement = factory()->NewIfStatement(
+      condition,
+      factory()->NewReturnStatement(throw_type_error, RelocInfo::kNoPosition),

[arv (Not doing code reviews), 2015/05/18 14:43:42]

NewExpressionStatement

[Dmitry Lomov (no reviews), 2015/05/18 16:28:22]

Done.

+      factory()->NewEmptyStatement(RelocInfo::kNoPosition),
+      RelocInfo::kNoPosition);
+  return if_statement;
+}
+
+
 ZoneList<Statement*>* Parser::ParseEagerFunctionBody(
     const AstRawString* function_name, int pos, Variable* fvar,
     Token::Value fvar_init_op, FunctionKind kind, bool* ok) {