Fix keyboard focus for OOPIF-<webview>.

third_party/WebKit/Source/web/WebViewImpl.cpp

Index: third_party/WebKit/Source/web/WebViewImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebViewImpl.cpp b/third_party/WebKit/Source/web/WebViewImpl.cpp
index dbce5b06b65f7e4ff4ef9e4783670e765bcfada8..9715c8df36cfc2d49038937704deba82d4fe3ddc 100644
--- a/third_party/WebKit/Source/web/WebViewImpl.cpp
+++ b/third_party/WebKit/Source/web/WebViewImpl.cpp
@@ -2901,6 +2901,13 @@ void WebViewImpl::focusDocumentView(WebFrame* frame)
     page()->focusController().focusDocumentView(frame->toImplBase()->frame(), false /* notifyEmbedder */);
 }
 
+void WebViewImpl::unfocusDocumentView(WebFrame* frame)
+{
+    if (page()->focusController().focusedFrame() == frame->toImplBase()->frame()) {

[dcheng, 2016/05/16 20:28:58]

Based on earlier comments, I thought we agreed that this would never be called
when this isn't true. Can we just ASSERT() this instead of making it an if()?

[avallee, 2016/05/17 15:13:18]

Sorry, hadn't hit save there. Done.

[avallee, 2016/05/24 20:07:07]

So after test it is very much possible for these to be different. I'm not sure
if it makes sens to move this into the page and always unfocus or keep it with
the if statement saying: "If this frame is focused, it should lose focus now".

+        page()->focusController().focusDocumentView(nullptr /* frame */, false /* notifyEmbedder */);
+    }
+}
+
 void WebViewImpl::setInitialFocus(bool reverse)
 {
     if (!m_page)