[WeakMemoryCache] Remove dead/live distinction of Resource outside core/fetch

[WeakMemoryCache] Remove dead/live distinction of Resource outside core/fetch

After https://codereview.chromium.org/2411243004/, dead capacities and
distinction between total/dead/live sizes are no longer meaningful.

BUG=603462
Committed: https://crrev.com/be1252e8ca2f4a59cc19f06c61fb795b56f31e8c
Cr-Commit-Position: refs/heads/master@{#435329}

[hiroshige, 2016-10-19 07:28:59]

Description was changed from

==========
[WeakMemoryCache] Remove dead/live distinction of Resource outside core/fetch

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

to

==========
[WeakMemoryCache] Remove dead/live distinction of Resource outside core/fetch

After https://codereview.chromium.org/2411243004/, dead capacities and
distinction between total/dead/live sizes are no longer meaningful.

BUG=603462
==========

[hiroshige, 2016-10-19 07:32:00]

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

[hiroshige, 2016-10-19 08:31:40]

The CQ bit was unchecked by hiroshige@chromium.org

[hiroshige, 2016-10-21 07:05:59]

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

[commit-bot: I haz the power, 2016-10-21 07:06:23]

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-10-21 07:40:57]

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

[commit-bot: I haz the power, 2016-10-21 07:40:59]

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

[hiroshige, 2016-10-24 05:03:08]

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

[commit-bot: I haz the power, 2016-10-24 05:03:20]

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-10-24 05:06:13]

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

[commit-bot: I haz the power, 2016-10-24 05:06:14]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[hiroshige, 2016-10-24 05:31:35]

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

[commit-bot: I haz the power, 2016-10-24 05:31:48]

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-10-24 05:53:12]

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

[commit-bot: I haz the power, 2016-10-24 05:53:13]

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

[bashi, 2016-11-25 07:04:34]

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

[bashi, 2016-11-25 07:04:35]

(drive-by)

https://codereview.chromium.org/2435603002/diff/70001/components/web_cache/br...
File components/web_cache/browser/web_cache_manager.cc (left):

https://codereview.chromium.org/2435603002/diff/70001/components/web_cache/br...
components/web_cache/browser/web_cache_manager.cc:397:
UMA_HISTOGRAM_MEMORY_MB("Cache.ActiveDeadSizeMB",
nit: Shouldn't we deprecate this metric in histograms.xml?

https://codereview.chromium.org/2435603002/diff/70001/components/web_cache/br...
components/web_cache/browser/web_cache_manager.cc:403:
UMA_HISTOGRAM_MEMORY_MB("Cache.InactiveDeadSizeMB",
Same as Cache.ActiveDeadSizeMB

[hiroshige, 2016-11-25 09:03:20]

https://codereview.chromium.org/2435603002/diff/70001/components/web_cache/br...
File components/web_cache/browser/web_cache_manager.cc (left):

https://codereview.chromium.org/2435603002/diff/70001/components/web_cache/br...
components/web_cache/browser/web_cache_manager.cc:397:
UMA_HISTOGRAM_MEMORY_MB("Cache.ActiveDeadSizeMB",
On 2016/11/25 07:04:35, bashi1 wrote:
> nit: Shouldn't we deprecate this metric in histograms.xml?

There are no corresponding entries for these UMAs in histograms.xml...

https://codereview.chromium.org/2435603002/diff/70001/components/web_cache/br...
components/web_cache/browser/web_cache_manager.cc:403:
UMA_HISTOGRAM_MEMORY_MB("Cache.InactiveDeadSizeMB",
On 2016/11/25 07:04:35, bashi1 wrote:
> Same as Cache.ActiveDeadSizeMB

ditto.

[hiroshige, 2016-11-27 08:46:15]

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

[commit-bot: I haz the power, 2016-11-27 08:46:19]

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-11-27 11:37:19]

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

[commit-bot: I haz the power, 2016-11-27 11:37:20]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-11-30 07:31:10]

hiroshige@chromium.org changed reviewers:
+ jochen@chromium.org, mkwst@chromium.org

[hiroshige, 2016-11-30 07:31:11]

PTAL,

mkwst@chromium.org as a ipc/SECURITY_OWNER for the following files:
- chrome/common/render_messages.h
- chrome/common/resource_usage_reporter_type_converters.cc
- components/web_cache/public/interfaces/web_cache.mojom

jochen@chromium.org for the rest of files as a chrome/, components/web_cache/,
third_party/WebKit/public and third_party/WebKit/Source/web OWNER.

[Mike West, 2016-11-30 09:38:35]

Dropping bits from IPC and mojom LGTM.

[hiroshige, 2016-11-30 11:25:53]

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

[commit-bot: I haz the power, 2016-11-30 11:26:12]

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-11-30 12:40:44]

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

[commit-bot: I haz the power, 2016-11-30 12:40:45]

Dry run: This issue passed the CQ dry run.

[jochen (gone - plz use gerrit), 2016-11-30 15:32:22]

lgtm

[hiroshige, 2016-11-30 17:56:08]

The CQ bit was checked by hiroshige@chromium.org

[commit-bot: I haz the power, 2016-11-30 17:56:43]

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-11-30 18:01:21]

CQ is committing da patch.

Bot data: {"patchset_id": 90001, "attempt_start_ts": 1480528568970860,
"parent_rev": "b6b1d319afcf472438924853e6e7d4e9f786869b", "commit_rev":
"fc4737987965d69f612a2e011164cdf23a7efa5e"}

[commit-bot: I haz the power, 2016-11-30 18:03:08]

Description was changed from

==========
[WeakMemoryCache] Remove dead/live distinction of Resource outside core/fetch

After https://codereview.chromium.org/2411243004/, dead capacities and
distinction between total/dead/live sizes are no longer meaningful.

BUG=603462
==========

to

==========
[WeakMemoryCache] Remove dead/live distinction of Resource outside core/fetch

After https://codereview.chromium.org/2411243004/, dead capacities and
distinction between total/dead/live sizes are no longer meaningful.

BUG=603462
==========

[commit-bot: I haz the power, 2016-11-30 18:03:09]

Committed patchset #6 (id:90001)

[commit-bot: I haz the power, 2016-11-30 18:05:30]

Description was changed from

==========
[WeakMemoryCache] Remove dead/live distinction of Resource outside core/fetch

After https://codereview.chromium.org/2411243004/, dead capacities and
distinction between total/dead/live sizes are no longer meaningful.

BUG=603462
==========

to

==========
[WeakMemoryCache] Remove dead/live distinction of Resource outside core/fetch

After https://codereview.chromium.org/2411243004/, dead capacities and
distinction between total/dead/live sizes are no longer meaningful.

BUG=603462

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

[commit-bot: I haz the power, 2016-11-30 18:05:31]

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