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

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

Issue 972313002: Make <webview> use out-of-process iframe architecture. (Closed) Base URL: ssh://saopaulo.wat/mnt/dev/shared/src@testoopif2z-better-chrome
Patch Set: Rebase after swapped out changes major rework with RFP Created 5 years, 6 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 9b28575a1ab9c3930ec268cdd26f6c9d39319237..260c795a2e85bc66e072450272d18d55f2ee423f 100644
--- a/content/browser/frame_host/render_frame_host_manager.h
+++ b/content/browser/frame_host/render_frame_host_manager.h
@@ -177,6 +177,13 @@ class CONTENT_EXPORT RenderFrameHostManager : public NotificationObserver {
// hidden state.
virtual bool IsHidden() = 0;
+ // Returns the FrameTreeNode id of the outer WebContents that is hosting
+ // this (inner) WebContents.
+ // This FrameTreeNode is the frame that represents this inner WebContents
+ // within the outer WebContents.
+ // Returns -1 if this WebContents does not have an outer WebContents.
+ virtual int GetOuterDelegateFrameTreeNodeID() = 0;
+
protected:
virtual ~Delegate() {}
};
@@ -222,7 +229,25 @@ class CONTENT_EXPORT RenderFrameHostManager : public NotificationObserver {
// there is no current one.
RenderWidgetHostView* GetRenderWidgetHostView() const;
+ // Returns the RenderWidgetHost of outer WebContents that can be used to fetch
+ // the last keyboard event.
+ // TODO(lazyboy): This can be removed once input events are sent directly to
+ // remote frames.
+ RenderWidgetHostImpl* GetOuterRenderWidgetHostForKeyboardInput();
+
RenderFrameProxyHost* GetProxyToParent();
+ RenderFrameProxyHost* GetProxyToOuterDelegate();
+
+ // Returns whether this manager belongs to a FrameTreeNode that is a main
+ // frame in an inner WebContents.
+ //
+ // TODO(lazyboy): Make this work correctly for subframes inside inner
+ // WebContents too.
+ bool ForInnerDelegate();
+
+ // Removes the FrameTreeNode in the outer WebContents that represents this
+ // FrameTreeNode.
+ void RemoveOuterDelegateFrame();
// Returns the pending RenderFrameHost, or NULL if there is no pending one.
RenderFrameHostImpl* pending_frame_host() const {
@@ -422,6 +447,15 @@ class CONTENT_EXPORT RenderFrameHostManager : public NotificationObserver {
RenderViewHostImpl* render_view_host,
SiteInstance* instance);
+ // Called on the RFHM of the inner WebContents to create a proxy for this
+ // inner WebContents's frame in its outer WebContents's process.
+ void CreateOuterDelegateProxy(SiteInstance* outer_contents_site_instance,
+ RenderFrameHostImpl* render_frame_host);
+
+ // Sets the child RWHV for this frame, which much be part of an inner
+ // WebContents.
+ void SetRWHViewForInnerContents(RenderWidgetHostView* child_rwhv);
+
private:
friend class FrameTreeVisualizer;
friend class NavigatorTestWithBrowserSideNavigation;

Powered by Google App Engine
This is Rietveld 408576698