[es6] Fix computed property names in nested literals

[es6] Fix computed property names in nested literals

Make ObjectLiteral::is_simple() false for literals containing computed
property names, which causes IsCompileTimeValue() to return false and
thus force code to be generated for setting up such properties. This
mirrors the handling of '__proto__' in literals.

BUG=v8:4387
LOG=y

Committed: https://crrev.com/233d62f8e306e56d5ed0597b008c205700e744bd
Cr-Commit-Position: refs/heads/master@{#30362}

[adamk, 2015-08-24 19:44:33]

adamk@chromium.org changed reviewers:
+ caitpotter88@gmail.com

[caitp (gmail), 2015-08-24 19:53:29]

On 2015/08/24 19:44:33, adamk wrote:

Looks pretty good to me

[adamk, 2015-08-24 20:20:41]

Does it look good enough to say the magic words? Rietveld is pretty picky :)

[caitp (gmail), 2015-08-24 21:13:06]

On 2015/08/24 20:20:41, adamk wrote:
> Does it look good enough to say the magic words? Rietveld is pretty picky :)

It does lgtm, but i think that word means nothing from non owners

[adamk, 2015-08-24 21:47:41]

adamk@chromium.org changed reviewers:
+ rossberg@chromium.org

[adamk, 2015-08-24 21:47:41]

Ah, thanks. I think I could actually land now (since I'm an owner), but I'll let
Andreas take a look tomorrow.

[adamk, 2015-08-25 18:36:28]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2015-08-25 18:36:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1307943007/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1307943007/1

[commit-bot: I haz the power, 2015-08-25 18:36:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-08-25 18:36:43]

No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[adamk, 2015-08-25 18:39:20]

adamk@chromium.org changed reviewers:
+ littledan@chromium.org
- rossberg@chromium.org

[adamk, 2015-08-25 18:39:20]

+littledan

[adamk, 2015-08-25 20:09:36]

adamk@chromium.org changed reviewers:
+ rossberg@chromium.org

[adamk, 2015-08-25 20:09:49]

and re-adding rossberg, the race is on!

[Dan Ehrenberg, 2015-08-25 20:27:17]

lgtm

Looks pretty straightforward!

[adamk, 2015-08-25 20:41:20]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2015-08-25 20:41:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1307943007/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1307943007/1

[commit-bot: I haz the power, 2015-08-25 21:10:40]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-08-25 21:10:55]

Patchset 1 (id:??) landed as
https://crrev.com/233d62f8e306e56d5ed0597b008c205700e744bd
Cr-Commit-Position: refs/heads/master@{#30362}