Mac: Create a pid->task_t mapping in the browser process.

chrome/browser/task_manager.cc

Index: chrome/browser/task_manager.cc
diff --git a/chrome/browser/task_manager.cc b/chrome/browser/task_manager.cc
index edd7b37552cbc7c4ad5ab2da48b0a99d75a83999..0667f945a02ff4864b1ff16bd121efed8d664d4c 100644
--- a/chrome/browser/task_manager.cc
+++ b/chrome/browser/task_manager.cc
@@ -29,6 +29,10 @@
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_job.h"
 
+#if defined(OS_MACOSX)
+#include "chrome/browser/mach_broker_mac.h"
+#endif
+
 namespace {
 
 // The delay between updates of the information (in ms).
@@ -536,9 +540,8 @@ void TaskManagerModel::AddResource(TaskManager::Resource* resource) {
 #if !defined(OS_MACOSX)
         base::ProcessMetrics::CreateProcessMetrics(process);
 #else
-        // TODO(thakis): Write a port provider that knows the task ports of
-        // child processes.
-        base::ProcessMetrics::CreateProcessMetrics(process, NULL);
+        base::ProcessMetrics::CreateProcessMetrics(process,
+                                                   MachBroker::instance());

[jeremy, 2009/12/19 20:49:10]

Can CreateProcessMetrics call MachBroker::instance() itself internally so you
can remove the ifdef here?  Or is there a case where you need to pass in another
broker somewhere else?

[Mark Mentovai, 2009/12/21 19:23:10]

If we can come up with a way to do this without the #ifdef, it'd be cool.

 #endif
 
     metrics_map_[process] = pm;