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

Unified Diff: content/browser/frame_host/render_frame_proxy_host.h

Issue 404613005: Start using RenderFrameProxyHost objects. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Bug fix + ncarter review comments addressed Created 6 years, 5 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: content/browser/frame_host/render_frame_proxy_host.h
diff --git a/content/browser/frame_host/render_frame_proxy_host.h b/content/browser/frame_host/render_frame_proxy_host.h
index 218bcc411df0d4cbbcd4c6e953306baf57aed725..0d021c62cda7b80ce19d12829e6bed4d233aeea8 100644
--- a/content/browser/frame_host/render_frame_proxy_host.h
+++ b/content/browser/frame_host/render_frame_proxy_host.h
@@ -60,9 +60,13 @@ class RenderFrameProxyHost
virtual ~RenderFrameProxyHost();
RenderProcessHost* GetProcess() {
- return site_instance_->GetProcess();
+ return site_instance_->HasProcess() ? site_instance_->GetProcess() : NULL;
Charlie Reis 2014/07/24 22:36:30 When does this matter? I'd like to avoid adding a
nasko 2014/07/25 07:13:20 Initially it was meant to not create a process if
kenrb 2014/07/25 23:42:05 Changed.
}
+ // Initializes the object and creates the RenderFrameProxy in the process
+ // for the SiteInstance.
+ bool InitRenderFrameProxy();
+
int GetRoutingID() {
return routing_id_;
}

Powered by Google App Engine
This is Rietveld 408576698