Implement WebContentsViewChildFrame::TakeFocus.

content/browser/frame_host/render_frame_proxy_host.cc

 // 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.
 
 #include "content/browser/frame_host/render_frame_proxy_host.h"
 
 #include <utility>
 
 #include "base/lazy_instance.h"
 #include "content/browser/bad_message.h"
@@ skipping 341 matching lines @@
                                           int32_t source_routing_id) {
   RenderFrameHostImpl* target_rfh =
       frame_tree_node_->render_manager()->current_frame_host();
 
   // Translate the source RenderFrameHost in this process to its equivalent
   // RenderFrameProxyHost in the target process.  This is needed for continuing
   // the focus traversal from correct place in a parent frame after one of its
   // child frames finishes its traversal.
   RenderFrameHostImpl* source_rfh =
       RenderFrameHostImpl::FromID(GetProcess()->GetID(), source_routing_id);
-  int32_t source_proxy_routing_id = MSG_ROUTING_NONE;
-  if (source_rfh) {
-    RenderFrameProxyHost* source_proxy =
-        source_rfh->frame_tree_node()
-            ->render_manager()
-            ->GetRenderFrameProxyHost(target_rfh->GetSiteInstance());
-    if (source_proxy)
-      source_proxy_routing_id = source_proxy->GetRoutingID();
-  }
-
-  target_rfh->Send(new FrameMsg_AdvanceFocus(target_rfh->GetRoutingID(), type,
-                                             source_proxy_routing_id));
+  target_rfh->AdvanceFocus(
+      type, (source_rfh

[alexmos, 2016/11/22 02:45:27]

Let's keep source_proxy defined as a separate var and pass it here.  I think it
makes the code a bit easier to follow.

[avallee, 2016/11/25 19:39:54]

Done.

+                 ? source_rfh->frame_tree_node()
+                       ->render_manager()
+                       ->GetRenderFrameProxyHost(target_rfh->GetSiteInstance())
+                 : nullptr));
 }
 
 void RenderFrameProxyHost::OnFrameFocused() {
   frame_tree_node_->current_frame_host()->delegate()->SetFocusedFrame(
       frame_tree_node_, GetSiteInstance());
 }
 
 void RenderFrameProxyHost::OnSetHasReceivedUserGesture() {
   frame_tree_node_->current_frame_host()->SetHasReceivedUserGesture();
 }
 
 }  // namespace content