[IME] migrate the xkb ID to extension based xkb ID.

chrome/browser/chromeos/input_method/input_method_manager_impl_unittest.cc

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 4eed0cb7352423a3aa916eb283a9d743e7204d07..8f7bc96ea6c4e26dd14ce51ab5cb0018922d218c 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
@@ -1209,5 +1209,20 @@ TEST_F(InputMethodManagerImplTest,
   EXPECT_EQ(ext_id2, manager_->GetCurrentInputMethod().id());
 }
 
+TEST_F(InputMethodManagerImplTest, MigrateXkbInputMethodTest) {
+  std::vector<std::string> input_method_ids;
+  input_method_ids.push_back("xkb:us::eng");
+  input_method_ids.push_back("xkb:fr::fra");
+  input_method_ids.push_back("_comp_ime_asdf_pinyin");
+
+  manager_->MigrateXkbInputMethods(&input_method_ids);
+
+  ASSERT_EQ(3U, input_method_ids.size());
+
+  for (size_t i = 0; i < input_method_ids.size(); i++) {
+    EXPECT_TRUE(StartsWithASCII("_comp_ime_", input_method_ids[i], true));

[Seigo Nonaka, 2014/02/27 04:30:39]

please also add checking unique.
If xkb:us::eng and _comp_ime_asdf_us_eng is co-exists, MigrateXkbInputMethods
should make unique the list.

[Shu Chen, 2014/02/27 09:22:00]

Done.

+  }

[Seigo Nonaka, 2014/02/27 04:30:39]

Also nice to add keeping order case.
The order of engine should be kept with migration.

[Shu Chen, 2014/02/27 09:22:00]

Done.

+}
+
 }  // namespace input_method
 }  // namespace chromeos