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

Unified Diff: chrome/browser/memory_details.cc

Issue 8515027: Define the public version of the browser side RenderProcessHost interface. This interface is not ... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/memory_details.cc
===================================================================
--- chrome/browser/memory_details.cc (revision 110571)
+++ chrome/browser/memory_details.cc (working copy)
@@ -18,11 +18,11 @@
#include "chrome/common/url_constants.h"
#include "content/browser/browser_child_process_host.h"
#include "content/browser/renderer_host/backing_store_manager.h"
-#include "content/browser/renderer_host/render_process_host.h"
#include "content/browser/renderer_host/render_view_host.h"
#include "content/browser/tab_contents/navigation_entry.h"
#include "content/browser/tab_contents/tab_contents.h"
#include "content/public/browser/browser_thread.h"
+#include "content/public/browser/render_process_host.h"
#include "content/public/common/bindings_policy.h"
#include "grit/chromium_strings.h"
#include "grit/generated_resources.h"
@@ -130,10 +130,11 @@
ProcessMemoryInformation& process =
chrome_browser->processes[index];
- for (RenderProcessHost::iterator renderer_iter(
- RenderProcessHost::AllHostsIterator()); !renderer_iter.IsAtEnd();
- renderer_iter.Advance()) {
- RenderProcessHost* render_process_host = renderer_iter.GetCurrentValue();
+ for (content::RenderProcessHost::iterator renderer_iter(
+ content::RenderProcessHost::AllHostsIterator());
+ !renderer_iter.IsAtEnd(); renderer_iter.Advance()) {
+ content::RenderProcessHost* render_process_host =
+ renderer_iter.GetCurrentValue();
DCHECK(render_process_host);
// Ignore processes that don't have a connection, such as crashed tabs.
if (!render_process_host->HasConnection() ||
@@ -142,7 +143,8 @@
}
process.type = ChildProcessInfo::RENDER_PROCESS;
Profile* profile =
- Profile::FromBrowserContext(render_process_host->browser_context());
+ Profile::FromBrowserContext(
+ render_process_host->GetBrowserContext());
ExtensionService* extension_service = profile->GetExtensionService();
extensions::ProcessMap* extension_process_map =
extension_service->process_map();
@@ -154,7 +156,7 @@
// NOTE: This is a bit dangerous. We know that for now, listeners
// are always RenderWidgetHosts. But in theory, they don't
// have to be.
- RenderProcessHost::listeners_iterator iter(
+ content::RenderProcessHost::listeners_iterator iter(
render_process_host->ListenersIterator());
for (; !iter.IsAtEnd(); iter.Advance()) {
const RenderWidgetHost* widget =
@@ -175,12 +177,12 @@
process.renderer_type = ChildProcessInfo::RENDERER_DEVTOOLS;
else
process.renderer_type = ChildProcessInfo::RENDERER_CHROME;
- } else if (extension_process_map->Contains(host->process()->id())) {
+ } else if (extension_process_map->Contains(host->process()->GetID())) {
// For our purposes, don't count processes containing only hosted apps
// as extension processes. See also: crbug.com/102533.
std::set<std::string> extension_ids =
extension_process_map->GetExtensionsInProcess(
- host->process()->id());
+ host->process()->GetID());
for (std::set<std::string>::iterator iter = extension_ids.begin();
iter != extension_ids.end(); ++iter) {
const Extension* extension =
@@ -193,7 +195,7 @@
}
TabContents* contents = host_delegate->GetAsTabContents();
if (!contents) {
- if (extension_process_map->Contains(host->process()->id())) {
+ if (extension_process_map->Contains(host->process()->GetID())) {
const Extension* extension =
extension_service->GetExtensionByURL(url);
if (extension) {
« no previous file with comments | « chrome/browser/geolocation/chrome_geolocation_permission_context_unittest.cc ('k') | chrome/browser/memory_purger.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698