Handle newCursorPosition correctly for Android's commitText()

third_party/WebKit/Source/core/editing/InputMethodController.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 215 matching lines @@
     // Event handler might destroy document.
     if (!frame().document())
         return false;
 
     // No DOM update after 'compositionend'.
     dispatchCompositionEndEvent(frame(), text);
 
     return true;
 }
 
-bool InputMethodController::confirmCompositionOrInsertText(const String& text, ConfirmCompositionBehavior confirmBehavior)
+bool InputMethodController::confirmCompositionOrInsertText(const String& text, ConfirmCompositionBehavior confirmBehavior, int newCursorPosition)
 {
+    if (confirmBehavior == DoNotKeepSelection) {
+        // If newCursorPosition > 0, it's relative to the end of the text - 1;
+        // if newCursorPosition <= 0, it is relative to the start of the text.
+        if (newCursorPosition > 0)
+            newCursorPosition += text.length() - 1;
+
+        // According to Android documentation, the new cursor position is

[Changwan Ryu, 2016/08/24 01:32:03]

This comment is platform-specific. Could you make it more generic?

[yabinh, 2016/08/24 06:05:01]

Done.

+        // relative to the composing text if any, or relative to the selection
+        // if no composing text.
+        if (hasComposition()) {
+            Element* rootEditableElement = frame().selection().rootEditableElement();
+            if (!rootEditableElement)
+                return false;
+            PlainTextRange compositionRange = PlainTextRange::create(*rootEditableElement, *m_compositionRange);
+
+            if (text.length() == 0)
+                newCursorPosition += compositionRange.end();
+            else
+                newCursorPosition += compositionRange.start();
+        } else {
+            PlainTextRange selectionRange = getSelectionOffsets();
+            if (selectionRange.isNull())
+                return false;
+            newCursorPosition += getSelectionOffsets().start();
+        }
+    }
+
     if (!hasComposition()) {
         if (!text.length())
             return false;
 
         if (dispatchBeforeInputInsertText(frame().document()->focusedElement(), text) != DispatchEventResult::NotCanceled)
             return false;
 
         editor().insertText(text, 0);
-        return true;
+    } else if (text.length()) {
+        confirmComposition(text);
+    } else if (confirmBehavior == DoNotKeepSelection) {
+        if (!confirmComposition(composingText(), DoNotKeepSelection))
+            return false;
+    } else {
+        SelectionOffsetsScope selectionOffsetsScope(this);
+        return confirmComposition();
     }
 
-    if (text.length()) {
-        confirmComposition(text);
+    if (confirmBehavior == KeepSelection)
         return true;
-    }
 
-    if (confirmBehavior == DoNotKeepSelection)
-        return confirmComposition(composingText(), DoNotKeepSelection);
+    frame().document()->updateStyleAndLayoutIgnorePendingStylesheets();
 
-    SelectionOffsetsScope selectionOffsetsScope(this);
-    return confirmComposition();
+    PlainTextRange selectedRange = createRangeForSelection(newCursorPosition, newCursorPosition, 0);
+    if (selectedRange.isNull())
+        return false;
+    return setEditableSelectionOffsets(selectedRange);
 }
 
 void InputMethodController::cancelComposition()
 {
     if (!hasComposition())
         return;
 
     Editor::RevealSelectionScope revealSelectionScope(&editor());
 
     if (frame().selection().isNone())
@@ skipping 306 matching lines @@
     TypingCommand::deleteSelection(*frame().document());
 }
 
 DEFINE_TRACE(InputMethodController)
 {
     visitor->trace(m_frame);
     visitor->trace(m_compositionRange);
 }
 
 } // namespace blink