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

Unified Diff: content/browser/frame_host/render_frame_proxy_host.cc

Issue 268543008: Cross-process iframe accessibility. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix compile error 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_proxy_host.cc
diff --git a/content/browser/frame_host/render_frame_proxy_host.cc b/content/browser/frame_host/render_frame_proxy_host.cc
index acbd80e36f2d1a15a8c36c5826480309b7aa76a4..35ed28639884b1216268a3f5b84f3171f1a7eb8f 100644
--- a/content/browser/frame_host/render_frame_proxy_host.cc
+++ b/content/browser/frame_host/render_frame_proxy_host.cc
@@ -4,6 +4,7 @@
#include "content/browser/frame_host/render_frame_proxy_host.h"
+#include "base/lazy_instance.h"
#include "content/browser/frame_host/cross_process_frame_connector.h"
#include "content/browser/frame_host/frame_tree.h"
#include "content/browser/frame_host/frame_tree_node.h"
@@ -13,16 +14,42 @@
#include "content/browser/renderer_host/render_widget_host_view_base.h"
#include "content/browser/site_instance_impl.h"
#include "content/common/frame_messages.h"
+#include "content/public/browser/browser_thread.h"
#include "ipc/ipc_message.h"
namespace content {
+namespace {
+
+// The (process id, routing id) pair that identifies one RenderFrameProxy.
+typedef std::pair<int32, int32> RenderFrameProxyHostID;
+typedef base::hash_map<RenderFrameProxyHostID, RenderFrameProxyHost*>
+ RoutingIDFrameProxyMap;
+base::LazyInstance<RoutingIDFrameProxyMap> g_routing_id_frame_proxy_map =
+ LAZY_INSTANCE_INITIALIZER;
+
+}
+
+// static
+RenderFrameProxyHost* RenderFrameProxyHost::FromID(int process_id,
+ int routing_id) {
+ DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+ RoutingIDFrameProxyMap* frames = g_routing_id_frame_proxy_map.Pointer();
+ RoutingIDFrameProxyMap::iterator it = frames->find(
+ RenderFrameProxyHostID(process_id, routing_id));
+ return it == frames->end() ? NULL : it->second;
+}
+
RenderFrameProxyHost::RenderFrameProxyHost(SiteInstance* site_instance,
FrameTreeNode* frame_tree_node)
: routing_id_(site_instance->GetProcess()->GetNextRoutingID()),
site_instance_(site_instance),
frame_tree_node_(frame_tree_node) {
GetProcess()->AddRoute(routing_id_, this);
+ CHECK(g_routing_id_frame_proxy_map.Get().insert(
+ std::make_pair(
+ RenderFrameProxyHostID(GetProcess()->GetID(), routing_id_),
+ this)).second);
if (!frame_tree_node_->IsMainFrame() &&
frame_tree_node_->parent()
@@ -45,6 +72,8 @@ RenderFrameProxyHost::~RenderFrameProxyHost() {
Send(new FrameMsg_DeleteProxy(routing_id_));
GetProcess()->RemoveRoute(routing_id_);
+ g_routing_id_frame_proxy_map.Get().erase(
+ RenderFrameProxyHostID(GetProcess()->GetID(), routing_id_));
}
void RenderFrameProxyHost::SetChildRWHView(RenderWidgetHostView* view) {
@@ -114,5 +143,4 @@ void RenderFrameProxyHost::DisownOpener() {
Send(new FrameMsg_DisownOpener(GetRoutingID()));
}
-
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698