[heap] Refactor IncrementalMarking::Stop().

[heap] Refactor IncrementalMarking::Stop().

BUG=

Committed: https://crrev.com/87b6e8806f450c3c0ade1ebec668b2323b06e8d4
Cr-Commit-Position: refs/heads/master@{#33593}

[Hannes Payer (out of office), 2016-01-28 15:10:14]

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

[Hannes Payer (out of office), 2016-01-28 16:21:19]

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

[ulan, 2016-01-28 16:41:24]

https://codereview.chromium.org/1649653004/diff/1/src/heap/incremental-markin...
File src/heap/incremental-marking.cc (left):

https://codereview.chromium.org/1649653004/diff/1/src/heap/incremental-markin...
src/heap/incremental-marking.cc:955: p->ClearFlag(Page::RESCAN_ON_EVACUATION);
Without this code, the flag becomes sticky, i.e. large pages will have
RESCAN_ON_EVACUATION after one incremental marking. Is this intended?

[Hannes Payer (out of office), 2016-01-28 16:49:52]

Large pages will never have this flag set. The change is intended.

[ulan, 2016-01-28 16:52:45]

lgtm

[Hannes Payer (out of office), 2016-01-28 16:52:53]

The CQ bit was checked by hpayer@chromium.org

[commit-bot: I haz the power, 2016-01-28 16:52:59]

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

[commit-bot: I haz the power, 2016-01-28 16:58:07]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-28 16:58:37]

Description was changed from

==========
[heap] Refactor IncrementalMarking::Stop().

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

to

==========
[heap] Refactor IncrementalMarking::Stop().

BUG=

Committed: https://crrev.com/87b6e8806f450c3c0ade1ebec668b2323b06e8d4
Cr-Commit-Position: refs/heads/master@{#33593}
==========

[commit-bot: I haz the power, 2016-01-28 16:58:38]

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