Just finalize incremental marking when marking is done in idle notification.

Just finalize incremental marking when marking is done in idle notification.

BUG=

[Hannes Payer (out of office), 2014-11-25 17:39:05]

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

[ulan, 2014-11-26 08:44:31]

On 2014/11/25 17:39:06, Hannes Payer wrote:

This no longer shrinks new space and can potentially regress committed memory as
in crbug.com/424423

What is the motivation for the change?

[Hannes Payer (out of office), 2014-11-26 10:36:26]

This path is triggered in a scenario where the web page is active and we are
trying to do small marking steps and finalize marking fast. We should not
trigger expensive compaction and shrink the new space. This should only be done
in a idle case with longer idle times.

[ulan, 2014-11-26 10:42:09]

On 2014/11/26 10:36:26, Hannes Payer wrote:
> This path is triggered in a scenario where the web page is active and we are
> trying to do small marking steps and finalize marking fast. We should not
> trigger expensive compaction and shrink the new space. This should only be
done
> in a idle case with longer idle times.

Thanks for explanation, lgtm.

[Hannes Payer (out of office), 2014-11-27 10:57:56]

The CQ bit was checked by hpayer@chromium.org

[commit-bot: I haz the power, 2014-11-27 10:58:04]

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

[commit-bot: I haz the power, 2014-11-27 11:03:20]

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

[commit-bot: I haz the power, 2014-11-27 11:03:21]

Try jobs failed on following builders:
  v8_android_arm_compile_rel on tryserver.v8
(http://build.chromium.org/p/tryserver.v8/builders/v8_android_arm_compile_rel/...)
  v8_linux64_asan_rel on tryserver.v8
(http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel/builds/190)

[Hannes Payer (out of office), 2014-11-27 11:48:19]

The CQ bit was checked by hpayer@chromium.org

[commit-bot: I haz the power, 2014-11-27 11:48:34]

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

[commit-bot: I haz the power, 2014-11-27 12:16:34]

Committed patchset #2 (id:20001)