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

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

Issue 9701006: Don't build the task manager into Android builds. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Resolve jam's commnents about not ifdef'ing includes Created 8 years, 9 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 | « chrome/browser/prefs/browser_prefs.cc ('k') | chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 57050b9b0e38d55c79721a4abba6ccfa6096f1fc..ccf2cc0891eb8689004955c6605cfc531ee6937a 100644
--- a/chrome/browser/renderer_host/chrome_render_message_filter.cc
+++ b/chrome/browser/renderer_host/chrome_render_message_filter.cc
@@ -189,8 +189,10 @@ void ChromeRenderMessageFilter::OnResourceTypeStats(
static_cast<int>(stats.fonts.size / 1024));
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+#if defined(ENABLE_TASK_MANAGER)
TaskManager::GetInstance()->model()->NotifyResourceTypeStats(
base::GetProcId(peer_handle()), stats);
+#endif // defined(ENABLE_TASK_MANAGER)
}
void ChromeRenderMessageFilter::OnUpdatedCacheStats(
@@ -210,8 +212,10 @@ void ChromeRenderMessageFilter::OnFPS(int routing_id, float fps) {
base::ProcessId renderer_id = base::GetProcId(peer_handle());
+#if defined(ENABLE_TASK_MANAGER)
TaskManager::GetInstance()->model()->NotifyFPS(
renderer_id, routing_id, fps);
+#endif // defined(ENABLE_TASK_MANAGER)
FPSDetails details(routing_id, fps);
content::NotificationService::current()->Notify(
@@ -233,10 +237,12 @@ void ChromeRenderMessageFilter::OnV8HeapStats(int v8_memory_allocated,
base::ProcessId renderer_id = base::GetProcId(peer_handle());
+#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(
« no previous file with comments | « chrome/browser/prefs/browser_prefs.cc ('k') | chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698