Adding RenderWidgetHostViewChildFrame for OOPIF view.

content/browser/frame_host/render_frame_host_impl.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_
 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_
 
 #include <string>
 
 #include "base/compiler_specific.h"
 #include "content/common/content_export.h"
 #include "content/public/browser/render_frame_host.h"
 
 class GURL;
 
 namespace base {
 class FilePath;
 }
 
 namespace content {
 
+class CrossProcessFrameConnector;
 class FrameTree;
 class FrameTreeNode;
 class RenderFrameHostDelegate;
 class RenderProcessHost;
 class RenderViewHostImpl;
 
 class CONTENT_EXPORT RenderFrameHostImpl : public RenderFrameHost {
  public:
   static RenderFrameHostImpl* FromID(int process_id, int routing_id);
 
@@ skipping 12 matching lines @@
   RenderProcessHost* GetProcess() const;
   int routing_id() const { return routing_id_; }
   void OnCreateChildFrame(int new_frame_routing_id,
                           int64 parent_frame_id,
                           int64 frame_id,
                           const std::string& frame_name);
 
   RenderViewHostImpl* render_view_host() { return render_view_host_; }
   RenderFrameHostDelegate* delegate() { return delegate_; }
 
+  void SetCrossProcessFrameConnector(

[Charlie Reis, 2013/12/17 01:09:31]

It feels odd to me to have no comments on the public contract.  We should at
least tell people to look at the member declaration for an explanation, but
perhaps we can say something about when this should be called.

[awong, 2013/12/17 02:45:27]

Done.

+      CrossProcessFrameConnector* cross_process_frame_connector);
+
  protected:
   friend class RenderFrameHostFactory;
 
   // TODO(nasko): Remove dependency on RenderViewHost here. RenderProcessHost
   // should be the abstraction needed here, but we need RenderViewHost to pass
   // into WebContentsObserver::FrameDetached for now.
   RenderFrameHostImpl(RenderViewHostImpl* render_view_host,
                       RenderFrameHostDelegate* delegate,
                       FrameTree* frame_tree,
                       FrameTreeNode* frame_tree_node,
@@ skipping 10 matching lines @@
                                          bool main_frame,
                                          const GURL& url);
 
   bool is_swapped_out() { return is_swapped_out_; }
 
   // TODO(nasko): This should be removed and replaced by RenderProcessHost.
   RenderViewHostImpl* render_view_host_;  // Not owned.
 
   RenderFrameHostDelegate* delegate_;
 
+  // |cross_process_frame_connector_| passes messages from an out-of-process
+  // child frame to the parent frame.
+  //
+  // This is only non-NULL when the RenderFrameHost object is
+  //   (a) swapped out
+  //   (b) has a child frame that 

[nasko, 2013/12/17 00:56:52]

This doesn't end properly, nor does it read coherently if you continue with next
line.

[awong, 2013/12/17 02:45:27]

Yep...sorry...I hadn't gotten this far yet.

+  // Object that handles messages destined to a child frame in another
+  // process from its parent. Non-NULL only when this RenderFrameHost is
+  // swapped out and corresponds to a direct child frame of a frame in its
+  // renderer process.
+  // This will move to RenderFrameProxyHost when that class is created.
+  CrossProcessFrameConnector* cross_process_frame_connector_;
+
   // Reference to the whole frame tree that this RenderFrameHost belongs too.
   // Allows this RenderFrameHost to add and remove nodes in response to
   // messages from the renderer requesting DOM manipulation.
   FrameTree* frame_tree_;
 
   // The FrameTreeNode which this RenderFrameHostImpl is hosted in.
   FrameTreeNode* frame_tree_node_;
 
   int routing_id_;
   bool is_swapped_out_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderFrameHostImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_