Oilpan: An Oilpan idle GC should follow a V8 major GC

Oilpan: An Oilpan idle GC should follow a V8 major GC

In bug 530064, V8 contexts remain for a long time because Oilpan's GC isn't scheduled
after V8 major GCs. To avoid that from happening, this CL forces an idle GC after
a V8 major GC finishes weak processings.

This CL won't affect performance of micro-benchmarks, since those benchmarks don't have
a chance to schedule idle GCs anyway.

BUG=530064
Committed: https://crrev.com/2e1ab18e2db149eb1b529c1cbf69545a4b21f2e5
git-svn-id: svn://svn.chromium.org/blink/trunk@202055 bbb929c8-8fbe-4397-9dbb-9b2b20218538

[haraken, 2015-09-10 12:45:16]

haraken@chromium.org changed reviewers:
+ keishi@chromium.org, oilpan-reviews@chromium.org, ulan@chromium.org

[haraken, 2015-09-10 12:45:17]

PTAL

(ulan: FYI)

[keishi, 2015-09-10 12:53:39]

LGTM

https://codereview.chromium.org/1334683002/diff/1/Source/platform/heap/Thread...
File Source/platform/heap/ThreadState.h (right):

https://codereview.chromium.org/1334683002/diff/1/Source/platform/heap/Thread...
Source/platform/heap/ThreadState.h:331: void scheduleV8FollowupGCIfNeeded(bool
shouldScheduleIdleGC);
nit: maybe enum is better https://www.chromium.org/blink/coding-style
[names-enum-to-bool]

[haraken, 2015-09-10 13:09:11]

https://codereview.chromium.org/1334683002/diff/1/Source/platform/heap/Thread...
File Source/platform/heap/ThreadState.h (right):

https://codereview.chromium.org/1334683002/diff/1/Source/platform/heap/Thread...
Source/platform/heap/ThreadState.h:331: void scheduleV8FollowupGCIfNeeded(bool
shouldScheduleIdleGC);
On 2015/09/10 12:53:39, keishi wrote:
> nit: maybe enum is better https://www.chromium.org/blink/coding-style
> [names-enum-to-bool]

Done.

[haraken, 2015-09-10 13:09:19]

The CQ bit was checked by haraken@chromium.org

[haraken, 2015-09-10 13:09:20]

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

[commit-bot: I haz the power, 2015-09-10 13:09:34]

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

[commit-bot: I haz the power, 2015-09-10 14:48:23]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=202055

[commit-bot: I haz the power, 2015-09-23 12:10:37]

Patchset 2 (id:??) landed as
https://crrev.com/2e1ab18e2db149eb1b529c1cbf69545a4b21f2e5