[builtins] Fix context for ConstructStub calls into C++.

[builtins] Fix context for ConstructStub calls into C++.

When calling into C++ for a ConstructStub, we need to enter the target
context manually currently, which seems to be too fragile and easy to
forget. So instead of doing that manually, we just always enter the
correct context in the trampoline.

Drive-by-fix: Trivial cleanups for some builtins.

R=cbruni@chromium.org

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

[Benedikt Meurer, 2015-12-26 19:17:11]

The CQ bit was checked by bmeurer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-12-26 19:17:19]

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

[Benedikt Meurer, 2015-12-26 19:17:51]

Hey Camillo,

Simple fix and some cleanup.
Please take a look.

Thanks,
Benedikt

[commit-bot: I haz the power, 2015-12-26 19:52:10]

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

[commit-bot: I haz the power, 2015-12-26 19:52:17]

Dry run: This issue passed the CQ dry run.

[Camillo Bruni, 2015-12-28 08:39:13]

lgtm

[Benedikt Meurer, 2015-12-28 19:30:31]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2015-12-28 19:30:33]

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

[commit-bot: I haz the power, 2015-12-28 19:32:28]

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

[commit-bot: I haz the power, 2015-12-28 19:32:29]

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

[Benedikt Meurer, 2015-12-28 19:45:23]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-12-28 19:45:24]

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/1551473002/#ps20001
(title: "REBASE")

[commit-bot: I haz the power, 2015-12-28 19:45:31]

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

[commit-bot: I haz the power, 2015-12-28 20:17:18]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-28 20:18:13]

Description was changed from

==========
[builtins] Fix context for ConstructStub calls into C++.

When calling into C++ for a ConstructStub, we need to enter the target
context manually currently, which seems to be too fragile and easy to
forget. So instead of doing that manually, we just always enter the
correct context in the trampoline.

Drive-by-fix: Trivial cleanups for some builtins.

R=cbruni@chromium.org
==========

to

==========
[builtins] Fix context for ConstructStub calls into C++.

When calling into C++ for a ConstructStub, we need to enter the target
context manually currently, which seems to be too fragile and easy to
forget. So instead of doing that manually, we just always enter the
correct context in the trampoline.

Drive-by-fix: Trivial cleanups for some builtins.

R=cbruni@chromium.org

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

[commit-bot: I haz the power, 2015-12-28 20:18:14]

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