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

Unified Diff: chrome/renderer/chrome_render_frame_observer.cc

Issue 2566043004: Add renderer memory metrics (Closed)
Patch Set: Created 4 years 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
Index: chrome/renderer/chrome_render_frame_observer.cc
diff --git a/chrome/renderer/chrome_render_frame_observer.cc b/chrome/renderer/chrome_render_frame_observer.cc
index 881ff7768c3a775d170b0105fe74a3adfbecba8a..833e6ce502760775fa3433c8c6aaa01c10e19753 100644
--- a/chrome/renderer/chrome_render_frame_observer.cc
+++ b/chrome/renderer/chrome_render_frame_observer.cc
@@ -25,6 +25,7 @@
#include "chrome/renderer/safe_browsing/phishing_classifier_delegate.h"
#include "components/translate/content/renderer/translate_helper.h"
#include "content/public/renderer/render_frame.h"
+#include "content/public/renderer/render_thread.h"
#include "content/public/renderer/render_view.h"
#include "extensions/common/constants.h"
#include "printing/features/features.h"
@@ -249,6 +250,24 @@ void ChromeRenderFrameObserver::DidFinishLoad() {
Send(new ChromeViewHostMsg_PageHasOSDD(
routing_id(), frame->document().url(), osdd_url));
}
+
+ content::RendererMemoryMetrics memory_metrics;
+ content::RenderThread::Get()->GetRendererMemoryMetrics(&memory_metrics);
haraken 2016/12/12 11:32:27 Can we measure # of tabs in this renderer at this
keishi 2017/01/06 11:08:27 Added Memory.Experimental.Renderer.*.TotalAllocate
+ UMA_HISTOGRAM_MEMORY_KB("Renderer.Memory.DidFinishLoad.PartitionAllocKB",
+ memory_metrics.partition_alloc_kb);
+ UMA_HISTOGRAM_MEMORY_KB("Renderer.Memory.DidFinishLoad.BlinkGCKB",
+ memory_metrics.blink_gc_kb);
+ UMA_HISTOGRAM_MEMORY_MB("Renderer.Memory.DidFinishLoad.MallocMB",
+ memory_metrics.malloc_mb);
+ UMA_HISTOGRAM_MEMORY_KB("Renderer.Memory.DidFinishLoad.DiscardableKB",
+ memory_metrics.discardable_kb);
+ UMA_HISTOGRAM_MEMORY_MB("Renderer.Memory.DidFinishLoad.V8MainThreadIsolateMB",
+ memory_metrics.v8_main_thread_isolate_mb);
+ UMA_HISTOGRAM_MEMORY_MB("Renderer.Memory.DidFinishLoad.TotalAllocatedMB",
+ memory_metrics.total_allocated_mb);
+ UMA_HISTOGRAM_MEMORY_MB(
+ "Renderer.Memory.DidFinishLoad.NonDiscardableTotalAllocatedMB",
+ memory_metrics.non_discardable_total_allocated_mb);
}
void ChromeRenderFrameObserver::DidStartProvisionalLoad() {
« no previous file with comments | « no previous file | chrome/renderer/chrome_render_thread_observer.cc » ('j') | chrome/renderer/chrome_render_thread_observer.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698