memory-infra: Plumb platform_private_footprint to service

memory-infra: Plumb platform_private_footprint to service

BUG=707019

[Primiano Tucci (use gerrit), 2017-04-28 15:32:25]

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

[Primiano Tucci (use gerrit), 2017-04-28 15:32:26]

Lovely, another struct that we have to keep in sync manually in two places :) (I
know, it's now you)
LGTM % 1 comment (you have a leftover logging).
Although squashing the CL together with the other one might make it easier even
for mojo people to review

https://codereview.chromium.org/2845813005/diff/1/components/tracing/common/p...
File components/tracing/common/process_metrics_memory_dump_provider.cc (right):

https://codereview.chromium.org/2845813005/diff/1/components/tracing/common/p...
components/tracing/common/process_metrics_memory_dump_provider.cc:674:
LOG(ERROR) << "! setting rss " << rss_bytes;
LOG(ERROR) << "! I don't think you mean to land this line!" :P

[hjd, 2017-05-02 11:39:47]

On 2017/04/28 15:32:26, Primiano Tucci wrote:
> Lovely, another struct that we have to keep in sync manually in two places :)
(I
> know, it's now you)
> LGTM % 1 comment (you have a leftover logging).
> Although squashing the CL together with the other one might make it easier
even
> for mojo people to review
> 
>
https://codereview.chromium.org/2845813005/diff/1/components/tracing/common/p...
> File components/tracing/common/process_metrics_memory_dump_provider.cc
(right):
> 
>
https://codereview.chromium.org/2845813005/diff/1/components/tracing/common/p...
> components/tracing/common/process_metrics_memory_dump_provider.cc:674:
> LOG(ERROR) << "! setting rss " << rss_bytes;
> LOG(ERROR) <<&nbsp;"! I don't think you mean to land this line!" :P

How did you find this CL ;)

[hjd, 2017-05-02 11:39:52]

Description was changed from

==========
memory-infra: Plumb platform_private_footprint to service

BUG=707019
==========

to

==========
memory-infra: Plumb platform_private_footprint to service

BUG=707019
==========