Modify some more GC layout tests to work with Ignition

Modify some more GC layout tests to work with Ignition

In Ignition, dead temporary registers holding call arguments may still
be holding on to old values which should have been GCed. Perform these
calls inside function closures so that their stacks (and these temporary
registers) are cleaned up before calling gc().

This was previously fixed in https://codereview.chromium.org/1972943002
and https://codereview.chromium.org/1950613005, but appeared again when
Ignition added implicit undefined registers and started overwriting
fewer registers in global calls.

BUG=chromium:595672
Review-Url: https://codereview.chromium.org/2813873002
Cr-Commit-Position: refs/heads/master@{#463636}
Committed: https://chromium.googlesource.com/chromium/src/+/2304169a1b293adf38af7f08f325e3d546a06e0c

[Leszek Swirski, 2017-04-11 14:12:20]

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

[commit-bot: I haz the power, 2017-04-11 14:12:34]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Leszek Swirski, 2017-04-11 14:12:58]

leszeks@chromium.org changed reviewers:
+ rmcilroy@chromium.org

[Leszek Swirski, 2017-04-11 14:12:58]

Hey Ross,
As discussed, PTAL.

[rmcilroy, 2017-04-11 14:18:09]

lgtm

[Leszek Swirski, 2017-04-11 14:19:16]

The CQ bit was unchecked by leszeks@chromium.org

[Leszek Swirski, 2017-04-11 14:19:16]

The CQ bit was checked by leszeks@chromium.org

[commit-bot: I haz the power, 2017-04-11 14:19:19]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-11 15:13:29]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491920356740000, "parent_rev":
"0161a7ae408f08b438abb477cd75c67a55f1c225", "commit_rev":
"2304169a1b293adf38af7f08f325e3d546a06e0c"}

[commit-bot: I haz the power, 2017-04-11 15:14:23]

Description was changed from

==========
Modify some more GC layout tests to work with Ignition

In Ignition, dead temporary registers holding call arguments may still
be holding on to old values which should have been GCed. Perform these
calls inside function closures so that their stacks (and these temporary
registers) are cleaned up before calling gc().

This was previously fixed in https://codereview.chromium.org/1972943002
and https://codereview.chromium.org/1950613005, but appeared again when
Ignition added implicit undefined registers and started overwriting
fewer registers in global calls.

BUG=chromium:595672
==========

to

==========
Modify some more GC layout tests to work with Ignition

In Ignition, dead temporary registers holding call arguments may still
be holding on to old values which should have been GCed. Perform these
calls inside function closures so that their stacks (and these temporary
registers) are cleaned up before calling gc().

This was previously fixed in https://codereview.chromium.org/1972943002
and https://codereview.chromium.org/1950613005, but appeared again when
Ignition added implicit undefined registers and started overwriting
fewer registers in global calls.

BUG=chromium:595672

Review-Url: https://codereview.chromium.org/2813873002
Cr-Commit-Position: refs/heads/master@{#463636}
Committed:
https://chromium.googlesource.com/chromium/src/+/2304169a1b293adf38af7f08f325...
==========

[commit-bot: I haz the power, 2017-04-11 15:14:24]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/2304169a1b293adf38af7f08f325...