[heap] Fix finalization of incremental marking race.

[heap] Fix finalization of incremental marking race.

Between requesting finalization of incremental marking and the time where we handle the request in the stack guard, the current full GC may have finished. In that case the stack guard triggers to late and tries to finalize marking in a state where marking is not going on.

Note that a cleaner fix would be to express the finalization phase in a special marking phase. I will do that in a follow-up CL.

BUG=

Committed: https://crrev.com/4f2009eb39a4a69ca61f5ca7aa09c972aeb9479a
Cr-Commit-Position: refs/heads/master@{#32552}

[Hannes Payer (out of office), 2015-12-03 11:34:12]

Description was changed from

==========
[heap] Fix finalization of incremental marking race.

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

to

==========
[heap] Fix finalization of incremental marking race.

Between requesting finalization of incremental marking and the time where we
handle the request in the stack guard, the current full GC may have finished. In
that case the stack guard triggers to late and tries to finalize marking in a
state where marking is not going on.

Note that a cleaner fix would be to express the finalization phase in a special
marking phase. I will do that in a follow-up CL.

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

[Hannes Payer (out of office), 2015-12-03 11:34:12]

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

[ulan, 2015-12-03 11:36:30]

lgtm

[ulan, 2015-12-03 11:36:33]

The CQ bit was checked by ulan@chromium.org

[commit-bot: I haz the power, 2015-12-03 11:36:43]

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

[commit-bot: I haz the power, 2015-12-03 12:09:48]

Description was changed from

==========
[heap] Fix finalization of incremental marking race.

Between requesting finalization of incremental marking and the time where we
handle the request in the stack guard, the current full GC may have finished. In
that case the stack guard triggers to late and tries to finalize marking in a
state where marking is not going on.

Note that a cleaner fix would be to express the finalization phase in a special
marking phase. I will do that in a follow-up CL.

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

to

==========
[heap] Fix finalization of incremental marking race.

Between requesting finalization of incremental marking and the time where we
handle the request in the stack guard, the current full GC may have finished. In
that case the stack guard triggers to late and tries to finalize marking in a
state where marking is not going on.

Note that a cleaner fix would be to express the finalization phase in a special
marking phase. I will do that in a follow-up CL.

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

[commit-bot: I haz the power, 2015-12-03 12:09:48]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-03 12:10:22]

Description was changed from

==========
[heap] Fix finalization of incremental marking race.

Between requesting finalization of incremental marking and the time where we
handle the request in the stack guard, the current full GC may have finished. In
that case the stack guard triggers to late and tries to finalize marking in a
state where marking is not going on.

Note that a cleaner fix would be to express the finalization phase in a special
marking phase. I will do that in a follow-up CL.

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

to

==========
[heap] Fix finalization of incremental marking race.

Between requesting finalization of incremental marking and the time where we
handle the request in the stack guard, the current full GC may have finished. In
that case the stack guard triggers to late and tries to finalize marking in a
state where marking is not going on.

Note that a cleaner fix would be to express the finalization phase in a special
marking phase. I will do that in a follow-up CL.

BUG=

Committed: https://crrev.com/4f2009eb39a4a69ca61f5ca7aa09c972aeb9479a
Cr-Commit-Position: refs/heads/master@{#32552}
==========

[commit-bot: I haz the power, 2015-12-03 12:10:23]

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