Start incremental marking in long idle notification for background tab

Start incremental marking in long idle notification for background tab
disregarding the allocation throughput.

BUG=chromium:506132
LOG=NO

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

[ulan, 2015-07-02 14:03:24]

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

[ulan, 2015-07-02 14:03:24]

PTAL

[ulan, 2015-07-06 08:38:44]

I uploaded new patch-test with new event kBackgroundIdleNotification

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

https://codereview.chromium.org/1213313004/diff/40001/src/heap/memory-reducer...
src/heap/memory-reducer.cc:120: switch (event.type) {
I rewrote this to use the switch statement instead of multiple if else.

The actual change is addition of (case kBackgroundIdleNotification) and (case
kTimer : if (state.started_gcs >= kMaxNumberOfGCs) { ... })

Everything else is the same.

[Hannes Payer (out of office), 2015-07-07 07:49:34]

lgtm

https://codereview.chromium.org/1213313004/diff/40001/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/1213313004/diff/40001/src/heap/heap.cc#newcod...
src/heap/heap.cc:4986: }
Let's make this block a method to keep IdleNotification small and readable.

[ulan, 2015-07-07 11:58:26]

Thanks

https://codereview.chromium.org/1213313004/diff/40001/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/1213313004/diff/40001/src/heap/heap.cc#newcod...
src/heap/heap.cc:4986: }
On 2015/07/07 07:49:34, Hannes Payer wrote:
> Let's make this block a method to keep IdleNotification small and readable.

Done.

[ulan, 2015-07-07 12:06:24]

The CQ bit was checked by ulan@chromium.org

[ulan, 2015-07-07 12:06:25]

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/1213313004/#ps80001
(title: "Rebase")

[commit-bot: I haz the power, 2015-07-07 12:06:37]

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

[commit-bot: I haz the power, 2015-07-07 12:30:57]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-07-07 12:31:10]

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