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

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

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

Committed: https://crrev.com/21a5bfbd2ced6e4a3545bbde712630c0d8734c0b
Cr-Commit-Position: refs/heads/master@{#34411}

[Toon Verwaest, 2016-03-01 17:30:35]

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

https://codereview.chromium.org/1756433002/diff/1/src/execution.cc#newcode184
src/execution.cc:184: if (!constructor->IsConstructor() &&
Move up to the fast-path in the is_construct case.
Simplify the condition; I presume you don't need the is_generator case for
example ...

[jochen (gone - plz use gerrit), 2016-03-01 18:09:57]

ptal

[Toon Verwaest, 2016-03-01 18:17:46]

Lgtm, but what about just moving that block into the fast-path in Invoke rather
than having this lonely check there that duplicates the check in the fast path?

[jochen (gone - plz use gerrit), 2016-03-01 19:40:57]

The CQ bit was checked by jochen@chromium.org

[jochen (gone - plz use gerrit), 2016-03-01 19:40:58]

The patchset sent to the CQ was uploaded after l-g-t-m from
verwaest@chromium.org
Link to the patchset: https://codereview.chromium.org/1756433002/#ps40001
(title: "updates")

[commit-bot: I haz the power, 2016-03-01 19:40:59]

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

[commit-bot: I haz the power, 2016-03-01 19:56:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-01 19:56:19]

Try jobs failed on following builders:
  v8_linux_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_rel_ng/builds/2184)

[jochen (gone - plz use gerrit), 2016-03-01 20:40:26]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2016-03-01 20:40:36]

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

[commit-bot: I haz the power, 2016-03-01 20:58:08]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-01 20:58:53]

Description was changed from

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

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

to

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

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

Committed: https://crrev.com/21a5bfbd2ced6e4a3545bbde712630c0d8734c0b
Cr-Commit-Position: refs/heads/master@{#34411}
==========

[commit-bot: I haz the power, 2016-03-01 20:58:55]

Patchset 3 (id:??) landed as
https://crrev.com/21a5bfbd2ced6e4a3545bbde712630c0d8734c0b
Cr-Commit-Position: refs/heads/master@{#34411}

[Michael Achenbach, 2016-03-02 07:35:13]

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

The reason for reverting is: [Sheriff] Breaks webkit unittests:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui....