Translate (Linux): add after infobar UI for server side language detection

chrome/browser/translate/translate_infobar_delegate.cc

Index: chrome/browser/translate/translate_infobar_delegate.cc
diff --git a/chrome/browser/translate/translate_infobar_delegate.cc b/chrome/browser/translate/translate_infobar_delegate.cc
index 579664db0d72582b21055062e9fe765835541cde..19290079016fe76be8dfa31b11ca844c2712929c 100644
--- a/chrome/browser/translate/translate_infobar_delegate.cc
+++ b/chrome/browser/translate/translate_infobar_delegate.cc
@@ -280,37 +280,39 @@ string16 TranslateInfoBarDelegate::GetLanguageDisplayableName(
 
 // static
 void TranslateInfoBarDelegate::GetAfterTranslateStrings(
-    std::vector<string16>* strings, bool* swap_languages) {
+    std::vector<string16>* strings,
+    bool* swap_languages,
+    bool autodetermined_source_language) {
   DCHECK(strings);
-  DCHECK(swap_languages);
 
-  std::vector<size_t> offsets;
-  string16 text =
-      l10n_util::GetStringFUTF16(IDS_TRANSLATE_INFOBAR_AFTER_MESSAGE,
-                                 string16(), string16(), &offsets);
-  DCHECK_EQ(2U, offsets.size());
+  if (autodetermined_source_language) {
+    size_t offset;
+    string16 text = l10n_util::GetStringFUTF16(
+        IDS_TRANSLATE_INFOBAR_AFTER_MESSAGE_AUTODETERMINED_SOURCE_LANGUAGE,
+        string16(),
+        &offset);
 
-  *swap_languages = (offsets[0] > offsets[1]);
-  if (*swap_languages)
-    std::swap(offsets[0], offsets[1]);
-
-  strings->push_back(text.substr(0, offsets[0]));
-  strings->push_back(text.substr(offsets[0], offsets[1] - offsets[0]));
-  strings->push_back(text.substr(offsets[1]));
-}
-
-// static
-void TranslateInfoBarDelegate::GetAfterTranslateWithAutoStrings(
-    std::vector<string16>* strings) {
-  DCHECK(strings);
-
-  size_t offset;
-  string16 text =
-      l10n_util::GetStringFUTF16(IDS_TRANSLATE_INFOBAR_AFTER_MESSAGE_WITH_AUTO,
-                                 string16(), &offset);
-
-  strings->push_back(text.substr(0, offset));
-  strings->push_back(text.substr(offset));
+    strings->push_back(text.substr(0, offset));
+    strings->push_back(text.substr(offset));
+  } else {

[Peter Kasting, 2013/05/23 19:59:56]

Tiny nit: Might be better to early-return here so the rest of the function can
be unindented (and look exactly as it did previously)

[Takashi Toyoshima, 2013/05/27 06:07:51]

Done.

+    DCHECK(swap_languages);
+
+    std::vector<size_t> offsets;
+    string16 text = l10n_util::GetStringFUTF16(
+        IDS_TRANSLATE_INFOBAR_AFTER_MESSAGE,
+        string16(),
+        string16(),
+        &offsets);
+    DCHECK_EQ(2U, offsets.size());
+
+    *swap_languages = (offsets[0] > offsets[1]);
+    if (*swap_languages)
+      std::swap(offsets[0], offsets[1]);
+
+    strings->push_back(text.substr(0, offsets[0]));
+    strings->push_back(text.substr(offsets[0], offsets[1] - offsets[0]));
+    strings->push_back(text.substr(offsets[1]));
+  }
 }
 
 TranslateInfoBarDelegate::TranslateInfoBarDelegate(