Fix ibus-m17n crash due to wrong parameter from Chrome.

ui/base/ime/input_method_ibus_unittest.cc

 // Copyright (c) 2012 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 <X11/Xlib.h>
 #undef Bool
 #undef FocusIn
 #undef FocusOut
 #undef None
 
@@ skipping 1129 matching lines @@
   chromeos::DBusThreadManager::Get()->InitIBusBus("dummy address",
                                                   base::Bind(&base::DoNothing));
   mock_ibus_daemon_controller_->EmulateConnect();
 
   // Set the TextInputClient behaviors.
   surrounding_text_ = UTF8ToUTF16("abcdef");
   text_range_ = ui::Range(0, 6);
   selection_range_ = ui::Range(3, 3);
 
   // Set the verifier for SetSurroundingText mock call.
-  SetSurroundingTextVerifier verifier(UTF16ToUTF8(surrounding_text_),
-                                      selection_range_.start(),
-                                      selection_range_.end());
+  SetSurroundingTextVerifier verifier(UTF16ToUTF8(surrounding_text_), 3, 3);
+
 
   mock_ibus_input_context_client_->set_set_surrounding_text_handler(
       base::Bind(&SetSurroundingTextVerifier::Verify,
                  base::Unretained(&verifier)));
   ime_->OnCaretBoundsChanged(this);
 
   // Check the call count.
   EXPECT_EQ(1,
             mock_ibus_input_context_client_->set_surrounding_text_call_count());
 }
 
 TEST_F(InputMethodIBusTest, SurroundingText_SelectionTest) {
   SetCreateContextSuccessHandler();
   ime_->Init(true);
   // Click a text input form.
   input_type_ = TEXT_INPUT_TYPE_TEXT;
   ime_->OnTextInputTypeChanged(this);
   // Start the daemon.
   chromeos::DBusThreadManager::Get()->InitIBusBus("dummy address",
                                                   base::Bind(&base::DoNothing));
   mock_ibus_daemon_controller_->EmulateConnect();
 
   // Set the TextInputClient behaviors.
   surrounding_text_ = UTF8ToUTF16("abcdef");
   text_range_ = ui::Range(0, 6);
   selection_range_ = ui::Range(2, 5);
 
   // Set the verifier for SetSurroundingText mock call.
-  SetSurroundingTextVerifier verifier(UTF16ToUTF8(surrounding_text_),
-                                      selection_range_.start(),
-                                      selection_range_.end());
+  SetSurroundingTextVerifier verifier(UTF16ToUTF8(surrounding_text_), 2, 5);
 
   mock_ibus_input_context_client_->set_set_surrounding_text_handler(
       base::Bind(&SetSurroundingTextVerifier::Verify,
+                 base::Unretained(&verifier)));
+  ime_->OnCaretBoundsChanged(this);
+
+  // Check the call count.
+  EXPECT_EQ(1,
+            mock_ibus_input_context_client_->set_surrounding_text_call_count());
+}
+
+TEST_F(InputMethodIBusTest, SurroundingText_PartialText) {
+  SetCreateContextSuccessHandler();
+  ime_->Init(true);
+  // Click a text input form.
+  input_type_ = TEXT_INPUT_TYPE_TEXT;
+  ime_->OnTextInputTypeChanged(this);
+  // Start the daemon.
+  chromeos::DBusThreadManager::Get()->InitIBusBus("dummy address",
+                                                  base::Bind(&base::DoNothing));
+  mock_ibus_daemon_controller_->EmulateConnect();
+
+  // Set the TextInputClient behaviors.
+  surrounding_text_ = UTF8ToUTF16("abcdefghij");
+  text_range_ = ui::Range(5, 10);
+  selection_range_ = ui::Range(7, 9);
+
+  // Set the verifier for SetSurroundingText mock call.
+  SetSurroundingTextVerifier verifier("fghij", 2, 4);

[kinaba, 2013/04/10 04:30:04]

please add comment about how the (2, 4) came up.

[Seigo Nonaka, 2013/04/10 05:15:54]

Done.

+
+  mock_ibus_input_context_client_->set_set_surrounding_text_handler(
+      base::Bind(&SetSurroundingTextVerifier::Verify,
                  base::Unretained(&verifier)));
   ime_->OnCaretBoundsChanged(this);
 
   // Check the call count.
   EXPECT_EQ(1,
             mock_ibus_input_context_client_->set_surrounding_text_call_count());
 }
 
 class InputMethodIBusKeyEventTest : public InputMethodIBusTest {
  public:
@@ skipping 485 matching lines @@
 
   // Do callback.
   success_consume_handler.RunCallback(KEYEVENT_CONSUME);
 
   EXPECT_EQ(0, ime_->process_key_event_post_ime_call_count());
 }
 
 // TODO(nona): Introduce ProcessKeyEventPostIME tests(crbug.com/156593).
 
 }  // namespace ui