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 RenderFrameHostDelegate;
 class RenderProcessHost;
 class RenderViewHostImpl;
 
 class CONTENT_EXPORT RenderFrameHostImpl : public RenderFrameHost {
  public:
   static RenderFrameHostImpl* FromID(int process_id, int routing_id);
 
   virtual ~RenderFrameHostImpl();
@@ skipping 11 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(CrossProcessFrameConnector*);

[Charlie Reis, 2013/12/12 02:42:05]

style: Please name parameters in .h files.

[kenrb, 2013/12/13 18:41:17]

Done.

+
  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,
                       int routing_id,
                       bool is_swapped_out);
 
  private:
   // IPC Message handlers.
   void OnDetach(int64 parent_frame_id, int64 frame_id);
   void OnDidStartProvisionalLoadForFrame(int64 frame_id,
                                          int64 parent_frame_id,
                                          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_;
 
+  // Object that handles messages destined to a child frame in another
+  // process from its parent.
+  // This will move to RenderFrameProxyHost when that class is created.

[Charlie Reis, 2013/12/12 02:42:05]

Does that mean it's only used (i.e., non-NULL) when this RFH is swapped out?

[kenrb, 2013/12/13 18:41:17]

Yes, although it will also be NULL on swapped out RFH's that correspond to
frames that are not direct children (which I don't think we have, yet).

I have made the comment more explicit about that.

+  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_;
   int routing_id_;
   bool is_swapped_out_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderFrameHostImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_