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

Unified Diff: chrome/browser/renderer_host/chrome_render_message_filter.cc

Issue 1081323003: Convert renderer JS memory usage reporting to use Mojo. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@utility-process-report-js-memory
Patch Set: Rebase and fix tests. Created 5 years, 7 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
Index: chrome/browser/renderer_host/chrome_render_message_filter.cc
diff --git a/chrome/browser/renderer_host/chrome_render_message_filter.cc b/chrome/browser/renderer_host/chrome_render_message_filter.cc
index 4c1c9838ffc425c464b3c9539ea08fecddefa267..66d65b1145d3a4b022de144ba03b2fd4a4a30a21 100644
--- a/chrome/browser/renderer_host/chrome_render_message_filter.cc
+++ b/chrome/browser/renderer_host/chrome_render_message_filter.cc
@@ -79,7 +79,6 @@ bool ChromeRenderMessageFilter::OnMessageReceived(const IPC::Message& message) {
OnResourceTypeStats)
IPC_MESSAGE_HANDLER(ChromeViewHostMsg_UpdatedCacheStats,
OnUpdatedCacheStats)
- IPC_MESSAGE_HANDLER(ChromeViewHostMsg_V8HeapStats, OnV8HeapStats)
IPC_MESSAGE_HANDLER(ChromeViewHostMsg_AllowDatabase, OnAllowDatabase)
IPC_MESSAGE_HANDLER(ChromeViewHostMsg_AllowDOMStorage, OnAllowDOMStorage)
IPC_MESSAGE_HANDLER_DELAY_REPLY(
@@ -161,32 +160,6 @@ void ChromeRenderMessageFilter::OnUpdatedCacheStats(
render_process_id_, stats);
}
-void ChromeRenderMessageFilter::OnV8HeapStats(int v8_memory_allocated,
- int v8_memory_used) {
- if (!BrowserThread::CurrentlyOn(BrowserThread::UI)) {
- BrowserThread::PostTask(
- BrowserThread::UI, FROM_HERE,
- base::Bind(&ChromeRenderMessageFilter::OnV8HeapStats, this,
- v8_memory_allocated, v8_memory_used));
- return;
- }
-
- base::ProcessId renderer_id = peer_pid();
-
-#if defined(ENABLE_TASK_MANAGER)
- TaskManager::GetInstance()->model()->NotifyV8HeapStats(
- renderer_id,
- static_cast<size_t>(v8_memory_allocated),
- static_cast<size_t>(v8_memory_used));
-#endif // defined(ENABLE_TASK_MANAGER)
-
- V8HeapStatsDetails details(v8_memory_allocated, v8_memory_used);
- content::NotificationService::current()->Notify(
- chrome::NOTIFICATION_RENDERER_V8_HEAP_STATS_COMPUTED,
- content::Source<const base::ProcessId>(&renderer_id),
- content::Details<const V8HeapStatsDetails>(&details));
-}
-
void ChromeRenderMessageFilter::OnAllowDatabase(
int render_frame_id,
const GURL& origin_url,

Powered by Google App Engine
This is Rietveld 408576698