Allow more scavenges in idle notification by increasing the new space limit distance.

Allow more scavenges in idle notification by increasing the new space limit distance.

BUG=chromium:468554
LOG=n

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

[Hannes Payer (out of office), 2015-03-27 12:43:54]

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

[Erik Corry, 2015-03-27 14:51:50]

erik.corry@gmail.com changed reviewers:
+ erik.corry@gmail.com

[Erik Corry, 2015-03-27 14:52:00]

lgtm

[rmcilroy, 2015-03-27 15:07:50]

https://codereview.chromium.org/1034403002/diff/1/src/heap/gc-idle-time-handl...
File src/heap/gc-idle-time-handler.cc (right):

https://codereview.chromium.org/1034403002/diff/1/src/heap/gc-idle-time-handl...
src/heap/gc-idle-time-handler.cc:136: kMaxScheduledIdleTime * 2;
Pull out 'kMaxScheduledIdleTime * 2' as a variable named something like
estimated_time_till_next_idle_notification or similar and add a comment you are
estimating that it will be 2 * frame time since we might miss an idle period (or
whatever your reasoning for this is).

[Hannes Payer (out of office), 2015-03-30 11:07:15]

https://codereview.chromium.org/1034403002/diff/1/src/heap/gc-idle-time-handl...
File src/heap/gc-idle-time-handler.cc (right):

https://codereview.chromium.org/1034403002/diff/1/src/heap/gc-idle-time-handl...
src/heap/gc-idle-time-handler.cc:136: kMaxScheduledIdleTime * 2;
On 2015/03/27 15:07:50, rmcilroy wrote:
> Pull out 'kMaxScheduledIdleTime * 2' as a variable named something like
> estimated_time_till_next_idle_notification or similar and add a comment you
are
> estimating that it will be 2 * frame time since we might miss an idle period
(or
> whatever your reasoning for this is).

Done.

[rmcilroy, 2015-03-30 11:16:08]

On 2015/03/30 11:07:15, Hannes Payer wrote:
>
https://codereview.chromium.org/1034403002/diff/1/src/heap/gc-idle-time-handl...
> File src/heap/gc-idle-time-handler.cc (right):
> 
>
https://codereview.chromium.org/1034403002/diff/1/src/heap/gc-idle-time-handl...
> src/heap/gc-idle-time-handler.cc:136: kMaxScheduledIdleTime * 2;
> On 2015/03/27 15:07:50, rmcilroy wrote:
> > Pull out 'kMaxScheduledIdleTime * 2' as a variable named something like
> > estimated_time_till_next_idle_notification or similar and add a comment you
> are
> > estimating that it will be 2 * frame time since we might miss an idle period
> (or
> > whatever your reasoning for this is).
> 
> Done.

lgtm, thanks!

[Hannes Payer (out of office), 2015-03-30 11:17:27]

The CQ bit was checked by hpayer@chromium.org

[Hannes Payer (out of office), 2015-03-30 11:17:27]

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

[commit-bot: I haz the power, 2015-03-30 11:17:38]

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

[commit-bot: I haz the power, 2015-03-30 11:39:54]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-30 11:40:23]

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