Translate: display alpha language name with (alpha) suffix

chrome/browser/translate/translate_language_list.h

Index: chrome/browser/translate/translate_language_list.h
diff --git a/chrome/browser/translate/translate_language_list.h b/chrome/browser/translate/translate_language_list.h
index c9178648258474ebd62f0f51efbd3e46d38704b0..32b0979a294346f7212b8dc75b2019055d641fd0 100644
--- a/chrome/browser/translate/translate_language_list.h
+++ b/chrome/browser/translate/translate_language_list.h
@@ -38,7 +38,9 @@ class TranslateLanguageList : public net::URLFetcherDelegate {
   // Returns true if |language| is supported by the translation server.
   bool IsSupportedLanguage(const std::string& language);
 
-  // TODO(toyoshim): Add IsSupportedAlphaLanguage() here.
+  // Returns true if |language| is supported by the translation server as a
+  // alpha language.
+  bool IsAlphaLanguage(const std::string& language);
 
   // Fetches the language list from the translate server. It will not retry
   // more than kMaxRetryLanguageListFetch times.
@@ -53,12 +55,23 @@ class TranslateLanguageList : public net::URLFetcherDelegate {
   // languages that the translate server can translate to and from.
   void SetSupportedLanguages(const std::string& language_list);
 
+  // Fetches the language list. If |alpha| is set, it requests a language list
+  // including alpha languages.
+  void FetchLanguageList(bool alpha);
+
   // The languages supported by the translation server.
   std::set<std::string> supported_languages_;
 
+  // The alpha languages supported by the translation server.
+  std::set<std::string> supported_alpha_languages_;
+
   // An URLFetcher instance to fetch a server providing supported language list.
   scoped_ptr<net::URLFetcher> url_fetcher_;
 
+  // Flag if |ufl_fetcher_| is requesting a language list including alpha

[MAD, 2013/05/29 14:09:32]

I usually try to avoid such flags, it's a bit risky. For example, you never
reset it to false after setting it to true, so if we would ever need to fetch
for languages again, we may have bugs.

I would prefer that you subclass the URLFetcher and add the boolean in there. It
will make things clearer.

[Takashi Toyoshima, 2013/05/30 06:25:31]

Done.

+  // languages.
+  bool requesting_alpha_;
+
   DISALLOW_COPY_AND_ASSIGN(TranslateLanguageList);
 };