API: Let v8::Object::DefineOwnProperty use new C++ implementation

API: Let v8::Object::DefineOwnProperty use new C++ implementation

Now that we have a C++ implementation, calling into JS builtins is needlessly inefficient.

Committed: https://crrev.com/406f557721419060c326f6ac3851ae6fb013aaf7
Cr-Commit-Position: refs/heads/master@{#31637}

[Jakob Kummerow, 2015-10-28 16:14:26]

Description was changed from

==========
API: Let v8::Object::DefineOwnProperty use new C++ implementation

Now that we have a C++ implementation, calling into JS code is needlessly
inefficient.
==========

to

==========
API: Let v8::Object::DefineOwnProperty use new C++ implementation

Now that we have a C++ implementation, calling into JS builtins is needlessly
inefficient.
==========

[Jakob Kummerow, 2015-10-28 16:14:40]

jkummerow@chromium.org changed reviewers:
+ cbruni@chromium.org

[Jakob Kummerow, 2015-10-28 16:14:41]

PTAL.

[Camillo Bruni, 2015-10-28 16:21:53]

lgtm

[Jakob Kummerow, 2015-10-28 16:23:02]

The CQ bit was checked by jkummerow@chromium.org

[commit-bot: I haz the power, 2015-10-28 16:23:07]

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

[commit-bot: I haz the power, 2015-10-28 16:48:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-10-28 16:49:25]

Patchset 1 (id:??) landed as
https://crrev.com/406f557721419060c326f6ac3851ae6fb013aaf7
Cr-Commit-Position: refs/heads/master@{#31637}