[heap] Disable concurrent marking for cctests that rely on manual GC.

[heap] Disable concurrent marking for cctests that rely on manual GC.

BUG=chromium:694255
Review-Url: https://codereview.chromium.org/2910933002
Cr-Commit-Position: refs/heads/master@{#45570}
Committed: https://chromium.googlesource.com/v8/v8/+/bc32a0a78fb9c125d8e3dc6e770b3f1aad69a31f

[ulan, 2017-05-29 07:35:04]

ulan@chromium.org changed reviewers:
+ mlippautz@chromium.org

[ulan, 2017-05-29 07:35:04]

ptal

[Michael Lippautz, 2017-05-29 07:42:14]

LGTM

are these the same tests that also disable stress incremental marking? If yes,
should we create a utility scope for disabling certain flags?

[Michael Lippautz, 2017-05-29 07:42:15]

LGTM

are these the same tests that also disable stress incremental marking? If yes,
should we create a utility scope for disabling certain flags?

[ulan, 2017-05-29 07:46:02]

On 2017/05/29 07:42:15, Michael Lippautz wrote:
> LGTM
> 
> are these the same tests that also disable stress incremental marking? If yes,
> should we create a utility scope for disabling certain flags?

Some of them work fine with incremental marking, but need control over marking
step timing.
Most of them disable also concurrent_sweeping, stress_compaction, and
stress_incremental_marking.
A scope for these flags is a good idea, will work on that in a separate CL.

[ulan, 2017-05-29 13:59:04]

The CQ bit was checked by ulan@chromium.org

[commit-bot: I haz the power, 2017-05-29 13:59:13]

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-05-29 14:43:06]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496066344901790, "parent_rev":
"8ad8319e23c22ee5583b4f83e174c59c6d3a6c38", "commit_rev":
"bc32a0a78fb9c125d8e3dc6e770b3f1aad69a31f"}

[commit-bot: I haz the power, 2017-05-29 14:43:14]

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

[commit-bot: I haz the power, 2017-05-29 14:43:14]

Description was changed from

==========
[heap] Disable concurrent marking for cctests that rely on manual GC.

BUG=chromium:694255
==========

to

==========
[heap] Disable concurrent marking for cctests that rely on manual GC.

BUG=chromium:694255

Review-Url: https://codereview.chromium.org/2910933002
Cr-Commit-Position: refs/heads/master@{#45570}
Committed:
https://chromium.googlesource.com/v8/v8/+/bc32a0a78fb9c125d8e3dc6e770b3f1aad6...
==========

[commit-bot: I haz the power, 2017-05-29 14:43:15]

Prior attempt to commit was detected, but we were not able to check whether the
issue was successfully committed. Please check Git history manually and re-check
CQ or close this issue as needed.

[commit-bot: I haz the power, 2017-05-29 14:43:15]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/bc32a0a78fb9c125d8e3dc6e770b3f1aad6...