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

Unified Diff: chrome/browser/task_manager/child_process_resource_provider.h

Issue 15196003: Create task_manager namespace and wrap classes related to TaskManager with it. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 7 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/task_manager/child_process_resource_provider.h
diff --git a/chrome/browser/task_manager/task_manager_child_process_resource_provider.h b/chrome/browser/task_manager/child_process_resource_provider.h
similarity index 75%
rename from chrome/browser/task_manager/task_manager_child_process_resource_provider.h
rename to chrome/browser/task_manager/child_process_resource_provider.h
index a3452e878cd0100c2920ba1d953567378b0ec20a..78e9683c582dd71b74c8b8789c6543f41db8e573 100644
--- a/chrome/browser/task_manager/task_manager_child_process_resource_provider.h
+++ b/chrome/browser/task_manager/child_process_resource_provider.h
@@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CHROME_BROWSER_TASK_MANAGER_TASK_MANAGER_CHILD_PROCESS_RESOURCE_PROVIDER_H_
-#define CHROME_BROWSER_TASK_MANAGER_TASK_MANAGER_CHILD_PROCESS_RESOURCE_PROVIDER_H_
+#ifndef CHROME_BROWSER_TASK_MANAGER_CHILD_PROCESS_RESOURCE_PROVIDER_H_
+#define CHROME_BROWSER_TASK_MANAGER_CHILD_PROCESS_RESOURCE_PROVIDER_H_
#include <map>
#include <vector>
@@ -13,17 +13,19 @@
#include "content/public/browser/browser_child_process_observer.h"
#include "content/public/browser/notification_registrar.h"
-class TaskManagerChildProcessResource;
-
namespace content {
struct ChildProcessData;
}
-class TaskManagerChildProcessResourceProvider
+namespace task_manager {
+
+class ChildProcessResource;
+
+class ChildProcessResourceProvider
: public TaskManager::ResourceProvider,
public content::BrowserChildProcessObserver {
public:
- explicit TaskManagerChildProcessResourceProvider(TaskManager* task_manager);
+ explicit ChildProcessResourceProvider(TaskManager* task_manager);
virtual TaskManager::Resource* GetResource(int origin_pid,
int render_process_host_id,
@@ -38,7 +40,7 @@ class TaskManagerChildProcessResourceProvider
const content::ChildProcessData& data) OVERRIDE;
private:
- virtual ~TaskManagerChildProcessResourceProvider();
+ virtual ~ChildProcessResourceProvider();
// Retrieves information about the running ChildProcessHosts (performed in the
// IO thread).
@@ -59,19 +61,21 @@ class TaskManagerChildProcessResourceProvider
// Maps the actual resources (the ChildProcessData) to the Task Manager
// resources.
- typedef std::map<base::ProcessHandle, TaskManagerChildProcessResource*>
+ typedef std::map<base::ProcessHandle, ChildProcessResource*>
ChildProcessMap;
ChildProcessMap resources_;
// Maps the pids to the resources (used for quick access to the resource on
// byte read notifications).
- typedef std::map<int, TaskManagerChildProcessResource*> PidResourceMap;
+ typedef std::map<int, ChildProcessResource*> PidResourceMap;
PidResourceMap pid_to_resources_;
// A scoped container for notification registries.
content::NotificationRegistrar registrar_;
- DISALLOW_COPY_AND_ASSIGN(TaskManagerChildProcessResourceProvider);
+ DISALLOW_COPY_AND_ASSIGN(ChildProcessResourceProvider);
};
-#endif // CHROME_BROWSER_TASK_MANAGER_TASK_MANAGER_CHILD_PROCESS_RESOURCE_PROVIDER_H_
+} // namespace task_manager
+
+#endif // CHROME_BROWSER_TASK_MANAGER_CHILD_PROCESS_RESOURCE_PROVIDER_H_

Powered by Google App Engine
This is Rietveld 408576698