Fix memory leaks from adding GPU memory to task manager

chrome/browser/task_manager/task_manager.h

Index: chrome/browser/task_manager/task_manager.h
diff --git a/chrome/browser/task_manager/task_manager.h b/chrome/browser/task_manager/task_manager.h
index fc47ddc5f181b10a790e948a04c282fee2744e27..056c9933a2db546c95e02b33bccdeb0c8af0b4ec 100644
--- a/chrome/browser/task_manager/task_manager.h
+++ b/chrome/browser/task_manager/task_manager.h
@@ -22,6 +22,7 @@
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebCache.h"
 
 class TaskManagerModel;
+class TaskManagerModelGpuDataManagerObserver;
 
 namespace base {
 class ProcessMetrics;
@@ -572,6 +573,11 @@ class TaskManagerModel : public base::RefCountedThreadSafe<TaskManagerModel> {
   content::GPUVideoMemoryUsageStats video_memory_usage_stats_;
   bool pending_video_memory_usage_stats_update_;
 

[ccameron, 2012/08/20 21:20:02]

The dtor for the observer will call into GpuDataManagerImpl::RemoveObserver --
do we have guarantees of dtor order here?

+  // An observer waiting for video memory usage stats updates from the GPU
+  // process
+  scoped_ptr<TaskManagerModelGpuDataManagerObserver>
+      video_memory_usage_stats_observer_;
+
   // A map that contains the private/shared memory usage of the process. We
   // cache this because the same APIs are called on linux and windows, and
   // because the linux call takes >10ms to complete. This cache is cleared on