Perform full GC if incremental marking is completed or marking deque is empty.

Perform full GC if incremental marking is completed or marking deque is empty.

BUG=

Committed: https://crrev.com/8e838f705c32c5a40e4618d47f8feaa4ca41757a
Cr-Commit-Position: refs/heads/master@{#26272}

[Hannes Payer (out of office), 2015-01-26 14:04:49]

hpayer@chromium.org changed reviewers:
+ jochen@chromium.org

[jochen (gone - plz use gerrit), 2015-01-26 14:09:01]

lgtm with question answered

https://codereview.chromium.org/874993003/diff/1/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/874993003/diff/1/src/heap/heap.cc#newcode831
src/heap/heap.cc:831: !mark_compact_collector_.marking_deque_.IsEmpty() &&
!FLAG_gc_global) {
should this check for overflow as well?

[Hannes Payer (out of office), 2015-01-26 14:52:45]

https://codereview.chromium.org/874993003/diff/1/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/874993003/diff/1/src/heap/heap.cc#newcode831
src/heap/heap.cc:831: !mark_compact_collector_.marking_deque_.IsEmpty() &&
!FLAG_gc_global) {
On 2015/01/26 14:09:01, jochen (slow) wrote:
> should this check for overflow as well?

An overflowed marking deque will be full.

[Hannes Payer (out of office), 2015-01-26 14:52:53]

The CQ bit was checked by hpayer@chromium.org

[commit-bot: I haz the power, 2015-01-26 14:53:59]

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

[commit-bot: I haz the power, 2015-01-26 14:55:05]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-01-26 14:55:21]

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