Implement JSProxy::GetOwnPropertyDescriptor.

Implement JSProxy::GetOwnPropertyDescriptor.

Next step: expose it to JS and test it.

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

Committed: https://crrev.com/c3688c1aec3a8658271d4d8d248c0b26f7925f9d
Cr-Commit-Position: refs/heads/master@{#32006}

[Jakob Kummerow, 2015-11-13 17:58:58]

Camillo: PTAL.

Georg: FYI.

[Camillo Bruni, 2015-11-16 10:54:06]

lgtm with minor nit.

https://codereview.chromium.org/1439353004/diff/1/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/1439353004/diff/1/src/objects.cc#newcode6814
src/objects.cc:6814: Handle<Object> args[] = {target, it->GetName()};
this time probably a temp var for GetName() would be nice, maybe?

[Jakob Kummerow, 2015-11-16 11:40:34]

https://codereview.chromium.org/1439353004/diff/1/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/1439353004/diff/1/src/objects.cc#newcode6814
src/objects.cc:6814: Handle<Object> args[] = {target, it->GetName()};
On 2015/11/16 10:54:06, cbruni wrote:
> this time probably a temp var for GetName() would be nice, maybe? 

Aiight, as you wish.

[Jakob Kummerow, 2015-11-16 11:40:43]

The CQ bit was checked by jkummerow@chromium.org

[Jakob Kummerow, 2015-11-16 11:40:43]

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/1439353004/#ps20001
(title: "address nit")

[commit-bot: I haz the power, 2015-11-16 11:40:47]

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

[commit-bot: I haz the power, 2015-11-16 11:59:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-16 12:00:05]

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