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..8093ad661bf2d45c016460ccb06a244cd0fe11f8 100644
--- a/third_party/WebKit/Source/web/WebViewImpl.cpp
+++ b/third_party/WebKit/Source/web/WebViewImpl.cpp
@@ -2901,6 +2901,14 @@ 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()) {
+        page()->focusController().focusDocumentView(nullptr /* frame */, false /* notifyEmbedder */);
+    }
+    // TODO(avallee): else NOTREACHED() ?

[lfg, 2016/05/06 21:23:33]

No need for this.

[dcheng, 2016/05/09 18:25:08]

Is there ever a case where we'll legitimately call this when focusedFrame() !=
frame->toImplBase()->frame()? Seems like we should just assert.

[avallee, 2016/05/11 18:26:11]

I don't think so, unless the browser and renderer get out of sync.

[avallee, 2016/05/11 18:26:11]

Done.

[dcheng, 2016/05/11 18:46:58]

If this can never happen, why do we need the if (...)?

+}
+
 void WebViewImpl::setInitialFocus(bool reverse)
 {
     if (!m_page)