[es6] fix nested object literals with computed property names

src/ast.h

Index: src/ast.h
diff --git a/src/ast.h b/src/ast.h
index 1366041387215bc4c018c4b1650d39ff5f6549c4..9b7d95e23dd46e1c07a0cd7c57e6125f61cfe9d9 100644
--- a/src/ast.h
+++ b/src/ast.h
@@ -1489,6 +1489,10 @@ class ObjectLiteral final : public MaterializedLiteral {
   // ObjectLiteral can vary, so num_ids() is not a static method.
   int num_ids() const { return parent_num_ids() + 1 + properties()->length(); }
 
+  bool has_computed_property_names() const {
+    return boilerplate_properties_ == 0 && properties_->length() > 0;

[adamk, 2015/08/24 18:00:53]

I don't think the LHS of this expression is correct: it only works for object
literals whose first property has a computed name. See the way the
number_of_boilerplate_properties is accumulated in ParseObjectLiteral(): it only
stops counting up after it hits the first computed name. I suspect test cases
where the first property is non-computed will keep failing with this patch.

[adamk, 2015/08/24 18:14:29]

Hmm, local testing suggests I'm wrong about this failing, so I'm not totally
sure what's going on.

[adamk, 2015/08/24 18:16:37]

Sorry for the noise, I've confirmed that this is exactly the behavior (forgot to
negate the is_simple check so everything was getting marked non-simple).

+  }
+
   // Object literals need one feedback slot for each non-trivial value, as well
   // as some slots for home objects.
   FeedbackVectorRequirements ComputeFeedbackRequirements(