Don't offer to translate between Simplified and Traditional Chinese

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 33d3c514a5d011646e21b4c5aedfbff7679a21f6..1c9183a27fb6df6deb62b1b12702caf4fe32e061 100644
--- a/components/translate/core/browser/translate_manager.cc
+++ b/components/translate/core/browser/translate_manager.cc
@@ -4,6 +4,7 @@
 
 #include "components/translate/core/browser/translate_manager.h"
 
+#include <iostream>

[kjellander_chromium, 2017/01/26 05:30:06]

I'm confused how this could pass the _CheckNoIOStreamInHeaders check in
https://chromium.googlesource.com/chromium/src/+/master/PRESUBMIT.py#395 

iostream will give you static initializers, which are not allowed in Chromium. I
filed crbug.com/685512 for that.

 #include <map>
 
 #include "base/bind.h"
@@ -224,12 +225,14 @@ void TranslateManager::InitiateTranslation(const std::string& page_lang) {
   InitTranslateEvent(language_code, target_lang, *translate_prefs);
 
   // Don't translate similar languages (ex: en-US to en).
-  if (language_code == target_lang) {
+  // Also do not offer to translate between Simplified and Traditional Chinese.
+  if (language_code == target_lang ||
+      (language_code == "zh-CN" && target_lang == "zh-TW") ||
+      (language_code == "zh-TW" && target_lang == "zh-CN")) {
     TranslateBrowserMetrics::ReportInitiationStatus(
         TranslateBrowserMetrics::INITIATION_STATUS_SIMILAR_LANGUAGES);
     return;
   }
-
   // Nothing to do if either the language Chrome is in or the language of the
   // page is not supported by the translation server.
   if (target_lang.empty() ||