[heap] Let caller figure out target space for evacuation when compacting.

[heap] Let caller figure out target space for evacuation when compacting.

R=hpayer@chromium.org
BUG=chromium:524425
LOG=N

Committed: https://crrev.com/7611c3b98b2b15c00aca318a7d24cd9161fb989d
Cr-Commit-Position: refs/heads/master@{#30742}

[Michael Lippautz, 2015-09-15 09:41:15]

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

[commit-bot: I haz the power, 2015-09-15 09:41:23]

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

[commit-bot: I haz the power, 2015-09-15 10:29:25]

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

[commit-bot: I haz the power, 2015-09-15 10:29:25]

Dry run: This issue passed the CQ dry run.

[Hannes Payer (out of office), 2015-09-15 10:38:42]

lgtm

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

https://codereview.chromium.org/1340253004/diff/1/src/heap/mark-compact.h#new...
src/heap/mark-compact.h:884: void EvacuateLiveObjectsFromPage(Page* p,
PagedSpace* to_space);
Let's rename it to target_space to avoid confusion with the to_space in
SemiSpace.

[Michael Lippautz, 2015-09-15 10:41:10]

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

https://codereview.chromium.org/1340253004/diff/1/src/heap/mark-compact.h#new...
src/heap/mark-compact.h:884: void EvacuateLiveObjectsFromPage(Page* p,
PagedSpace* to_space);
On 2015/09/15 10:38:42, Hannes Payer wrote:
> Let's rename it to target_space to avoid confusion with the to_space in
> SemiSpace.

Done.

[Michael Lippautz, 2015-09-15 10:41:14]

The CQ bit was checked by mlippautz@chromium.org

[Michael Lippautz, 2015-09-15 10:41:14]

The patchset sent to the CQ was uploaded after l-g-t-m from hpayer@chromium.org
Link to the patchset: https://codereview.chromium.org/1340253004/#ps20001
(title: "Address comment")

[commit-bot: I haz the power, 2015-09-15 10:41:17]

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

[commit-bot: I haz the power, 2015-09-15 11:44:51]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-15 11:45:09]

Patchset 2 (id:??) landed as
https://crrev.com/7611c3b98b2b15c00aca318a7d24cd9161fb989d
Cr-Commit-Position: refs/heads/master@{#30742}