| Index: chrome/browser/chromeos/input_method/input_method_manager_impl_unittest.cc
|
| diff --git a/chrome/browser/chromeos/input_method/input_method_manager_impl_unittest.cc b/chrome/browser/chromeos/input_method/input_method_manager_impl_unittest.cc
|
| index c5b5e8d095f5b8303df8b21712800c14d5af89b2..aa38eea0f0875d997aecb91217dfd2749de341b6 100644
|
| --- a/chrome/browser/chromeos/input_method/input_method_manager_impl_unittest.cc
|
| +++ b/chrome/browser/chromeos/input_method/input_method_manager_impl_unittest.cc
|
| @@ -16,7 +16,6 @@
|
| #include "chrome/browser/chromeos/input_method/mock_candidate_window_controller.h"
|
| #include "chrome/browser/chromeos/input_method/mock_ibus_controller.h"
|
| #include "chromeos/dbus/fake_dbus_thread_manager.h"
|
| -#include "chromeos/dbus/ibus/mock_ibus_client.h"
|
| #include "chromeos/ime/extension_ime_util.h"
|
| #include "chromeos/ime/fake_input_method_delegate.h"
|
| #include "chromeos/ime/mock_component_extension_ime_manager_delegate.h"
|
| @@ -151,9 +150,6 @@ class InputMethodManagerImplTest : public testing::Test {
|
| // Helper function to initialize IBus bus connection for testing. Do not use
|
| // ibus related mocks before calling this function.
|
| void InitIBusBus() {
|
| - mock_ibus_client_ = new MockIBusClient;
|
| - fake_dbus_thread_manager_->SetIBusClient(
|
| - scoped_ptr<IBusClient>(mock_ibus_client_));
|
| mock_ibus_daemon_controller_->EmulateConnect();
|
| }
|
|
|
| @@ -163,7 +159,6 @@ class InputMethodManagerImplTest : public testing::Test {
|
| MockCandidateWindowController* candidate_window_controller_;
|
| MockIBusDaemonController* mock_ibus_daemon_controller_;
|
| scoped_ptr<MockIMEEngineHandler> mock_engine_handler_;
|
| - MockIBusClient* mock_ibus_client_;
|
| FakeDBusThreadManager* fake_dbus_thread_manager_;
|
| MockXKeyboard* xkeyboard_;
|
| base::MessageLoop message_loop_;
|
| @@ -1105,26 +1100,6 @@ TEST_F(InputMethodManagerImplTest, TestAddExtensionInputThenLockScreen) {
|
| manager_->RemoveObserver(&observer);
|
| }
|
|
|
| -TEST_F(InputMethodManagerImplTest, TestReset) {
|
| - InitComponentExtension();
|
| - InitIBusBus();
|
| - manager_->SetState(InputMethodManager::STATE_BROWSER_SCREEN);
|
| - std::vector<std::string> ids;
|
| - ids.push_back("xkb:us::eng");
|
| - ids.push_back(nacl_mozc_us_id);
|
| - EXPECT_TRUE(manager_->EnableInputMethods(ids));
|
| - EXPECT_EQ(2U, manager_->GetNumActiveInputMethods());
|
| - EXPECT_EQ(0, mock_engine_handler_->reset_call_count());
|
| - manager_->ChangeInputMethod(nacl_mozc_us_id);
|
| - EXPECT_EQ(1, mock_ibus_client_->set_global_engine_call_count());
|
| - EXPECT_EQ(nacl_mozc_us_id, mock_ibus_client_->latest_global_engine_name());
|
| - EXPECT_EQ(0, mock_engine_handler_->reset_call_count());
|
| - manager_->ChangeInputMethod("xkb:us::eng");
|
| - EXPECT_EQ(1, mock_ibus_client_->set_global_engine_call_count());
|
| - EXPECT_EQ(nacl_mozc_us_id, mock_ibus_client_->latest_global_engine_name());
|
| - EXPECT_EQ(0, mock_engine_handler_->reset_call_count());
|
| -}
|
| -
|
| TEST_F(InputMethodManagerImplTest,
|
| ChangeInputMethodBeforeComponentExtensionInitialization_OneIME) {
|
| manager_->SetState(InputMethodManager::STATE_BROWSER_SCREEN);
|
| @@ -1136,8 +1111,7 @@ TEST_F(InputMethodManagerImplTest,
|
|
|
| InitIBusBus();
|
| InitComponentExtension();
|
| - EXPECT_EQ(1, mock_ibus_client_->set_global_engine_call_count());
|
| - EXPECT_EQ(nacl_mozc_us_id, mock_ibus_client_->latest_global_engine_name());
|
| + EXPECT_EQ(nacl_mozc_us_id, manager_->GetCurrentInputMethod().id());
|
| }
|
|
|
| TEST_F(InputMethodManagerImplTest,
|
| @@ -1153,8 +1127,7 @@ TEST_F(InputMethodManagerImplTest,
|
|
|
| InitComponentExtension();
|
| InitIBusBus();
|
| - EXPECT_EQ(1, mock_ibus_client_->set_global_engine_call_count());
|
| - EXPECT_EQ(nacl_mozc_jp_id, mock_ibus_client_->latest_global_engine_name());
|
| + EXPECT_EQ(nacl_mozc_jp_id, manager_->GetCurrentInputMethod().id());
|
| }
|
|
|
| TEST_F(InputMethodManagerImplTest,
|
| @@ -1172,8 +1145,7 @@ TEST_F(InputMethodManagerImplTest,
|
|
|
| InitComponentExtension();
|
| InitIBusBus();
|
| - EXPECT_EQ(1, mock_ibus_client_->set_global_engine_call_count());
|
| - EXPECT_EQ(ext_id, mock_ibus_client_->latest_global_engine_name());
|
| + EXPECT_EQ(ext_id, manager_->GetCurrentInputMethod().id());
|
| }
|
|
|
| TEST_F(InputMethodManagerImplTest,
|
| @@ -1197,8 +1169,7 @@ TEST_F(InputMethodManagerImplTest,
|
|
|
| InitComponentExtension();
|
| InitIBusBus();
|
| - EXPECT_EQ(1, mock_ibus_client_->set_global_engine_call_count());
|
| - EXPECT_EQ(ext_id2, mock_ibus_client_->latest_global_engine_name());
|
| + EXPECT_EQ(ext_id2, manager_->GetCurrentInputMethod().id());
|
| }
|
|
|
| TEST_F(InputMethodManagerImplTest,
|
| @@ -1214,8 +1185,7 @@ TEST_F(InputMethodManagerImplTest,
|
| ids.push_back(ext_id);
|
| EXPECT_TRUE(manager_->EnableInputMethods(ids));
|
| EXPECT_EQ(1U, manager_->GetNumActiveInputMethods());
|
| - EXPECT_EQ(1, mock_ibus_client_->set_global_engine_call_count());
|
| - EXPECT_EQ(ext_id, mock_ibus_client_->latest_global_engine_name());
|
| + EXPECT_EQ(ext_id, manager_->GetCurrentInputMethod().id());
|
| }
|
|
|
| TEST_F(InputMethodManagerImplTest,
|
| @@ -1236,11 +1206,9 @@ TEST_F(InputMethodManagerImplTest,
|
| ids.push_back(ext_id2);
|
| EXPECT_TRUE(manager_->EnableInputMethods(ids));
|
| EXPECT_EQ(2U, manager_->GetNumActiveInputMethods());
|
| - EXPECT_EQ(1, mock_ibus_client_->set_global_engine_call_count());
|
| - EXPECT_EQ(ext_id1, mock_ibus_client_->latest_global_engine_name());
|
| + EXPECT_EQ(ext_id1, manager_->GetCurrentInputMethod().id());
|
| manager_->ChangeInputMethod(ext_id2);
|
| - EXPECT_EQ(2, mock_ibus_client_->set_global_engine_call_count());
|
| - EXPECT_EQ(ext_id2, mock_ibus_client_->latest_global_engine_name());
|
| + EXPECT_EQ(ext_id2, manager_->GetCurrentInputMethod().id());
|
| }
|
|
|
| TEST_F(InputMethodManagerImplTest,
|
|
|