Perform full GC in background idle notification.

Perform full GC in background idle notification.

BUG=chromium:515873
LOG=NO

Committed: https://crrev.com/9aff1d352d772ddcca8d317c31d6ced895bf115c
Cr-Commit-Position: refs/heads/master@{#29996}

[ulan, 2015-08-03 17:37:40]

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

[ulan, 2015-08-03 17:37:40]

PTAL

[Hannes Payer (out of office), 2015-08-03 19:11:39]

LGTM

https://codereview.chromium.org/1263373002/diff/1/src/heap/memory-reducer.cc
File src/heap/memory-reducer.cc (right):

https://codereview.chromium.org/1263373002/diff/1/src/heap/memory-reducer.cc#...
src/heap/memory-reducer.cc:98:
heap()->CollectAllGarbage(Heap::kReduceMemoryFootprintMask,
Let's leave a comment here why we do a full GCs. Or actually a TODO, it may be
beneficial to incremental marking here.

[ulan, 2015-08-04 11:26:22]

Added TODO and added a check for last GC time.

https://codereview.chromium.org/1263373002/diff/1/src/heap/memory-reducer.cc
File src/heap/memory-reducer.cc (right):

https://codereview.chromium.org/1263373002/diff/1/src/heap/memory-reducer.cc#...
src/heap/memory-reducer.cc:98:
heap()->CollectAllGarbage(Heap::kReduceMemoryFootprintMask,
On 2015/08/03 19:11:39, Hannes Payer wrote:
> Let's leave a comment here why we do a full GCs. Or actually a TODO, it may be
> beneficial to incremental marking here.

Done.

[ulan, 2015-08-04 12:07:04]

The CQ bit was checked by ulan@chromium.org

[ulan, 2015-08-04 12:07:04]

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/1263373002/#ps40001
(title: "Guard full gc")

[commit-bot: I haz the power, 2015-08-04 12:07:07]

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

[commit-bot: I haz the power, 2015-08-04 12:41:59]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-08-04 12:42:16]

Patchset 3 (id:??) landed as
https://crrev.com/9aff1d352d772ddcca8d317c31d6ced895bf115c
Cr-Commit-Position: refs/heads/master@{#29996}