Remove logic to reset input method more than needed

third_party/WebKit/Source/web/tests/WebViewTest.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 1532 matching lines @@
   EXPECT_EQ(6, info.selectionStart);
   EXPECT_EQ(11, info.selectionEnd);
   EXPECT_EQ(6, info.compositionStart);
   EXPECT_EQ(11, info.compositionEnd);
 
   frame->setEditableSelectionOffsets(2, 2);
   info = activeInputMethodController->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);
+  // Composition range should be reset by browser process or keyboard apps.
+  EXPECT_EQ(6, info.compositionStart);
+  EXPECT_EQ(11, info.compositionEnd);
 }
 
 TEST_P(WebViewTest, IsSelectionAnchorFirst) {
   registerMockedHttpURLLoad("input_field_populated.html");
   WebViewImpl* webView = m_webViewHelper.initializeAndLoad(
       m_baseURL + "input_field_populated.html");
   WebLocalFrame* frame = webView->mainFrameImpl();
 
   webView->setInitialFocus(false);
   frame->setEditableSelectionOffsets(4, 10);
@@ skipping 793 matching lines @@
   WebInputMethodController* activeInputMethodController =
       frame->frameWidget()->getActiveWebInputMethodController();
   EXPECT_TRUE(tapElementById(WebInputEvent::GestureTap, target));
   WebVector<WebCompositionUnderline> emptyUnderlines;
   frame->setEditableSelectionOffsets(8, 8);
   EXPECT_TRUE(activeInputMethodController->setComposition(
       "12345", emptyUnderlines, 8, 13));
   EXPECT_TRUE(frame->frame()->inputMethodController().hasComposition());
   EXPECT_EQ("", std::string(frame->selectionAsText().utf8().data()));
   EXPECT_FALSE(frame->frame()->selection().isHandleVisible());
+  EXPECT_TRUE(frame->frame()->inputMethodController().hasComposition());
 
   EXPECT_TRUE(tapElementById(WebInputEvent::GestureLongPress, target));
   EXPECT_EQ("testword12345",
             std::string(frame->selectionAsText().utf8().data()));
   EXPECT_TRUE(frame->frame()->selection().isHandleVisible());
+  EXPECT_TRUE(frame->frame()->inputMethodController().hasComposition());
 
   // Check that finishComposingText(KeepSelection) does not dismiss handles.
   activeInputMethodController->finishComposingText(
       WebInputMethodController::KeepSelection);
   EXPECT_TRUE(frame->frame()->selection().isHandleVisible());
 }
 
 #if !OS(MACOSX)
 TEST_P(WebViewTest, TouchDoesntSelectEmptyTextarea) {
   registerMockedHttpURLLoad("longpress_textarea.html");
@@ skipping 1852 matching lines @@
   EXPECT_EQ(expectedWidth, vwElement->offsetWidth());
   EXPECT_EQ(expectedHeight, vwElement->offsetHeight());
 
   webView->resize(WebSize(800, 600));
   frame->printEnd();
 
   EXPECT_EQ(800, vwElement->offsetWidth());
 }
 
 }  // namespace blink