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

Unified Diff: chrome/browser/metrics/metrics_service.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
« no previous file with comments | « chrome/browser/metrics/metrics_service.h ('k') | chrome/browser/metrics/tracking_synchronizer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/metrics/metrics_service.cc
===================================================================
--- chrome/browser/metrics/metrics_service.cc (revision 110571)
+++ chrome/browser/metrics/metrics_service.cc (working copy)
@@ -178,8 +178,8 @@
#include "chrome/common/render_messages.h"
#include "content/browser/load_notification_details.h"
#include "content/browser/plugin_service.h"
-#include "content/browser/renderer_host/render_process_host.h"
#include "content/public/browser/notification_service.h"
+#include "content/public/browser/render_process_host.h"
#include "content/public/common/url_fetcher.h"
#include "webkit/plugins/webplugininfo.h"
@@ -525,11 +525,12 @@
break;
case content::NOTIFICATION_RENDERER_PROCESS_CLOSED: {
- RenderProcessHost::RendererClosedDetails* process_details =
- content::Details<RenderProcessHost::RendererClosedDetails>(
- details).ptr();
- RenderProcessHost* host =
- content::Source<RenderProcessHost>(source).ptr();
+ content::RenderProcessHost::RendererClosedDetails* process_details =
+ content::Details<
+ content::RenderProcessHost::RendererClosedDetails>(
+ details).ptr();
+ content::RenderProcessHost* host =
+ content::Source<content::RenderProcessHost>(source).ptr();
LogRendererCrash(
host, process_details->status, process_details->was_alive);
}
@@ -880,7 +881,8 @@
details->StartFetch();
// Collect WebCore cache information to put into a histogram.
- for (RenderProcessHost::iterator i(RenderProcessHost::AllHostsIterator());
+ for (content::RenderProcessHost::iterator i(
+ content::RenderProcessHost::AllHostsIterator());
!i.IsAtEnd(); i.Advance())
i.GetCurrentValue()->Send(new ChromeViewMsg_GetCacheResourceStats());
}
@@ -1239,13 +1241,13 @@
// might be lost due to a crash :-(.
}
-void MetricsService::LogRendererCrash(RenderProcessHost* host,
+void MetricsService::LogRendererCrash(content::RenderProcessHost* host,
base::TerminationStatus status,
bool was_alive) {
- Profile* profile = Profile::FromBrowserContext(host->browser_context());
+ Profile* profile = Profile::FromBrowserContext(host->GetBrowserContext());
ExtensionService* service = profile->GetExtensionService();
bool was_extension_process =
- service && service->process_map()->Contains(host->id());
+ service && service->process_map()->Contains(host->GetID());
if (status == base::TERMINATION_STATUS_PROCESS_CRASHED ||
status == base::TERMINATION_STATUS_ABNORMAL_TERMINATION) {
if (was_extension_process)
« no previous file with comments | « chrome/browser/metrics/metrics_service.h ('k') | chrome/browser/metrics/tracking_synchronizer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698