Disable MemoryPressureListener when memory coordinator is enabled

Disable MemoryPressureListener when memory coordinator is enabled

Memory coordinator is a replacement of MemoryPressureListener and
MemoryPressureMonitor. They shouldn't be enabled at a same time
because some components use both MemoryCoordinatorClient and
MemoryPressureListener to handle memory pressure.

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

[bashi, 2016-10-14 02:27:36]

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

[commit-bot: I haz the power, 2016-10-14 02:27:48]

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

[bashi, 2016-10-14 02:59:41]

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

[bashi, 2016-10-14 02:59:42]

PTAL

[bashi, 2016-10-14 03:01:26]

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

[bashi, 2016-10-14 03:01:26]

piman@: could you review an owner of content/ ?

[commit-bot: I haz the power, 2016-10-14 03:11:50]

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

[commit-bot: I haz the power, 2016-10-14 03:11:51]

Dry run: This issue passed the CQ dry run.

[haraken, 2016-10-14 04:06:05]

non-owner LGTM

[piman, 2016-10-14 17:54:06]

lgtm

[chrisha, 2016-10-14 18:16:29]

lgtm

(We should open up a conversation with the memory-infra folks about how MC
should be integrated into devtools. They'll need similar functionality for
simulating pressure levels, etc.)

[bashi, 2016-10-16 23:16:37]

On 2016/10/14 18:16:29, chrisha (slow) wrote:
> lgtm
> 
> (We should open up a conversation with the memory-infra folks about how MC
> should be integrated into devtools. They'll need similar functionality for
> simulating pressure levels, etc.)

It seems that there is no telemetry benchmark which uses
SimulatePressureNotification() except for blink_memory_mobile. I wrote
blink_memory_mobile so I can update it accordingly. That said having devtools
methods for memory coordinator would be great. We may add them at some point.

[bashi, 2016-10-16 23:16:56]

The CQ bit was checked by bashi@chromium.org

[commit-bot: I haz the power, 2016-10-16 23:17: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-10-17 00:01:30]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-17 00:03:18]

Description was changed from

==========
Disable MemoryPressureListener when memory coordinator is enabled

Memory coordinator is a replacement of MemoryPressureListener and
MemoryPressureMonitor. They shouldn't be enabled at a same time
because some components use both MemoryCoordinatorClient and
MemoryPressureListener to handle memory pressure.

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

to

==========
Disable MemoryPressureListener when memory coordinator is enabled

Memory coordinator is a replacement of MemoryPressureListener and
MemoryPressureMonitor. They shouldn't be enabled at a same time
because some components use both MemoryCoordinatorClient and
MemoryPressureListener to handle memory pressure.

BUG=617492

Committed: https://crrev.com/58dd2fac8722507277dad2615a0691e9dd77077d
Cr-Commit-Position: refs/heads/master@{#425602}
==========

[commit-bot: I haz the power, 2016-10-17 00:03:19]

Patchset 1 (id:??) landed as
https://crrev.com/58dd2fac8722507277dad2615a0691e9dd77077d
Cr-Commit-Position: refs/heads/master@{#425602}