Add delegate and factory function for ChildMemoryCoordinator

Add delegate and factory function for ChildMemoryCoordinator

On Android we need to take immediate action when the OS sends
critical memory pressure notifications [1]. This CL adds
ChildMemoryCoordinatorDelegate and a factory function for
ChildMemoryCoordinator to satisfy the requirement without adding
"#if defined(OS_ANDROID)" in various place. In renderer,
RenderThreadImpl becomes a delegate of ChildMemoryCoordinator
so that it can trigger V8 GC.

This CL doesn't implement Java part. Follow-up CLs will implement
that part.

[1] https://docs.google.com/document/d/1X2HglPS9zIOWAzmIdp7ghh0AGQsVa__sJbE_wim3RaY/edit#heading=h.xqzy4prx9j3x

BUG=617492
Committed: https://crrev.com/c4b4afcb34459e5f159667f80c385a06cf64b240
Cr-Commit-Position: refs/heads/master@{#413680}

[bashi, 2016-08-19 02:15:38]

The CQ bit was checked by bashi@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-19 02:16:07]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[bashi, 2016-08-19 03:43:30]

Description was changed from

==========
Add factory function for ChildMemoryCoordinator

There could be platform-dependent implementations in
ChildMemoryCoordinator. For example, it would need to listen
to some events from the OS like OnTrimMemory() on Android.
Instead of adding ifdefs in various places, introduce a factory
function which returns platform specific ChildMemoryCoordinator.

BUG=617492
==========

to

==========
Add delegate and factory function for ChildMemoryCoordinator

On Android we need to take immediate action when the OS sends
critical memory pressure notifications [1]. This CL adds
ChildMemoryCoordinatorDelegate and a factory function for
ChildMemoryCoordinator to satisfy the requirement without adding
"#if defined(OS_ANDROID)" in various place. In renderer,
RenderThreadImpl becomes a delegate of ChildMemoryCoordinator
so that it can trigger V8 GC.

This CL doesn't implement Java part. Follow-up CLs will implement
that part.

[1]
https://docs.google.com/document/d/1X2HglPS9zIOWAzmIdp7ghh0AGQsVa__sJbE_wim3R...

BUG=617492
==========

[bashi, 2016-08-19 03:44:27]

bashi@chromium.org changed reviewers:
+ chrisha@chromium.org, haraken@chromium.org

[bashi, 2016-08-19 03:44:28]

chrisha@: FYI

haraken@: What do you think about this?

https://codereview.chromium.org/2260433006/diff/1/components/memory_coordinat...
File components/memory_coordinator/child/child_memory_coordinator_impl.h (left):

https://codereview.chromium.org/2260433006/diff/1/components/memory_coordinat...
components/memory_coordinator/child/child_memory_coordinator_impl.h:40: }  //
namespace content
This was wrong...

[haraken, 2016-08-19 06:48:23]

LGTM

[bashi, 2016-08-19 08:05:38]

bashi@chromium.org changed reviewers:
+ esprehn@chromium.org

[bashi, 2016-08-19 08:05:38]

esprehn@: could you review content/ ?

[bashi, 2016-08-21 23:24:25]

bashi@chromium.org changed reviewers:
+ piman@chromium.org

[bashi, 2016-08-21 23:24:26]

piman@, could you review content/ ? esprehn@ seems busy.

[piman, 2016-08-22 19:28:53]

lgtm

[bashi, 2016-08-22 23:03:55]

The CQ bit was checked by bashi@chromium.org

[commit-bot: I haz the power, 2016-08-22 23:04:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-22 23:25:27]

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

[commit-bot: I haz the power, 2016-08-22 23:25:29]

Try jobs failed on following builders:
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[bashi, 2016-08-22 23:50:40]

The CQ bit was checked by bashi@chromium.org

[bashi, 2016-08-22 23:50:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, piman@chromium.org
Link to the patchset: https://codereview.chromium.org/2260433006/#ps40001
(title: "override")

[commit-bot: I haz the power, 2016-08-22 23:51:22]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-23 00:57:01]

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

[commit-bot: I haz the power, 2016-08-23 00:57:05]

Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[bashi, 2016-08-23 03:35:35]

The CQ bit was checked by bashi@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-23 03:35:56]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-23 04:00:15]

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

[commit-bot: I haz the power, 2016-08-23 04:00:16]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[bashi, 2016-08-23 04:05:39]

The CQ bit was checked by bashi@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-23 04:06:03]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-23 06:02:36]

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

[commit-bot: I haz the power, 2016-08-23 06:02:37]

Dry run: This issue passed the CQ dry run.

[bashi, 2016-08-23 06:33:40]

The CQ bit was checked by bashi@chromium.org

[bashi, 2016-08-23 06:33:40]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, piman@chromium.org
Link to the patchset: https://codereview.chromium.org/2260433006/#ps80001
(title: "export")

[commit-bot: I haz the power, 2016-08-23 06:34:04]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-23 06:38:32]

Description was changed from

==========
Add delegate and factory function for ChildMemoryCoordinator

On Android we need to take immediate action when the OS sends
critical memory pressure notifications [1]. This CL adds
ChildMemoryCoordinatorDelegate and a factory function for
ChildMemoryCoordinator to satisfy the requirement without adding
"#if defined(OS_ANDROID)" in various place. In renderer,
RenderThreadImpl becomes a delegate of ChildMemoryCoordinator
so that it can trigger V8 GC.

This CL doesn't implement Java part. Follow-up CLs will implement
that part.

[1]
https://docs.google.com/document/d/1X2HglPS9zIOWAzmIdp7ghh0AGQsVa__sJbE_wim3R...

BUG=617492
==========

to

==========
Add delegate and factory function for ChildMemoryCoordinator

On Android we need to take immediate action when the OS sends
critical memory pressure notifications [1]. This CL adds
ChildMemoryCoordinatorDelegate and a factory function for
ChildMemoryCoordinator to satisfy the requirement without adding
"#if defined(OS_ANDROID)" in various place. In renderer,
RenderThreadImpl becomes a delegate of ChildMemoryCoordinator
so that it can trigger V8 GC.

This CL doesn't implement Java part. Follow-up CLs will implement
that part.

[1]
https://docs.google.com/document/d/1X2HglPS9zIOWAzmIdp7ghh0AGQsVa__sJbE_wim3R...

BUG=617492
==========

[commit-bot: I haz the power, 2016-08-23 06:38:35]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-08-23 06:39:47]

Description was changed from

==========
Add delegate and factory function for ChildMemoryCoordinator

On Android we need to take immediate action when the OS sends
critical memory pressure notifications [1]. This CL adds
ChildMemoryCoordinatorDelegate and a factory function for
ChildMemoryCoordinator to satisfy the requirement without adding
"#if defined(OS_ANDROID)" in various place. In renderer,
RenderThreadImpl becomes a delegate of ChildMemoryCoordinator
so that it can trigger V8 GC.

This CL doesn't implement Java part. Follow-up CLs will implement
that part.

[1]
https://docs.google.com/document/d/1X2HglPS9zIOWAzmIdp7ghh0AGQsVa__sJbE_wim3R...

BUG=617492
==========

to

==========
Add delegate and factory function for ChildMemoryCoordinator

On Android we need to take immediate action when the OS sends
critical memory pressure notifications [1]. This CL adds
ChildMemoryCoordinatorDelegate and a factory function for
ChildMemoryCoordinator to satisfy the requirement without adding
"#if defined(OS_ANDROID)" in various place. In renderer,
RenderThreadImpl becomes a delegate of ChildMemoryCoordinator
so that it can trigger V8 GC.

This CL doesn't implement Java part. Follow-up CLs will implement
that part.

[1]
https://docs.google.com/document/d/1X2HglPS9zIOWAzmIdp7ghh0AGQsVa__sJbE_wim3R...

BUG=617492

Committed: https://crrev.com/c4b4afcb34459e5f159667f80c385a06cf64b240
Cr-Commit-Position: refs/heads/master@{#413680}
==========

[commit-bot: I haz the power, 2016-08-23 06:39:48]

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