Copy the implementations of TranslateUIDelegate to TransalteInfobarDelegate

chrome/browser/translate/translate_ui_delegate.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_TRANSLATE_TRANSLATE_UI_DELEGATE_H_
 #define CHROME_BROWSER_TRANSLATE_TRANSLATE_UI_DELEGATE_H_
 
 #include <string>
 #include <vector>
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/strings/string16.h"
 #include "chrome/common/chrome_constants.h"
 
 class TranslatePrefs;
 
 namespace content {
 class WebContents;
 }  // namespace content
 
 // The TranslateUIDelegate is a generic delegate for UI which offers Translate
 // feature to the user.
 class TranslateUIDelegate {
  public:
   enum {
     NO_INDEX = -1,
   };
 
+  // A pair whose key is a language code and value is a language displyayable

[Peter Kasting, 2013/11/08 23:47:47]

Nit: a language displyayable -> the language's displayable

[hajimehoshi, 2013/11/11 04:01:02]

Done.

+  // name.
+  typedef std::pair<std::string, string16> LanguageNamePair;
+
   TranslateUIDelegate(content::WebContents* web_contents,
                       const std::string& original_language,
                       const std::string& target_language);
   virtual ~TranslateUIDelegate();
 
+  // Gets the language names. The result is set |language_names| and is sorted
+  // in alphabetical order taking |locale| into account.

[Peter Kasting, 2013/11/08 23:47:47]

I have no idea what language names this is returning or what "sorted taking
|locale| into account" means.

[hajimehoshi, 2013/11/11 04:01:02]

Done.

+  static void GetSortedLanguageNames(
+      const std::string& locale,
+      std::vector<LanguageNamePair>* language_names);

[Peter Kasting, 2013/11/08 23:47:47]

Don't return via outparam, return by value.  RVO will make this fast.

[hajimehoshi, 2013/11/11 04:01:02]

Done.

+
+  // Gets the host of the page of |web_content|, or an empty string if no URL is

[Peter Kasting, 2013/11/08 23:47:47]

Nit: Call this |web_contents|

[hajimehoshi, 2013/11/11 04:01:02]

Done.

+  // associated with the current page.
+  static std::string GetPageHost(content::WebContents* web_content);
+
   content::WebContents* web_contents() { return web_contents_; }
 
   // Returns the number of languages supported.
   size_t GetNumberOfLanguages() const;
 
   // Returns the original language index.
   size_t GetOriginalLanguageIndex() const;
 
   // Updates the original language index.
   void UpdateOriginalLanguageIndex(size_t language_index);
@@ skipping 39 matching lines @@
 
   // Returns true if the webpage in the current original language should be
   // translated into the current target language automatically.
   bool ShouldAlwaysTranslate();
 
   // Sets the value if the webpage in the current original language should be
   // translated into the current target language automatically.
   void SetAlwaysTranslate(bool value);
 
  private:
-  // Gets the host of the page being translated, or an empty string if no URL is
-  // associated with the current page.
-  std::string GetPageHost();
-
   content::WebContents* web_contents_;
 
-  typedef std::pair<std::string, string16> LanguageNamePair;
-
   // The list supported languages for translation.
   // The pair first string is the language ISO code (ex: en, fr...), the second
   // string is the displayable name on the current locale.
   // The languages are sorted alphabetically based on the displayable name.
   std::vector<LanguageNamePair> languages_;
 
   // The index for language the page is originally in.
   size_t original_language_index_;
 
-  // The index for language the page is originally in that was originally
-  // reported (original_language_index_ changes if the user selects a new
-  // original language, but this one does not).  This is necessary to report
-  // language detection errors with the right original language even if the user
-  // changed the original language.
-  size_t initial_original_language_index_;
-
   // The index for language the page should be translated to.
   size_t target_language_index_;
 
   // The translation related preferences.
   scoped_ptr<TranslatePrefs> prefs_;
 
   DISALLOW_COPY_AND_ASSIGN(TranslateUIDelegate);
 };
 
 #endif  // CHROME_BROWSER_TRANSLATE_TRANSLATE_UI_DELEGATE_H_