[es6] Partially implement Reflect.set.

[es6] Partially implement Reflect.set.

Proxies are not properly supported yet.

R=rossberg
BUG=v8:3931
LOG=n

Committed: https://crrev.com/5820adf276e46ab75e84438ffbe3a017efee44fc
Cr-Commit-Position: refs/heads/master@{#31685}

[rossberg, 2015-10-29 16:38:25]

LGTM except for the deactivated test.

https://codereview.chromium.org/1415883007/diff/1/test/mjsunit/harmony/reflec...
File test/mjsunit/harmony/reflect.js (right):

https://codereview.chromium.org/1415883007/diff/1/test/mjsunit/harmony/reflec...
test/mjsunit/harmony/reflect.js:256: // assertFalse(Reflect.set({}, "bla",
null));
Something is missing here...

[neis, 2015-10-29 17:05:51]

On 2015/10/29 16:38:25, rossberg wrote:
> LGTM except for the deactivated test.
> 
>
https://codereview.chromium.org/1415883007/diff/1/test/mjsunit/harmony/reflec...
> File test/mjsunit/harmony/reflect.js (right):
> 
>
https://codereview.chromium.org/1415883007/diff/1/test/mjsunit/harmony/reflec...
> test/mjsunit/harmony/reflect.js:256: // assertFalse(Reflect.set({}, "bla",
> null));
> Something is missing here...

Turned out it was a bug in me.

[neis, 2015-10-30 10:57:04]

Description was changed from

==========
[es6] Partially implement Reflect.set.

Proxies are not properly supported yet.

This depends on issue #1431443003.

R=rossberg
BUG=
==========

to

==========
[es6] Partially implement Reflect.set.

Proxies are not properly supported yet.

R=rossberg
BUG=v8:3931
LOG=n
==========

[neis, 2015-10-30 10:57:14]

The CQ bit was checked by neis@chromium.org

[neis, 2015-10-30 10:57:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
rossberg@chromium.org
Link to the patchset: https://codereview.chromium.org/1415883007/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2015-10-30 10:57:23]

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

[commit-bot: I haz the power, 2015-10-30 11:48:24]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-10-30 11:48:50]

Patchset 4 (id:??) landed as
https://crrev.com/5820adf276e46ab75e84438ffbe3a017efee44fc
Cr-Commit-Position: refs/heads/master@{#31685}