Implement WebContentsViewChildFrame::TakeFocus.

content/browser/web_contents/web_contents_view_guest.cc

 // Copyright (c) 2012 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/web_contents/web_contents_view_guest.h"
 
 #include <utility>
 
 #include "build/build_config.h"
 #include "content/browser/browser_plugin/browser_plugin_embedder.h"
@@ skipping 199 matching lines @@
   RenderViewHostDelegateView* view =
       embedder_render_view_host->GetDelegate()->GetDelegateView();
   if (view)
     view->UpdateDragCursor(operation);
 }
 
 void WebContentsViewGuest::GotFocus() {
 }
 
 void WebContentsViewGuest::TakeFocus(bool reverse) {
+  LOG(ERROR) << this << "WebContentsViewChildFrame::TakeFocus: " << reverse;

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

Don't forget to remove.  And by the way, does the fact that this is empty mean
that tab focus traversal doesn't work across <webview> in non-OOPIF mode today?

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

No, ViewHostMsg_TakeFocus is handled by BrowserPluginGuest::OnTakeFocus.

[alexmos, 2016/11/28 23:52:23]

Acknowledged.

 }
 
 void WebContentsViewGuest::ShowContextMenu(RenderFrameHost* render_frame_host,
                                            const ContextMenuParams& params) {
   platform_view_delegate_view_->ShowContextMenu(render_frame_host, params);
 }
 
 void WebContentsViewGuest::StartDragging(
     const DropData& drop_data,
     WebDragOperationsMask ops,
@@ skipping 12 matching lines @@
   if (view) {
     RecordAction(base::UserMetricsAction("BrowserPlugin.Guest.StartDrag"));
     view->StartDragging(
         drop_data, ops, image, image_offset, event_info, source_rwh);
   } else {
     embedder_web_contents->SystemDragEnded(source_rwh);
   }
 }
 
 }  // namespace content