Make purge+throttled feature work with memory coordinator

Make purge+throttled feature work with memory coordinator

Purge+throttled (previously called purge+suspend) used to depend on
MemoryCoordinatorClient::OnStateChange() and that's why we couldn't enable
purge+throttled and memory coordinator at the same time. As of
crrev.com/2695923003 all purging logic has moved to
MemoryCoordinatorClient::OnPurgeMemory(), which can be used by both
memory coordinator and purge+throttled feature. Both features don't have to be
mutually exclusive anymore.

BUG=635419
Review-Url: https://codereview.chromium.org/2701473005
Cr-Commit-Position: refs/heads/master@{#451207}
Committed: https://chromium.googlesource.com/chromium/src/+/9c3ce4099b593b5172fe94028c949730f72716ef

[bashi, 2017-02-16 07:41:51]

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

[commit-bot: I haz the power, 2017-02-16 07:42:31]

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

[bashi, 2017-02-16 07:42:53]

bashi@chromium.org changed reviewers:
+ haraken@chromium.org, tasak@google.com

[bashi, 2017-02-16 07:42:53]

tasak@: does this look correct to you? Please let me know if I missed something.

[commit-bot: I haz the power, 2017-02-16 09:39:46]

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

[commit-bot: I haz the power, 2017-02-16 09:39:47]

Dry run: Try jobs failed on following builders:
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)

[tasak, 2017-02-16 12:00:26]

lgtm

[haraken, 2017-02-16 12:00:54]

LGTM

[bashi, 2017-02-16 23:00:22]

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

[bashi, 2017-02-16 23:00:23]

kinuko@: PTAL?

[kinuko, 2017-02-17 01:55:02]

lgtm

[bashi, 2017-02-17 01:55:43]

The CQ bit was checked by bashi@chromium.org

[commit-bot: I haz the power, 2017-02-17 01:57:44]

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, 2017-02-17 04:01:13]

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

[commit-bot: I haz the power, 2017-02-17 04:01:14]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)

[bashi, 2017-02-17 04:12:23]

The CQ bit was checked by bashi@chromium.org

[commit-bot: I haz the power, 2017-02-17 04:13:20]

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, 2017-02-17 04:26:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487304743165450, "parent_rev":
"6ede8b88d716c54c8a0d62b9c3907141137b28f8", "commit_rev":
"9c3ce4099b593b5172fe94028c949730f72716ef"}

[commit-bot: I haz the power, 2017-02-17 04:28:00]

Description was changed from

==========
Make purge+throttled feature work with memory coordinator

Purge+throttled (previously called purge+suspend) used to depend on
MemoryCoordinatorClient::OnStateChange() and that's why we couldn't enable
purge+throttled and memory coordinator at the same time. As of
crrev.com/2695923003 all purging logic has moved to
MemoryCoordinatorClient::OnPurgeMemory(), which can be used by both
memory coordinator and purge+throttled feature. Both features don't have to be
mutually exclusive anymore.

BUG=635419
==========

to

==========
Make purge+throttled feature work with memory coordinator

Purge+throttled (previously called purge+suspend) used to depend on
MemoryCoordinatorClient::OnStateChange() and that's why we couldn't enable
purge+throttled and memory coordinator at the same time. As of
crrev.com/2695923003 all purging logic has moved to
MemoryCoordinatorClient::OnPurgeMemory(), which can be used by both
memory coordinator and purge+throttled feature. Both features don't have to be
mutually exclusive anymore.

BUG=635419

Review-Url: https://codereview.chromium.org/2701473005
Cr-Commit-Position: refs/heads/master@{#451207}
Committed:
https://chromium.googlesource.com/chromium/src/+/9c3ce4099b593b5172fe94028c94...
==========

[commit-bot: I haz the power, 2017-02-17 04:28:01]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/9c3ce4099b593b5172fe94028c94...