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

Unified Diff: chrome/browser/task_manager/extension_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/extension_process_resource_provider.h
diff --git a/chrome/browser/task_manager/task_manager_extension_process_resource_provider.h b/chrome/browser/task_manager/extension_process_resource_provider.h
similarity index 69%
rename from chrome/browser/task_manager/task_manager_extension_process_resource_provider.h
rename to chrome/browser/task_manager/extension_process_resource_provider.h
index 01840b98bc6796c69ae66a38d8fab710683ee9bf..f6abfb76340f87d6bb32e5a81005e2ed54278eed 100644
--- a/chrome/browser/task_manager/task_manager_extension_process_resource_provider.h
+++ b/chrome/browser/task_manager/extension_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_EXTENSION_PROCESS_RESOURCE_PROVIDER_H_
-#define CHROME_BROWSER_TASK_MANAGER_TASK_MANAGER_EXTENSION_PROCESS_RESOURCE_PROVIDER_H_
+#ifndef CHROME_BROWSER_TASK_MANAGER_EXTENSION_PROCESS_RESOURCE_PROVIDER_H_
+#define CHROME_BROWSER_TASK_MANAGER_EXTENSION_PROCESS_RESOURCE_PROVIDER_H_
#include <map>
@@ -12,18 +12,19 @@
#include "content/public/browser/notification_observer.h"
#include "content/public/browser/notification_registrar.h"
-class TaskManagerExtensionProcessResource;
-
namespace content {
class RenderViewHost;
}
-class TaskManagerExtensionProcessResourceProvider
+namespace task_manager {
+
+class ExtensionProcessResource;
+
+class ExtensionProcessResourceProvider
: public TaskManager::ResourceProvider,
public content::NotificationObserver {
public:
- explicit TaskManagerExtensionProcessResourceProvider(
- TaskManager* task_manager);
+ explicit ExtensionProcessResourceProvider(TaskManager* task_manager);
virtual TaskManager::Resource* GetResource(int origin_pid,
int render_process_host_id,
@@ -37,7 +38,7 @@ class TaskManagerExtensionProcessResourceProvider
const content::NotificationDetails& details) OVERRIDE;
private:
- virtual ~TaskManagerExtensionProcessResourceProvider();
+ virtual ~ExtensionProcessResourceProvider();
bool IsHandledByThisProvider(content::RenderViewHost* render_view_host);
void AddToTaskManager(content::RenderViewHost* render_view_host);
@@ -47,8 +48,8 @@ class TaskManagerExtensionProcessResourceProvider
// Maps the actual resources (content::RenderViewHost*) to the Task Manager
// resources.
- typedef std::map<content::RenderViewHost*,
- TaskManagerExtensionProcessResource*> ExtensionRenderViewHostMap;
+ typedef std::map<content::RenderViewHost*, ExtensionProcessResource*>
+ ExtensionRenderViewHostMap;
ExtensionRenderViewHostMap resources_;
// A scoped container for notification registries.
@@ -56,7 +57,9 @@ class TaskManagerExtensionProcessResourceProvider
bool updating_;
- DISALLOW_COPY_AND_ASSIGN(TaskManagerExtensionProcessResourceProvider);
+ DISALLOW_COPY_AND_ASSIGN(ExtensionProcessResourceProvider);
};
-#endif // CHROME_BROWSER_TASK_MANAGER_TASK_MANAGER_EXTENSION_PROCESS_RESOURCE_PROVIDER_H_
+} // namespace task_manager
+
+#endif // CHROME_BROWSER_TASK_MANAGER_EXTENSION_PROCESS_RESOURCE_PROVIDER_H_

Powered by Google App Engine
This is Rietveld 408576698