Wait in heap tests for active concurrent sweeper threads.

Wait in heap tests for active concurrent sweeper threads.


BUG=

Committed: https://code.google.com/p/v8/source/detail?r=13895

[Michael Starzinger, 2013-03-11 09:50:26]

https://codereview.chromium.org/12708003/diff/1/src/mark-compact.h
File src/mark-compact.h (right):

https://codereview.chromium.org/12708003/diff/1/src/mark-compact.h#newcode701
src/mark-compact.h:701: bool sweeping_pending() const {
Drop this in favor of IsConcurrentSweepingInProgress().

https://codereview.chromium.org/12708003/diff/1/test/cctest/test-heap.cc
File test/cctest/test-heap.cc (right):

https://codereview.chromium.org/12708003/diff/1/test/cctest/test-heap.cc#newc...
test/cctest/test-heap.cc:55: if (collector->sweeping_pending()) {
There already is a predicate IsConcurrentSweepingInProgress() for that flag, use
that one instead.

[Hannes Payer (out of office), 2013-03-11 10:14:52]

https://codereview.chromium.org/12708003/diff/1/src/mark-compact.h
File src/mark-compact.h (right):

https://codereview.chromium.org/12708003/diff/1/src/mark-compact.h#newcode701
src/mark-compact.h:701: bool sweeping_pending() const {
On 2013/03/11 09:50:26, Michael Starzinger wrote:
> Drop this in favor of IsConcurrentSweepingInProgress().

Done.

https://codereview.chromium.org/12708003/diff/1/test/cctest/test-heap.cc
File test/cctest/test-heap.cc (right):

https://codereview.chromium.org/12708003/diff/1/test/cctest/test-heap.cc#newc...
test/cctest/test-heap.cc:55: if (collector->sweeping_pending()) {
On 2013/03/11 09:50:26, Michael Starzinger wrote:
> There already is a predicate IsConcurrentSweepingInProgress() for that flag,
use
> that one instead.

Done.

https://codereview.chromium.org/12708003/diff/1/test/cctest/test-heap.cc#newc...
test/cctest/test-heap.cc:55: if (collector->sweeping_pending()) {
On 2013/03/11 09:50:26, Michael Starzinger wrote:
> There already is a predicate IsConcurrentSweepingInProgress() for that flag,
use
> that one instead.

Done.

[Michael Starzinger, 2013-03-11 10:15:36]

LGTM.

[Hannes Payer (out of office), 2013-03-11 10:23:29]

Committed patchset #2 manually as r13895 (presubmit successful).