Add support for edit commands in OOPIFs.

third_party/WebKit/Source/web/EditorClientImpl.cpp

Index: third_party/WebKit/Source/web/EditorClientImpl.cpp
diff --git a/third_party/WebKit/Source/web/EditorClientImpl.cpp b/third_party/WebKit/Source/web/EditorClientImpl.cpp
index 80b7a2ff31cf2caf14f968f9cdc1fbb6aeb2383a..c93fc041af232fcfaa70e4553494b093caa8e1e2 100644
--- a/third_party/WebKit/Source/web/EditorClientImpl.cpp
+++ b/third_party/WebKit/Source/web/EditorClientImpl.cpp
@@ -73,9 +73,10 @@ bool EditorClientImpl::canPaste(LocalFrame* frame, bool defaultValue) const
     return webFrame->contentSettingsClient()->allowReadFromClipboard(defaultValue);
 }
 
-bool EditorClientImpl::handleKeyboardEvent()
+bool EditorClientImpl::handleKeyboardEvent(LocalFrame* frame)
 {
-    return m_webView->client() && m_webView->client()->handleCurrentKeyboardEvent();
+    WebLocalFrameImpl* webFrame = WebLocalFrameImpl::fromFrame(frame);
+    return webFrame->client() && webFrame->client()->handleCurrentKeyboardEvent();

[lfg, 2016/09/02 17:20:02]

I think the WebLocalFrame client is always non-null. Can you try that?

[alexmos, 2016/09/07 00:03:00]

I thought it can be null in blink unit tests.  There's an example just above in
respondToChangedSelection, and in general the other uses of
WebLocalFrameImpl::client() seem null-checked:
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/web/WebLocalFr...

 }
 
 } // namespace blink