Introduce WebInputMethodController to blink

third_party/WebKit/Source/web/WebViewImpl.cpp

 /*
  * Copyright (C) 2011, 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 155 matching lines @@
 #include "web/LinkHighlightImpl.h"
 #include "web/PageOverlay.h"
 #include "web/PrerendererClientImpl.h"
 #include "web/ResizeViewportAnchor.h"
 #include "web/RotationViewportAnchor.h"
 #include "web/SpeechRecognitionClientProxy.h"
 #include "web/StorageQuotaClientImpl.h"
 #include "web/ValidationMessageClientImpl.h"
 #include "web/WebDevToolsAgentImpl.h"
 #include "web/WebInputEventConversion.h"
+#include "web/WebInputMethodControllerImpl.h"
 #include "web/WebLocalFrameImpl.h"
 #include "web/WebPagePopupImpl.h"
 #include "web/WebPluginContainerImpl.h"
 #include "web/WebRemoteFrameImpl.h"
 #include "web/WebSettingsImpl.h"
 #include "wtf/AutoReset.h"
 #include "wtf/CurrentTime.h"
 #include "wtf/PtrUtil.h"
 #include "wtf/RefPtr.h"
 #include <memory>
@@ skipping 2117 matching lines @@
                 focusedFrame->inputMethodController().finishComposingText(InputMethodController::KeepSelection);
 
                 if (autofillClient)
                     autofillClient->setIgnoreTextChanges(false);
             }
             m_imeAcceptEvents = false;
         }
     }
 }
 
-
 // TODO(ekaramad):This method is almost duplicated in WebFrameWidgetImpl as
 // well. This code needs to be refactored  (http://crbug.com/629721).
 bool WebViewImpl::setComposition(
     const WebString& text,
     const WebVector<WebCompositionUnderline>& underlines,
     int selectionStart,
     int selectionEnd)
 {
-    LocalFrame* focused = focusedLocalFrameAvailableForIme();
-    if (!focused)
+    if (!getActiveWebInputMethodController())
         return false;
-
-    if (WebPlugin* plugin = focusedPluginIfInputMethodSupported(focused))
-        return plugin->setComposition(text, underlines, selectionStart, selectionEnd);
-
-    // The input focus has been moved to another WebWidget object.
-    // We should use this |editor| object only to complete the ongoing
-    // composition.
-    InputMethodController& inputMethodController = focused->inputMethodController();
-    if (!focused->editor().canEdit() && !inputMethodController.hasComposition())
-        return false;
-
-    // We should verify the parent node of this IME composition node are
-    // editable because JavaScript may delete a parent node of the composition
-    // node. In this case, WebKit crashes while deleting texts from the parent
-    // node, which doesn't exist any longer.
-    const EphemeralRange range = inputMethodController.compositionEphemeralRange();
-    if (range.isNotNull()) {
-        Node* node = range.startPosition().computeContainerNode();
-        focused->document()->updateStyleAndLayoutTree();
-        if (!node || !hasEditableStyle(*node))
-            return false;
-    }
-
-    // A keypress event is canceled. If an ongoing composition exists, then the
-    // keydown event should have arisen from a handled key (e.g., backspace).
-    // In this case we ignore the cancellation and continue; otherwise (no
-    // ongoing composition) we exit and signal success only for attempts to
-    // clear the composition.
-    if (m_suppressNextKeypressEvent && !inputMethodController.hasComposition())
-        return text.isEmpty();
-
-    UserGestureIndicator gestureIndicator(DefinitelyProcessingNewUserGesture);
-
-    // When the range of composition underlines overlap with the range between
-    // selectionStart and selectionEnd, WebKit somehow won't paint the selection
-    // at all (see InlineTextBox::paint() function in InlineTextBox.cpp).
-    // But the selection range actually takes effect.
-    inputMethodController.setComposition(String(text),
-                           CompositionUnderlineVectorBuilder(underlines),
-                           selectionStart, selectionEnd);
-
-    return text.isEmpty() || inputMethodController.hasComposition();
+    return getActiveWebInputMethodController()->setComposition(text, underlines, selectionStart, selectionEnd);
 }
 
-// TODO(ekaramad):These methods are almost duplicated in WebFrameWidgetImpl as
+// TODO(ekaramad):This method is almost duplicated in WebFrameWidgetImpl as
 // well. This code needs to be refactored  (http://crbug.com/629721).
 bool WebViewImpl::finishComposingText(ConfirmCompositionBehavior selectionBehavior)
 {
-    LocalFrame* focused = focusedLocalFrameAvailableForIme();
-    if (!focused)
+    if (!getActiveWebInputMethodController())
         return false;
-
-    if (WebPlugin* plugin = focusedPluginIfInputMethodSupported(focused))
-        return plugin->finishComposingText(selectionBehavior);
-
-    return focused->inputMethodController().finishComposingText(selectionBehavior == KeepSelection ? InputMethodController::KeepSelection : InputMethodController::DoNotKeepSelection);
-}
-
-bool WebViewImpl::commitText(const WebString& text, int relativeCaretPosition)
-{
-    UserGestureIndicator gestureIndicator(DefinitelyProcessingNewUserGesture);
-
-    LocalFrame* focused = focusedLocalFrameAvailableForIme();
-    if (!focused)
-        return false;
-
-    if (WebPlugin* plugin = focusedPluginIfInputMethodSupported(focused))
-        return plugin->commitText(text, relativeCaretPosition);
-
-    // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
-    // needs to be audited.  See http://crbug.com/590369 for more details.
-    focused->document()->updateStyleAndLayoutIgnorePendingStylesheets();
-
-    return focused->inputMethodController().commitText(text, relativeCaretPosition);
+    return getActiveWebInputMethodController()->finishComposingText(selectionBehavior);
 }
 
 // TODO(ekaramad):This method is almost duplicated in WebFrameWidgetImpl as
+// well. This code needs to be refactored  (http://crbug.com/629721).
+bool WebViewImpl::commitText(const WebString& text, int relativeCaretPosition)
+{
+    if (!getActiveWebInputMethodController())
+        return false;
+    return getActiveWebInputMethodController()->commitText(text, relativeCaretPosition);
+}
+
+// TODO(ekaramad):This method is almost duplicated in WebFrameWidgetImpl as
 // well. This code needs to be refactored  (http://crbug.com/629721).
 WebRange WebViewImpl::compositionRange()
 {
     LocalFrame* focused = focusedLocalFrameAvailableForIme();
     if (!focused)
         return WebRange();
 
     const EphemeralRange range = focused->inputMethodController().compositionEphemeralRange();
     if (range.isNull())
         return WebRange();
@@ skipping 1549 matching lines @@
 
     if (m_layerTreeView)
         m_layerTreeView->setHasTransparentBackground(this->isTransparent());
 }
 
 bool WebViewImpl::isTransparent() const
 {
     return m_isTransparent;
 }
 
+WebInputMethodControllerImpl* WebViewImpl::getActiveWebInputMethodController() const
+{
+    return WebInputMethodControllerImpl::fromFrame(focusedLocalFrameAvailableForIme());
+}
+
 void WebViewImpl::setBaseBackgroundColor(WebColor color)
 {
     if (m_baseBackgroundColor == color)
         return;
 
     m_baseBackgroundColor = color;
 
     if (m_page->mainFrame() && m_page->mainFrame()->isLocalFrame())
         m_page->deprecatedLocalMainFrame()->view()->setBaseBackgroundColor(color);
 }
@@ skipping 624 matching lines @@
         return nullptr;
     return focusedFrame;
 }
 
 LocalFrame* WebViewImpl::focusedLocalFrameAvailableForIme() const
 {
     return m_imeAcceptEvents ? focusedLocalFrameInWidget() : nullptr;
 }
 
 } // namespace blink