Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4370)

Unified Diff: chrome/browser/chromeos/input_method/input_method_manager_impl_unittest.cc

Issue 118273002: Set the active input method correctly when EnableLayouts is called. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 8cd99824d46106e04f5d476fabe133856bfb82b1..b447ac9f0d1a2bcfbe7b457136321b49410b689d 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
@@ -294,6 +294,13 @@ TEST_F(InputMethodManagerImplTest, TestEnableLayouts) {
EXPECT_EQ(2U, manager_->GetNumActiveInputMethods());
}
+TEST_F(InputMethodManagerImplTest, TestEnableLayoutsAndCurrentInputMethod) {
+ // For http://crbug.com/329061
+ manager_->EnableLayouts("en-US", "xkb:se::swe");
+ const std::string im_id = manager_->GetCurrentInputMethod().id();
+ EXPECT_EQ("xkb:se::swe", im_id);
+}
+
TEST_F(InputMethodManagerImplTest, TestEnableLayoutsNonUsHardwareKeyboard) {
// The physical layout is French.
delegate_->set_hardware_keyboard_layout("xkb:fr::fra");

Powered by Google App Engine
This is Rietveld 408576698