Translate: split language code typo correction to apply unit tests.

chrome/renderer/translate_helper_unittest.cc

Index: chrome/renderer/translate_helper_unittest.cc
diff --git a/chrome/renderer/translate_helper_unittest.cc b/chrome/renderer/translate_helper_unittest.cc
index 16bb9154de06d77a3a314b0e13b36d1d042b19b7..e03c454e5d223effec4241526f815a1e2de52d78 100644
--- a/chrome/renderer/translate_helper_unittest.cc
+++ b/chrome/renderer/translate_helper_unittest.cc
@@ -8,6 +8,26 @@
 
 typedef testing::Test TranslateHelperTest;
 
+// Tests that well-known language code typos are fixed.
+TEST_F(TranslateHelperTest, LanguageCodeTypoCorrection) {
+  std::string language;
+
+  // Strip the second and later codes.
+  language = std::string("ja,en");
+  TranslateHelper::CorrectLanguageCodeTypo(&language);
+  EXPECT_EQ(0, language.compare("ja"));

[sky, 2013/02/12 17:30:07]

EXPECT_EQ("ja", language) same pattern for other tests.

[Takashi Toyoshima, 2013/02/14 08:00:18]

Done.

+
+  // Replace dash with hyphen.
+  language = std::string("ja_JP");
+  TranslateHelper::CorrectLanguageCodeTypo(&language);
+  EXPECT_EQ(0, language.compare("ja-JP"));
+
+  // Correct wrong cases.
+  language = std::string("JA-jp");
+  TranslateHelper::CorrectLanguageCodeTypo(&language);
+  EXPECT_EQ(0, language.compare("ja-JP"));
+}
+
 // Tests that synonym language code is converted to one used in supporting list.
 TEST_F(TranslateHelperTest, LanguageCodeSynonyms) {
   std::string language;