[builtins] Use JSObject::New in C++ builtins.

[builtins] Use JSObject::New in C++ builtins.

We already have a proper abstraction for dealing with subclassable
builtins, namely JSObject::New, so we should use that instead of
inlining this rather complex code sequence multiple times.

R=verwaest@chromium.org

Committed: https://crrev.com/528bb4b960aa2b980d5e1716bd0eebf059c131e7
Cr-Commit-Position: refs/heads/master@{#34094}

[Benedikt Meurer, 2016-02-17 17:42:33]

Hey Toon,

Simple refactoring.
Please take a look.

Thanks,
Benedikt

[Toon Verwaest, 2016-02-17 18:04:50]

lgtm

[Benedikt Meurer (Google), 2016-02-17 19:54:52]

The CQ bit was checked by bmeurer@google.com

[commit-bot: I haz the power, 2016-02-17 19:55:03]

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

[commit-bot: I haz the power, 2016-02-17 19:58:32]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-17 19:59:05]

Description was changed from

==========
[builtins] Use JSObject::New in C++ builtins.

We already have a proper abstraction for dealing with subclassable
builtins, namely JSObject::New, so we should use that instead of
inlining this rather complex code sequence multiple times.

R=verwaest@chromium.org
==========

to

==========
[builtins] Use JSObject::New in C++ builtins.

We already have a proper abstraction for dealing with subclassable
builtins, namely JSObject::New, so we should use that instead of
inlining this rather complex code sequence multiple times.

R=verwaest@chromium.org

Committed: https://crrev.com/528bb4b960aa2b980d5e1716bd0eebf059c131e7
Cr-Commit-Position: refs/heads/master@{#34094}
==========

[commit-bot: I haz the power, 2016-02-17 19:59:07]

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