Idle old generation limit is used when allocation rate is low.

Idle old generation limit is used when allocation rate is low.

BUG=

Committed: https://crrev.com/953bf90bcbce3b5e8780ea306f3f2cc7dcc7ddf3
Cr-Commit-Position: refs/heads/master@{#28593}

[Hannes Payer (out of office), 2015-05-22 11:57:10]

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

[Hannes Payer (out of office), 2015-05-22 11:57:10]

This simplifies the code, but the idle limit may get removed soon.

[ulan, 2015-05-22 12:04:13]

lgtm!

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

https://codereview.chromium.org/1153763003/diff/1/src/heap/heap.cc#newcode4641
src/heap/heap.cc:4641: // TODO(ulan): Start incremental marking only for large
heaps.
We can remove the todo.

[Hannes Payer (out of office), 2015-05-22 12:08:34]

The CQ bit was checked by hpayer@chromium.org

[Hannes Payer (out of office), 2015-05-22 12:08:34]

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

[commit-bot: I haz the power, 2015-05-22 12:08:45]

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

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

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

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

Try jobs failed on following builders:
  v8_mac_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel/builds/6077)

[Hannes Payer (out of office), 2015-05-22 12:19:06]

The CQ bit was checked by hpayer@chromium.org

[Hannes Payer (out of office), 2015-05-22 12:19:07]

The patchset sent to the CQ was uploaded after l-g-t-m from ulan@chromium.org
Link to the patchset: https://codereview.chromium.org/1153763003/#ps60001
(title: " ")

[commit-bot: I haz the power, 2015-05-22 12:19:14]

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

[Hannes Payer (out of office), 2015-05-22 12:19:46]

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

https://codereview.chromium.org/1153763003/diff/1/src/heap/heap.cc#newcode4641
src/heap/heap.cc:4641: // TODO(ulan): Start incremental marking only for large
heaps.
On 2015/05/22 12:04:13, ulan wrote:
> We can remove the todo.

Done.

[commit-bot: I haz the power, 2015-05-22 12:23:23]

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

[commit-bot: I haz the power, 2015-05-22 12:23:24]

Try jobs failed on following builders:
  v8_mac_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel/builds/6079)

[Hannes Payer (out of office), 2015-05-22 12:37:45]

The CQ bit was checked by hpayer@chromium.org

[commit-bot: I haz the power, 2015-05-22 12:37:57]

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

[commit-bot: I haz the power, 2015-05-22 13:10:25]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-05-22 13:10:40]

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