fix target language issue under three locales (Hebrew[iw->he], Fillipion[tl->fil], and Nowreign[no-…

components/translate/core/browser/translate_manager.cc

Index: components/translate/core/browser/translate_manager.cc
diff --git a/components/translate/core/browser/translate_manager.cc b/components/translate/core/browser/translate_manager.cc
index 8491a2adb164dbb1db06b4f48d0af8ca082ea006..4f5bc5fbd6765d12d5e54cbc2e2f2d7df1d4136d 100644
--- a/components/translate/core/browser/translate_manager.cc
+++ b/components/translate/core/browser/translate_manager.cc
@@ -514,6 +514,8 @@ std::string TranslateManager::GetTargetLanguage(const TranslatePrefs* prefs) {
     // Get the browser's user interface language.
     language = TranslateDownloadManager::GetLanguageCode(
         TranslateDownloadManager::GetInstance()->application_locale());
+    // Map 'he', 'nb', 'fil' back to 'iw', 'no', 'tl'
+    translate::ToTranslateLanguageSynonym(&language);

[jungshik at Google, 2017/04/11 18:45:32]

Shouldn't Google Translate API take care of this instead of clients like Chrome
doing this on their end?  

That is, Google Translate API should accept both {he,iw}, {tl, fil}, {nb, no}
and treat them as synonyms? 

   }
   if (TranslateDownloadManager::IsSupportedLanguage(language))
     return language;