[heap] Add a guard for restarting the memory reducer after mark-compact.

[heap] Add a guard for restarting the memory reducer after mark-compact.

Currently it is possible to get into a cycle of
mark-compact -> memory reducer -> mark-compact -> memory reducer ...
where the memory reducer does not free memory.

This patch ensures that the memory reducer restarts only if the
committed memory increased by sufficient amount after the last run.

BUG=

Committed: https://crrev.com/0a82f091106e5404ac9bcd515ca1d8152bf38604
Cr-Commit-Position: refs/heads/master@{#40457}

[ulan, 2016-10-19 18:33:10]

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

[ulan, 2016-10-19 18:33:10]

ptal

[Hannes Payer (out of office), 2016-10-20 07:21:21]

lgtm

[ulan, 2016-10-20 08:38:20]

The CQ bit was checked by ulan@chromium.org

[commit-bot: I haz the power, 2016-10-20 08:38:28]

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-20 09:04:04]

Committed patchset #1 (id:1)

[davidroutier17, 2016-10-20 09:36:26]

On 2016/10/20 07:21:21, Hannes Payer wrote:
> lgtm

CC

[davidroutier17, 2016-10-21 12:05:05]

davidroutier17@gmail.com changed reviewers:
+ davidroutier17@gmail.com

[davidroutier17, 2016-10-21 12:05:05]

Normal

[ulan, 2016-11-03 16:16:13]

A revert of this CL (patchset #1 id:1) has been created in
https://chromiumcodereview.appspot.com/2472053003/ by ulan@chromium.org.

The reason for reverting is: Speculative revert to see impact on
crbug.com/659531.

[commit-bot: I haz the power, 2016-11-17 22:07:30]

Description was changed from

==========
[heap] Add a guard for restarting the memory reducer after mark-compact.

Currently it is possible to get into a cycle of
mark-compact -> memory reducer -> mark-compact -> memory reducer ...
where the memory reducer does not free memory.

This patch ensures that the memory reducer restarts only if the
committed memory increased by sufficient amount after the last run.

BUG=
==========

to

==========
[heap] Add a guard for restarting the memory reducer after mark-compact.

Currently it is possible to get into a cycle of
mark-compact -> memory reducer -> mark-compact -> memory reducer ...
where the memory reducer does not free memory.

This patch ensures that the memory reducer restarts only if the
committed memory increased by sufficient amount after the last run.

BUG=

Committed: https://crrev.com/0a82f091106e5404ac9bcd515ca1d8152bf38604
Cr-Commit-Position: refs/heads/master@{#40457}
==========

[commit-bot: I haz the power, 2016-11-17 22:07:31]

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