Take freed handles into account when scheduling idle GCs.

Take freed handles into account when scheduling idle GCs.

BUG=

Committed: https://crrev.com/f2f8001f1b3df3e7f9492222b3f996af5e64d329
Cr-Commit-Position: refs/heads/master@{#28508}

[ulan, 2015-05-20 11:18:20]

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

[ulan, 2015-05-20 11:18:20]

PTAL

[Hannes Payer (out of office), 2015-05-20 11:39:13]

lgtm

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

https://codereview.chromium.org/1145103002/diff/1/src/heap/heap.cc#newcode934
src/heap/heap.cc:934:
gc_idle_time_handler_.NotifyMarkCompact(next_gc_likely_to_collect_more);
Here it would also make sense to take fragmentation into account. We can do that
as a separate CL. Maybe leave a todo here.

[ulan, 2015-05-20 11:45:10]

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

https://codereview.chromium.org/1145103002/diff/1/src/heap/heap.cc#newcode934
src/heap/heap.cc:934:
gc_idle_time_handler_.NotifyMarkCompact(next_gc_likely_to_collect_more);
On 2015/05/20 11:39:13, Hannes Payer wrote:
> Here it would also make sense to take fragmentation into account. We can do
that
> as a separate CL. Maybe leave a todo here.

I wanted to add fragmentation check after landing
https://codereview.chromium.org/1038313003/. I'll keep this in mind.

[ulan, 2015-05-20 11:45:13]

The CQ bit was checked by ulan@chromium.org

[commit-bot: I haz the power, 2015-05-20 11:45:17]

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

[commit-bot: I haz the power, 2015-05-20 12:10:16]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-05-20 12:10:28]

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