Remove runtime flags for Proxy and Reflect

Remove runtime flags for Proxy and Reflect

Both of them shipped in Chrome 49 without incident.

Also move relevant tests from harmony/ to es6/.

Committed: https://crrev.com/249bf75e0c99c0fee36cfbd09cad428129edd1c1
Cr-Commit-Position: refs/heads/master@{#34964}

[adamk, 2016-03-18 20:24:18]

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

[adamk, 2016-03-18 20:26:19]

https://codereview.chromium.org/1815773002/diff/1/test/mjsunit/es6/proxies-fu...
File test/mjsunit/es6/proxies-function.js (right):

https://codereview.chromium.org/1815773002/diff/1/test/mjsunit/es6/proxies-fu...
test/mjsunit/es6/proxies-function.js:31: // TODO(neis): These tests are
temporarily commented out because of ongoing
Seems like it would be good to get this test back in working order. I also
suspect it might have caught the "shouldn't be able to call Proxies that wrap
non-callables".

[neis, 2016-03-21 09:40:03]

lgtm


On 2016/03/18 20:26:19, adamk wrote:
>
https://codereview.chromium.org/1815773002/diff/1/test/mjsunit/es6/proxies-fu...
> File test/mjsunit/es6/proxies-function.js (right):
> 
>
https://codereview.chromium.org/1815773002/diff/1/test/mjsunit/es6/proxies-fu...
> test/mjsunit/es6/proxies-function.js:31: // TODO(neis): These tests are
> temporarily commented out because of ongoing
> Seems like it would be good to get this test back in working order. I also
> suspect it might have caught the "shouldn't be able to call Proxies that wrap
> non-callables".

Thanks for spotting, I will have a look.

[adamk, 2016-03-21 18:55:19]

The CQ bit was checked by adamk@chromium.org

[adamk, 2016-03-21 18:55:20]

The patchset sent to the CQ was uploaded after l-g-t-m from neis@chromium.org
Link to the patchset: https://codereview.chromium.org/1815773002/#ps20001
(title: "Rebased")

[commit-bot: I haz the power, 2016-03-21 18:55:29]

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

[commit-bot: I haz the power, 2016-03-21 18:59:00]

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

[commit-bot: I haz the power, 2016-03-21 18:59:01]

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

[adamk, 2016-03-21 18:59:40]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2016-03-21 18:59:52]

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

[commit-bot: I haz the power, 2016-03-21 19:39:22]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-21 19:40:11]

Description was changed from

==========
Remove runtime flags for Proxy and Reflect

Both of them shipped in Chrome 49 without incident.

Also move relevant tests from harmony/ to es6/.
==========

to

==========
Remove runtime flags for Proxy and Reflect

Both of them shipped in Chrome 49 without incident.

Also move relevant tests from harmony/ to es6/.

Committed: https://crrev.com/249bf75e0c99c0fee36cfbd09cad428129edd1c1
Cr-Commit-Position: refs/heads/master@{#34964}
==========

[commit-bot: I haz the power, 2016-03-21 19:40:12]

Patchset 2 (id:??) landed as
https://crrev.com/249bf75e0c99c0fee36cfbd09cad428129edd1c1
Cr-Commit-Position: refs/heads/master@{#34964}