[proxies] Implement the Proxy constructor in C++ fully.

[proxies] Implement the Proxy constructor in C++ fully.

The main part of the Proxy constructor was already in C++, there's
actually no point in keeping a JavaScript wrapper.

R=cbruni@chromium.org
BUG=v8:1543
LOG=n

Committed: https://crrev.com/2377170d076fc5d79688ce0a3863cf9179a79bfa
Cr-Commit-Position: refs/heads/master@{#32507}

[Benedikt Meurer, 2015-12-02 11:58:08]

Hey Camillo,

The move (plus fix) we talked about offline.
Please take a look.

Thanks,
Benedikt

[Camillo Bruni, 2015-12-02 12:02:56]

LGTM with nit.

https://codereview.chromium.org/1491893002/diff/1/src/builtins.cc
File src/builtins.cc (right):

https://codereview.chromium.org/1491893002/diff/1/src/builtins.cc#newcode1758
src/builtins.cc:1758: // ES6 section 26.2.1.1 Proxy ( target, handler ) for the
[[Construct]] case.
Could you add section "9.5.15 ProxyCreate(target, handler)" as well?

[Benedikt Meurer, 2015-12-02 12:07:21]

https://codereview.chromium.org/1491893002/diff/1/src/builtins.cc
File src/builtins.cc (right):

https://codereview.chromium.org/1491893002/diff/1/src/builtins.cc#newcode1758
src/builtins.cc:1758: // ES6 section 26.2.1.1 Proxy ( target, handler ) for the
[[Construct]] case.
Moved to dedicated ProxyCreate function.

[Benedikt Meurer, 2015-12-02 12:07:23]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-12-02 12:07:23]

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

[commit-bot: I haz the power, 2015-12-02 12:07:36]

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

[commit-bot: I haz the power, 2015-12-02 12:29:57]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-02 12:30:22]

Description was changed from

==========
[proxies] Implement the Proxy constructor in C++ fully.

The main part of the Proxy constructor was already in C++, there's
actually no point in keeping a JavaScript wrapper.

R=cbruni@chromium.org
BUG=v8:1543
LOG=n
==========

to

==========
[proxies] Implement the Proxy constructor in C++ fully.

The main part of the Proxy constructor was already in C++, there's
actually no point in keeping a JavaScript wrapper.

R=cbruni@chromium.org
BUG=v8:1543
LOG=n

Committed: https://crrev.com/2377170d076fc5d79688ce0a3863cf9179a79bfa
Cr-Commit-Position: refs/heads/master@{#32507}
==========

[commit-bot: I haz the power, 2015-12-02 12:30:23]

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