Fix setComposingText when newCursorPosition != 1

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

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/editing/InputMethodController.h"
 
 #include "core/dom/Element.h"
 #include "core/dom/Range.h"
 #include "core/editing/FrameSelection.h"
 #include "core/frame/LocalFrame.h"
@@ skipping 156 matching lines @@
 }
 
 TEST_F(InputMethodControllerTest, SetCompositionFromExistingTextWithInvalidOffsets)
 {
     insertHTMLElement("<div id='sample' contenteditable='true'>test</div>", "sample");
 
     Vector<CompositionUnderline> underlines;
     underlines.append(CompositionUnderline(7, 8, Color(255, 0, 0), false, 0));
     controller().setCompositionFromExistingText(underlines, 7, 8);
 
-    EXPECT_FALSE(controller().compositionRange());
+    // If it exceeds the right boundary, it should stay at the right boundary
+    RawPtr<Range> range = controller().compositionRange();
+    EXPECT_EQ(4, range->startOffset());
+    EXPECT_EQ(4, range->endOffset());
 }
 
 TEST_F(InputMethodControllerTest, ConfirmPasswordComposition)
 {
     HTMLInputElement* input = toHTMLInputElement(
         insertHTMLElement("<input id='sample' type='password' size='24'>", "sample"));
 
     Vector<CompositionUnderline> underlines;
     underlines.append(CompositionUnderline(0, 5, Color(255, 0, 0), false, 0));
     controller().setComposition("foo", underlines, 0, 3);
     controller().confirmComposition();
 
     EXPECT_STREQ("foo", input->value().utf8().data());
 }
 
+TEST_F(InputMethodControllerTest, SetCompositionForInputWithDifferentNewCursorPositions)
+{
+    HTMLInputElement* input = toHTMLInputElement(
+        insertHTMLElement("<input id='sample'>", "sample"));
+
+    input->setValue("hello");
+    controller().setEditableSelectionOffsets(PlainTextRange(2, 2));
+    EXPECT_STREQ("hello", input->value().utf8().data());
+    EXPECT_EQ(2, frame().selection().start().computeOffsetInContainerNode());
+    EXPECT_EQ(2, frame().selection().end().computeOffsetInContainerNode());
+
+    Vector<CompositionUnderline> underlines;
+    underlines.append(CompositionUnderline(0, 2, Color(255, 0, 0), false, 0));
+
+    // The cursor exceeds left boundary.
+    // "*heABllo", where * stands for cursor.
+    controller().setComposition("AB", underlines, -10, -10);
+    EXPECT_STREQ("heABllo", input->value().utf8().data());
+    EXPECT_EQ(0, frame().selection().start().computeOffsetInContainerNode());
+    EXPECT_EQ(0, frame().selection().end().computeOffsetInContainerNode());
+
+    // The cursor is on left boundary.
+    // "*heABllo".
+    controller().setComposition("AB", underlines, -2, -2);
+    EXPECT_STREQ("heABllo", input->value().utf8().data());
+    EXPECT_EQ(0, frame().selection().start().computeOffsetInContainerNode());
+    EXPECT_EQ(0, frame().selection().end().computeOffsetInContainerNode());
+
+    // The cursor is before the composing text.
+    // "he*ABllo".
+    controller().setComposition("AB", underlines, 0, 0);
+    EXPECT_STREQ("heABllo", input->value().utf8().data());
+    EXPECT_EQ(2, frame().selection().start().computeOffsetInContainerNode());
+    EXPECT_EQ(2, frame().selection().end().computeOffsetInContainerNode());
+
+    // The cursor is after the composing text.
+    // "heAB*llo".
+    controller().setComposition("AB", underlines, 2, 2);
+    EXPECT_STREQ("heABllo", input->value().utf8().data());
+    EXPECT_EQ(4, frame().selection().start().computeOffsetInContainerNode());
+    EXPECT_EQ(4, frame().selection().end().computeOffsetInContainerNode());
+
+    // The cursor is on right boundary.
+    // "heABllo*".
+    controller().setComposition("AB", underlines, 5, 5);
+    EXPECT_STREQ("heABllo", input->value().utf8().data());
+    EXPECT_EQ(7, frame().selection().start().computeOffsetInContainerNode());
+    EXPECT_EQ(7, frame().selection().end().computeOffsetInContainerNode());
+
+    // The cursor exceeds right boundary.
+    // "heABllo*".
+    controller().setComposition("AB", underlines, 10, 10);
+    EXPECT_STREQ("heABllo", input->value().utf8().data());
+    EXPECT_EQ(7, frame().selection().start().computeOffsetInContainerNode());
+    EXPECT_EQ(7, frame().selection().end().computeOffsetInContainerNode());
+}
+
+TEST_F(InputMethodControllerTest, SetCompositionForEditableWithDifferentNewCursorPositions)
+{
+    // There are 3 nodes and 5+1+6 characters: "hello", '\n', "world!".
+    Element* div = insertHTMLElement(
+        "<div id='sample' contenteditable='true'>"
+        "hello"
+        "<p id='sample2' contenteditable='true'>world!</p>"
+        "</div>",
+        "sample");
+
+    controller().setEditableSelectionOffsets(PlainTextRange(2, 2));
+    EXPECT_STREQ("hello\nworld!", div->innerText().utf8().data());
+    EXPECT_EQ(2, frame().selection().start().computeOffsetInContainerNode());
+    EXPECT_EQ(2, frame().selection().end().computeOffsetInContainerNode());
+
+    Vector<CompositionUnderline> underlines;
+    underlines.append(CompositionUnderline(0, 2, Color(255, 0, 0), false, 0));
+
+    // The cursor exceeds left boundary.
+    // "*heABllo\nworld!", where * stands for cursor.
+    controller().setComposition("AB", underlines, -10, -10);
+    EXPECT_STREQ("heABllo\nworld!", div->innerText().utf8().data());
+    EXPECT_EQ(0, frame().selection().start().computeOffsetInContainerNode());
+    EXPECT_EQ(0, frame().selection().end().computeOffsetInContainerNode());
+
+    // The cursor is on left boundary.
+    // "*heABllo\nworld!"
+    controller().setComposition("AB", underlines, -2, -2);
+    EXPECT_STREQ("heABllo\nworld!", div->innerText().utf8().data());
+    EXPECT_EQ(0, frame().selection().start().computeOffsetInContainerNode());
+    EXPECT_EQ(0, frame().selection().end().computeOffsetInContainerNode());
+
+    // The cursor is before the composing text.
+    // "he*ABllo\nworld!"
+    controller().setComposition("AB", underlines, 0, 0);
+    EXPECT_STREQ("heABllo\nworld!", div->innerText().utf8().data());
+    EXPECT_EQ(2, frame().selection().start().computeOffsetInContainerNode());
+    EXPECT_EQ(2, frame().selection().end().computeOffsetInContainerNode());
+
+    // The cursor is after the composing text.
+    // "heAB*llo\nworld!"
+    controller().setComposition("AB", underlines, 2, 2);
+    EXPECT_STREQ("heABllo\nworld!", div->innerText().utf8().data());
+    EXPECT_EQ(4, frame().selection().start().computeOffsetInContainerNode());
+    EXPECT_EQ(4, frame().selection().end().computeOffsetInContainerNode());
+
+    // The cursor is on right boundary of the 1st node.
+    // "heABllo*\nworld!"
+    controller().setComposition("AB", underlines, 5, 5);
+    EXPECT_STREQ("heABllo\nworld!", div->innerText().utf8().data());
+    EXPECT_EQ(7, frame().selection().start().computeOffsetInContainerNode());
+    EXPECT_EQ(7, frame().selection().end().computeOffsetInContainerNode());
+
+    // The cursor is on left boundary of the 3rd node.
+    // "heABllo\n*world!".
+    controller().setComposition("AB", underlines, 6, 6);
+    EXPECT_STREQ("heABllo\nworld!", div->innerText().utf8().data());
+    EXPECT_EQ(0, frame().selection().start().computeOffsetInContainerNode());
+    EXPECT_EQ(0, frame().selection().end().computeOffsetInContainerNode());
+
+    // The cursor is on right boundary of the 3rd node.
+    // "heABllo\nworld!*"
+    controller().setComposition("AB", underlines, 12, 12);
+    EXPECT_STREQ("heABllo\nworld!", div->innerText().utf8().data());
+    EXPECT_EQ(6, frame().selection().start().computeOffsetInContainerNode());
+    EXPECT_EQ(6, frame().selection().end().computeOffsetInContainerNode());
+
+    // The cursor exceeds right boundary.
+    // "heABllo\nworld!*"
+    controller().setComposition("AB", underlines, 20, 20);
+    EXPECT_STREQ("heABllo\nworld!", div->innerText().utf8().data());
+    EXPECT_EQ(6, frame().selection().start().computeOffsetInContainerNode());
+    EXPECT_EQ(6, frame().selection().end().computeOffsetInContainerNode());
+}
 } // namespace blink