Reland "[api] Don't go to javascript to construct API functions"

Reland "[api] Don't go to javascript to construct API functions"

BUG=
R=verwaest@chromium.org

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

[jochen (gone - plz use gerrit), 2016-03-02 18:15:53]

jochen@chromium.org changed reviewers:
+ bmeurer@chromium.org

[jochen (gone - plz use gerrit), 2016-03-02 18:15:54]

ptal

https://codereview.chromium.org/1756973002/diff/1/src/execution.cc
File src/execution.cc (right):

https://codereview.chromium.org/1756973002/diff/1/src/execution.cc#newcode66
src/execution.cc:66: if (is_construct && !function->IsConstructor()) {
in the original CL, the is_construct && was missing. Added a test for that case.

[Toon Verwaest, 2016-03-04 13:41:17]

lgtm

[jochen (gone - plz use gerrit), 2016-03-04 13:41:37]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2016-03-04 13:41:48]

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

[commit-bot: I haz the power, 2016-03-04 14:03:12]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-04 14:04:00]

Description was changed from

==========
Reland "[api] Don't go to javascript to construct API functions"

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

to

==========
Reland "[api] Don't go to javascript to construct API functions"

BUG=
R=verwaest@chromium.org

Committed: https://crrev.com/d5d4f0b80f585878130bffb40ff8e960646823d8
Cr-Commit-Position: refs/heads/master@{#34496}
==========

[commit-bot: I haz the power, 2016-03-04 14:04:00]

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

[Michael Achenbach, 2016-03-04 15:54:42]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1767663002/ by machenbach@chromium.org.

The reason for reverting is: [Sheriff] Breaks blink tests. Please fix upstream
first.
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui....

[Michael Achenbach, 2016-03-04 15:56:01]

Actually the failing test is a timeout - so not sure about "fix upstream".