[es6] Implement parts of the Reflect object.

[es6] Implement parts of the Reflect object.

- Reflect.deleteProperty
- Reflect.get
- Reflect.has
- Reflect.isExtensible

Reflect.get doesn't support the receiver argument yet, and
some of the others don't support proxies yet.

R=rossberg
BUG=v8:3931
LOG=n

Committed: https://crrev.com/09184acad233389bab5acf12579c4e015d84446b
Cr-Commit-Position: refs/heads/master@{#31146}

[PhistucK, 2015-10-03 18:14:15]

It would be great if you could add BUG=v8:3931. :)

[PhistucK, 2015-10-03 18:15:50]

phistuck@gmail.com changed reviewers:
+ phistuck@gmail.com

[rossberg, 2015-10-05 13:30:00]

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

https://codereview.chromium.org/1379313002/diff/1/test/mjsunit/harmony/reflec...
test/mjsunit/harmony/reflect.js:16: (function testReflectGetOnNonObject() {
`null` always is a good candidate for such a test as well, because its `typeof
null = "object".

https://codereview.chromium.org/1379313002/diff/1/test/mjsunit/harmony/reflec...
test/mjsunit/harmony/reflect.js:39: assertEquals(Reflect.get(tgt, 0),
undefined);
It would be good to parameterise these tests over a set of different kinds of
objects, such as:

- normal objects
- arrays
- typed arrays
- functions
- the global object
- builtins (Object, String)
- wrapped values (Object(6))
- proxies (later)

[neis, 2015-10-07 09:58:33]

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

https://codereview.chromium.org/1379313002/diff/1/test/mjsunit/harmony/reflec...
test/mjsunit/harmony/reflect.js:16: (function testReflectGetOnNonObject() {
On 2015/10/05 13:30:00, rossberg wrote:
> `null` always is a good candidate for such a test as well, because its `typeof
> null = "object".

Done.

https://codereview.chromium.org/1379313002/diff/1/test/mjsunit/harmony/reflec...
test/mjsunit/harmony/reflect.js:39: assertEquals(Reflect.get(tgt, 0),
undefined);
On 2015/10/05 13:29:59, rossberg wrote:
> It would be good to parameterise these tests over a set of different kinds of
> objects, such as:
> 
> - normal objects
> - arrays
> - typed arrays
> - functions
> - the global object
> - builtins (Object, String)
> - wrapped values (Object(6))
> - proxies (later)

Done.

[rossberg, 2015-10-07 10:31:47]

lgtm

https://codereview.chromium.org/1379313002/diff/20001/test/mjsunit/harmony/re...
File test/mjsunit/harmony/reflect.js (right):

https://codereview.chromium.org/1379313002/diff/20001/test/mjsunit/harmony/re...
test/mjsunit/harmony/reflect.js:85: for (var i = 0; i < objects.length; i++) {
Nit:

for (let tgt of objects)

[neis, 2015-10-07 11:12:43]

The CQ bit was checked by neis@chromium.org

[neis, 2015-10-07 11:12:43]

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/1379313002/#ps40001
(title: " ")

[commit-bot: I haz the power, 2015-10-07 11:12:56]

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

[commit-bot: I haz the power, 2015-10-07 11:14:29]

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

[commit-bot: I haz the power, 2015-10-07 11:14:29]

Try jobs failed on following builders:
  v8_presubmit on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/6435)

[rossberg, 2015-10-07 11:15:02]

https://codereview.chromium.org/1379313002/diff/40001/test/mjsunit/harmony/re...
File test/mjsunit/harmony/reflect.js (right):

https://codereview.chromium.org/1379313002/diff/40001/test/mjsunit/harmony/re...
test/mjsunit/harmony/reflect.js:86: var tgt = prepare(obj);
Nit: no need for this extra binding, or 'prepare' returning its argument. ;)

[neis, 2015-10-07 11:24:22]

The CQ bit was checked by neis@chromium.org

[neis, 2015-10-07 11:24:22]

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/1379313002/#ps60001
(title: "Rebase and use strict")

[commit-bot: I haz the power, 2015-10-07 11:24:29]

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

[commit-bot: I haz the power, 2015-10-07 11:45:17]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-10-07 11:45:36]

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