Translate: Adding a flag to enable "alpha language" translation

chrome/browser/translate/translate_manager.cc

Index: chrome/browser/translate/translate_manager.cc
diff --git a/chrome/browser/translate/translate_manager.cc b/chrome/browser/translate/translate_manager.cc
index c2a8424a1540c86bdfe277b385037484f2031b72..c2ed9428d888e6e1f2ec9ec2e34f7e18ae8d52ea 100644
--- a/chrome/browser/translate/translate_manager.cc
+++ b/chrome/browser/translate/translate_manager.cc
@@ -141,7 +141,9 @@ const char* const kTranslateScriptHeader =
 const char* const kReportLanguageDetectionErrorURL =
     "https://translate.google.com/translate_error";
 const char* const kLanguageListFetchURL =
-    "https://translate.googleapis.com/translate_a/l?client=chrome&cb=sl&hl=%s";
+    "https://translate.googleapis.com/translate_a/l"
+    "?client=chrome&cb=sl&hl=%s%s";

[MAD, 2013/04/05 14:18:34]

At this point, I think we should start using  AppendQueryParameter or manual use
of the url_canon::Replacements class to incrementally add query params for us,
this would make the code below cleaner and this string a little less cryptic.

[Takashi Toyoshima, 2013/04/08 03:39:05]

Agreed.

I'll apply AppendQueryParameter on modifying kLanguageListFetchURL in this CL,
and will make another CL to use it for others, e.g., AddApiKeyToUrl.

+const char* const kAlphaLanguageQuery = "&alpha=1";
 const int kMaxRetryLanguageListFetch = 5;
 const int kTranslateScriptExpirationDelayDays = 1;
 
@@ -776,11 +778,17 @@ void TranslateManager::FetchLanguageListFromTranslateServer(
     return;
   }
 
+  const CommandLine& command_line = *CommandLine::ForCurrentProcess();
   GURL language_list_fetch_url = GURL(
       base::StringPrintf(
           kLanguageListFetchURL,
-          GetLanguageCode(g_browser_process->GetApplicationLocale()).c_str()));
+          GetLanguageCode(g_browser_process->GetApplicationLocale()).c_str(),
+          command_line.HasSwitch(switches::kEnableTranslateAlphaLanguages) ?
+              kAlphaLanguageQuery : ""));
   AddApiKeyToUrl(&language_list_fetch_url);
+  VLOG(9) << "Fetch supporting language list from: "
+          << language_list_fetch_url.spec().c_str();
+
   language_list_request_pending_.reset(net::URLFetcher::Create(
       1, language_list_fetch_url, net::URLFetcher::GET, this));
   language_list_request_pending_->SetLoadFlags(net::LOAD_DO_NOT_SEND_COOKIES |