If a RemoteFrame asks to navigate, send an OpenURL IPC

content/browser/frame_host/render_frame_proxy_host.cc

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 35ed28639884b1216268a3f5b84f3171f1a7eb8f..153ea8bb91338c0da4e489fc15ff571e54012b1d 100644
--- a/content/browser/frame_host/render_frame_proxy_host.cc
+++ b/content/browser/frame_host/render_frame_proxy_host.cc
@@ -100,6 +100,14 @@ bool RenderFrameProxyHost::Send(IPC::Message *msg) {
 }
 
 bool RenderFrameProxyHost::OnMessageReceived(const IPC::Message& msg) {
+  bool handled = true;

[Charlie Reis, 2014/09/18 20:22:27]

Let's move this new block to the end of the method.  We usually defer to other
potential handlers first, and then process the message.

(I realize this means render_frame_host_ will intercept it when the
CommitPending hack is in place, but it should work properly without that hack.)

[Nate Chapin, 2014/09/18 22:39:02]

Done.

+  IPC_BEGIN_MESSAGE_MAP(RenderFrameProxyHost, msg)
+    IPC_MESSAGE_HANDLER(FrameHostMsg_OpenURL, OnOpenURL)
+    IPC_MESSAGE_UNHANDLED(handled = false)
+  IPC_END_MESSAGE_MAP()
+  if (handled)
+    return true;
+
   if (cross_process_frame_connector_.get() &&
       cross_process_frame_connector_->OnMessageReceived(msg))
     return true;
@@ -143,4 +151,9 @@ void RenderFrameProxyHost::DisownOpener() {
   Send(new FrameMsg_DisownOpener(GetRoutingID()));
 }
 
+void RenderFrameProxyHost::OnOpenURL(
+    const FrameHostMsg_OpenURL_Params& params) {
+  frame_tree_node_->current_frame_host()->OpenURL(params);
+}
+
 }  // namespace content