Cross-process iframe accessibility.

content/browser/frame_host/render_frame_proxy_host.h

 // Copyright 2014 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_PROXY_HOST_H_
 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_PROXY_HOST_H_
 
 #include "base/memory/scoped_ptr.h"
 #include "content/browser/frame_host/render_frame_host_impl.h"
 #include "content/browser/site_instance_impl.h"
@@ skipping 53 matching lines @@
   }
 
   // Initializes the object and creates the RenderFrameProxy in the process
   // for the SiteInstance.
   bool InitRenderFrameProxy();
 
   int GetRoutingID() {
     return routing_id_;
   }
 
+  static RenderFrameProxyHost* FromID(int process_id, int routing_id);

[Charlie Reis, 2014/08/25 20:58:43]

nit: Static method should be first, before the constructor.

[dmazzoni, 2014/08/26 23:31:03]

Done.

+
   SiteInstance* GetSiteInstance() {
     return site_instance_.get();
   }
 
   void SetChildRWHView(RenderWidgetHostView* view);
 
   // TODO(nasko): The following methods should be removed once we don't have a
   // swapped out state on RenderFrameHosts. See https://crbug.com/357747.
   RenderFrameHostImpl* render_frame_host() {
     return render_frame_host_.get();
   }
   RenderViewHostImpl* GetRenderViewHost();
 
   void TakeFrameHostOwnership(
       scoped_ptr<RenderFrameHostImpl> render_frame_host) {
     render_frame_host_ = render_frame_host.Pass();
   }
   scoped_ptr<RenderFrameHostImpl> PassFrameHostOwnership();
 
   // IPC::Sender
   virtual bool Send(IPC::Message* msg) OVERRIDE;
 
   // IPC::Listener
   virtual bool OnMessageReceived(const IPC::Message& msg) OVERRIDE;
 
   CrossProcessFrameConnector* cross_process_frame_connector() {
     return cross_process_frame_connector_.get();
   }
 
+  FrameTreeNode* frame_tree_node() const { return frame_tree_node_; };

[Charlie Reis, 2014/08/25 20:58:43]

nit: This one's fairly fundamental, so let's list it higher up, maybe near
GetSiteInstance().

[dmazzoni, 2014/08/26 23:31:03]

Done.

+
  private:
   // This RenderFrameProxyHost's routing id.
   int routing_id_;
 
   // The SiteInstance this proxy is associated with.
   scoped_refptr<SiteInstance> site_instance_;
 
   // The node in the frame tree where this proxy is located.
   FrameTreeNode* frame_tree_node_;
 
   // When a RenderFrameHost is in a different process from its parent in the
   // frame tree, this class connects its associated RenderWidgetHostView
   // to this RenderFrameProxyHost, which corresponds to the same frame in the
   // parent's renderer process.
   scoped_ptr<CrossProcessFrameConnector> cross_process_frame_connector_;
 
   // TODO(nasko): This can be removed once we don't have a swapped out state on
   // RenderFrameHosts. See https://crbug.com/357747.
   scoped_ptr<RenderFrameHostImpl> render_frame_host_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderFrameProxyHost);
 };
 
 }  // namespace
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_PROXY_HOST_H_