Plumb opener information when creating RenderFrames and RenderFrameProxies for subframes.

content/renderer/render_thread_impl.cc

Index: content/renderer/render_thread_impl.cc
diff --git a/content/renderer/render_thread_impl.cc b/content/renderer/render_thread_impl.cc
index acd76fd870e51cb08e8bd00a73a64f74af6d33c7..28366fbcd7f656305ca4753b6fd4a8d5c5c35cd4 100644
--- a/content/renderer/render_thread_impl.cc
+++ b/content/renderer/render_thread_impl.cc
@@ -1626,18 +1626,20 @@ bool RenderThreadImpl::OnControlMessageReceived(const IPC::Message& msg) {
 void RenderThreadImpl::OnCreateNewFrame(FrameMsg_NewFrame_Params params) {
   CompositorDependencies* compositor_deps = this;
   RenderFrameImpl::CreateFrame(
-      params.routing_id, params.parent_routing_id,
-      params.previous_sibling_routing_id, params.proxy_routing_id,
+      params.routing_id, params.proxy_routing_id, params.opener_routing_id,
+      params.parent_routing_id, params.previous_sibling_routing_id,
       params.replication_state, compositor_deps, params.widget_params);
 }
 
 void RenderThreadImpl::OnCreateNewFrameProxy(
     int routing_id,
-    int parent_routing_id,
     int render_view_routing_id,

[nasko, 2015/09/02 17:27:44]

This doesn't match the header file, which seems to have them in the wrong order.

[alexmos, 2015/09/02 22:41:19]

Done.  Thanks for catching this!

+    int opener_routing_id,
+    int parent_routing_id,
     const FrameReplicationState& replicated_state) {
-  RenderFrameProxy::CreateFrameProxy(routing_id, parent_routing_id,
-                                     render_view_routing_id, replicated_state);
+  RenderFrameProxy::CreateFrameProxy(routing_id, render_view_routing_id,
+                                     opener_routing_id, parent_routing_id,
+                                     replicated_state);
 }
 
 void RenderThreadImpl::OnSetZoomLevelForCurrentURL(const std::string& scheme,