Fixing the bounds check in cancelCompositionIfSelectionIsInvalid.

Source/web/tests/WebViewTest.cpp

Index: Source/web/tests/WebViewTest.cpp
diff --git a/Source/web/tests/WebViewTest.cpp b/Source/web/tests/WebViewTest.cpp
index eabc0c1f9cde469ded3300f76a658f3732baf5f8..b877116cb84e4b06ee2189d1b56ea8ec2e0e6943 100644
--- a/Source/web/tests/WebViewTest.cpp
+++ b/Source/web/tests/WebViewTest.cpp
@@ -683,6 +683,52 @@ TEST_F(WebViewTest, SetCompositionFromExistingTextInTextArea)
     webView->close();
 }
 
+TEST_F(WebViewTest, SetEditableSelectionOffsetsKeepsComposition)
+{
+    URLTestHelpers::registerMockedURLFromBaseURL(WebString::fromUTF8(m_baseURL.c_str()), WebString::fromUTF8("input_field_populated.html"));
+    WebView* webView = FrameTestHelpers::createWebViewAndLoad(m_baseURL + "input_field_populated.html");
+    webView->setInitialFocus(false);
+
+    std::string compositionTextFirst("hello ");
+    std::string compositionTextSecond("world");
+    WebVector<WebCompositionUnderline> emptyUnderlines;
+
+    webView->confirmComposition(WebString::fromUTF8(compositionTextFirst.c_str()));
+    webView->setComposition(WebString::fromUTF8(compositionTextSecond.c_str()), emptyUnderlines, 5, 5);
+
+    WebTextInputInfo info = webView->textInputInfo();
+    EXPECT_EQ("hello world", std::string(info.value.utf8().data()));
+    EXPECT_EQ(11, info.selectionStart);
+    EXPECT_EQ(11, info.selectionEnd);
+    EXPECT_EQ(6, info.compositionStart);
+    EXPECT_EQ(11, info.compositionEnd);
+
+    webView->setEditableSelectionOffsets(6, 6);
+    info = webView->textInputInfo();
+    EXPECT_EQ("hello world", std::string(info.value.utf8().data()));
+    EXPECT_EQ(6, info.selectionStart);
+    EXPECT_EQ(6, info.selectionEnd);
+    EXPECT_EQ(6, info.compositionStart);
+    EXPECT_EQ(11, info.compositionEnd);
+
+    webView->setEditableSelectionOffsets(8, 8);
+    info = webView->textInputInfo();
+    EXPECT_EQ("hello world", std::string(info.value.utf8().data()));
+    EXPECT_EQ(8, info.selectionStart);
+    EXPECT_EQ(8, info.selectionEnd);
+    EXPECT_EQ(6, info.compositionStart);
+    EXPECT_EQ(11, info.compositionEnd);
+
+    webView->setEditableSelectionOffsets(2, 2);

[yosin_UTC9, 2013/09/06 01:17:55]

Please add test case for moving selection offset to start=11,end=11 and
start=6,end=11.

[aurimas (slooooooooow), 2013/09/06 01:25:35]

Done.

+    info = webView->textInputInfo();
+    EXPECT_EQ("hello world", std::string(info.value.utf8().data()));
+    EXPECT_EQ(2, info.selectionStart);
+    EXPECT_EQ(2, info.selectionEnd);
+    EXPECT_EQ(-1, info.compositionStart);
+    EXPECT_EQ(-1, info.compositionEnd);
+    webView->close();
+}
+
 TEST_F(WebViewTest, IsSelectionAnchorFirst)
 {
     URLTestHelpers::registerMockedURLFromBaseURL(WebString::fromUTF8(m_baseURL.c_str()), WebString::fromUTF8("input_field_populated.html"));