Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(841)

Unified Diff: components/tracing/common/process_metrics_memory_dump_provider.cc

Issue 2845813005: memory-infra: Plumb platform_private_footprint to service (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | services/resource_coordinator/public/cpp/memory/memory_instrumentation.typemap » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/tracing/common/process_metrics_memory_dump_provider.cc
diff --git a/components/tracing/common/process_metrics_memory_dump_provider.cc b/components/tracing/common/process_metrics_memory_dump_provider.cc
index 3f2abde07f9c7143049f87ab7065c24e39efa490..41a20a68472c21872f42122e6c8bb4e68cee443a 100644
--- a/components/tracing/common/process_metrics_memory_dump_provider.cc
+++ b/components/tracing/common/process_metrics_memory_dump_provider.cc
@@ -671,6 +671,7 @@ bool ProcessMetricsMemoryDumpProvider::DumpProcessTotals(
pmd->process_totals()->set_resident_set_bytes(rss_bytes);
pmd->set_has_process_totals();
pmd->process_totals()->set_peak_resident_set_bytes(peak_rss_bytes);
+ LOG(ERROR) << "! setting rss " << rss_bytes;
Primiano Tucci (use gerrit) 2017/04/28 15:32:26 LOG(ERROR) << "! I don't think you mean to land th
// Returns true even if other metrics failed, since rss is reported.
return true;
« no previous file with comments | « no previous file | services/resource_coordinator/public/cpp/memory/memory_instrumentation.typemap » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698