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

Side by Side Diff: chrome/browser/task_manager_resource_providers.h

Issue 345037: Fifth patch in getting rid of caching MessageLoop pointers. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_TASK_MANAGER_RESOURCE_PROVIDERS_H_ 5 #ifndef CHROME_BROWSER_TASK_MANAGER_RESOURCE_PROVIDERS_H_
6 #define CHROME_BROWSER_TASK_MANAGER_RESOURCE_PROVIDERS_H_ 6 #define CHROME_BROWSER_TASK_MANAGER_RESOURCE_PROVIDERS_H_
7 7
8 #include <map> 8 #include <map>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 155 matching lines...) Expand 10 before | Expand all | Expand 10 after
166 std::vector<ChildProcessInfo> existing_child_process_info_; 166 std::vector<ChildProcessInfo> existing_child_process_info_;
167 167
168 private: 168 private:
169 void Add(ChildProcessInfo child_process_info); 169 void Add(ChildProcessInfo child_process_info);
170 void Remove(ChildProcessInfo child_process_info); 170 void Remove(ChildProcessInfo child_process_info);
171 171
172 void AddToTaskManager(ChildProcessInfo child_process_info); 172 void AddToTaskManager(ChildProcessInfo child_process_info);
173 173
174 TaskManager* task_manager_; 174 TaskManager* task_manager_;
175 175
176 MessageLoop* ui_loop_;
177
178 // Maps the actual resources (the ChildProcessInfo) to the Task Manager 176 // Maps the actual resources (the ChildProcessInfo) to the Task Manager
179 // resources. 177 // resources.
180 std::map<ChildProcessInfo, TaskManagerChildProcessResource*> resources_; 178 std::map<ChildProcessInfo, TaskManagerChildProcessResource*> resources_;
181 179
182 // Maps the pids to the resources (used for quick access to the resource on 180 // Maps the pids to the resources (used for quick access to the resource on
183 // byte read notifications). 181 // byte read notifications).
184 std::map<int, TaskManagerChildProcessResource*> pid_to_resources_; 182 std::map<int, TaskManagerChildProcessResource*> pid_to_resources_;
185 183
186 // A scoped container for notification registries. 184 // A scoped container for notification registries.
187 NotificationRegistrar registrar_; 185 NotificationRegistrar registrar_;
(...skipping 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
308 private: 306 private:
309 void AddToTaskManager(ChildProcessInfo child_process_info); 307 void AddToTaskManager(ChildProcessInfo child_process_info);
310 308
311 TaskManager* task_manager_; 309 TaskManager* task_manager_;
312 TaskManagerBrowserProcessResource resource_; 310 TaskManagerBrowserProcessResource resource_;
313 311
314 DISALLOW_COPY_AND_ASSIGN(TaskManagerBrowserProcessResourceProvider); 312 DISALLOW_COPY_AND_ASSIGN(TaskManagerBrowserProcessResourceProvider);
315 }; 313 };
316 314
317 #endif // CHROME_BROWSER_TASK_MANAGER_RESOURCE_PROVIDERS_H_ 315 #endif // CHROME_BROWSER_TASK_MANAGER_RESOURCE_PROVIDERS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698