[heap] Fix Sweeper::IsSweepingCompleted

[heap] Fix Sweeper::IsSweepingCompleted

R=ulan@chromium.org

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

[Michael Lippautz, 2016-06-08 20:41:10]

Description was changed from

==========
[heap] Fix Sweeper::IsSweepingCompleted

BUG=
==========

to

==========
[heap] Fix Sweeper::IsSweepingCompleted

R=ulan@chromium.org
==========

[Michael Lippautz, 2016-06-08 20:46:28]

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

[Michael Lippautz, 2016-06-08 20:46:28]

PTAL

This has actually been there for a long time (even before my refactoring) and
should not be correct with >1 sweeper tasks.

The call so far guards only "fast" paths, i.e., the more expensive call to
EnsureCompleted(). A future CL needs this one to be correct.

[Michael Lippautz, 2016-06-08 20:46:47]

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

[commit-bot: I haz the power, 2016-06-08 20:46:53]

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

[commit-bot: I haz the power, 2016-06-08 21:12:37]

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

[commit-bot: I haz the power, 2016-06-08 21:12:37]

Dry run: This issue passed the CQ dry run.

[ulan, 2016-06-09 08:58:04]

lgtm

[Michael Lippautz, 2016-06-09 09:00:42]

The CQ bit was checked by mlippautz@chromium.org

[commit-bot: I haz the power, 2016-06-09 09:00:54]

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

[commit-bot: I haz the power, 2016-06-09 09:02:52]

Description was changed from

==========
[heap] Fix Sweeper::IsSweepingCompleted

R=ulan@chromium.org
==========

to

==========
[heap] Fix Sweeper::IsSweepingCompleted

R=ulan@chromium.org
==========

[commit-bot: I haz the power, 2016-06-09 09:02:53]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-09 09:03:29]

Description was changed from

==========
[heap] Fix Sweeper::IsSweepingCompleted

R=ulan@chromium.org
==========

to

==========
[heap] Fix Sweeper::IsSweepingCompleted

R=ulan@chromium.org

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

[commit-bot: I haz the power, 2016-06-09 09:03:30]

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

[Michael Lippautz, 2016-06-09 10:27:05]

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

The reason for reverting is:
https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20TSAN/builds/10009.