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

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

Issue 616133002: Make RenderFrame(Host) own a RenderWidget(Host). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Comments addressed Created 6 years, 3 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_host_manager.h
diff --git a/content/browser/frame_host/render_frame_host_manager.h b/content/browser/frame_host/render_frame_host_manager.h
index 4ca45465a893d4baef082852fc331a7f6a924c31..bfbc35315fdc5f32d7e137e937486b2c042bd10f 100644
--- a/content/browser/frame_host/render_frame_host_manager.h
+++ b/content/browser/frame_host/render_frame_host_manager.h
@@ -45,6 +45,21 @@ class TestWebContents;
class WebUIImpl;
struct NavigationBeforeCommitInfo;
+// Flag arguments for RenderFrameHostManager::CreateRenderFrame.
+enum CreateRenderFrameFlags {
+ // Used for creation of a swapped out RenderFrame.
+ SWAPPED_OUT = 1 << 0,
+ // The new RenderFrame is being created for a navigation of the
+ // top-level frame.
+ FOR_MAIN_FRAME_NAVIGATION = 1 << 1,
+ // The RenderFrame is initially hidden.
+ HIDDEN = 1 << 2,
+ // The RenderFrameHost will have a new RenderWidgetHost created and
+ // attached to it. This is used when the RenderFrameHost is the root
+ // of a subtree of frames that are being rendered in the same process.
+ CREATE_RENDER_WIDGET_HOST = 1 << 3
+};
+
// Manages RenderFrameHosts for a FrameTreeNode. This class acts as a state
// machine to make cross-process navigations in a frame possible.
class CONTENT_EXPORT RenderFrameHostManager : public NotificationObserver {
@@ -257,9 +272,7 @@ class CONTENT_EXPORT RenderFrameHostManager : public NotificationObserver {
// Returns the routing id of the *view* associated with the frame.
int CreateRenderFrame(SiteInstance* instance,
int opener_route_id,
- bool swapped_out,
- bool for_main_frame_navigation,
- bool hidden);
+ int flags);
// Helper method to create and initialize a RenderFrameProxyHost and return
// its routing id.

Powered by Google App Engine
This is Rietveld 408576698