Remove text encoding UI

chrome/browser/character_encoding.h

Index: chrome/browser/character_encoding.h
diff --git a/chrome/browser/character_encoding.h b/chrome/browser/character_encoding.h
index 2c70a178dd7f2a2d4180ff3ca604ad9d04784740..2820d62b02e6d8e0a2f1f0355764843672924ff6 100644
--- a/chrome/browser/character_encoding.h
+++ b/chrome/browser/character_encoding.h
@@ -6,101 +6,10 @@
 #define CHROME_BROWSER_CHARACTER_ENCODING_H_
 
 #include <string>
-#include <vector>
 
 #include "base/macros.h"

[msw, 2016/08/23 23:10:55]

nit: not needed.

[Jinsuk Kim, 2016/08/24 08:23:42]

Moved to .cc (for |arraysize|)

-#include "base/strings/string16.h"
 
-class CharacterEncoding {
- public:
-  // Enumeration of the types of Browser encoding name we
-  // currently support. This is defined outside of Browser
-  // to avoid cyclical dependencies.
-
-  // Structure to save encoding information.
-  struct EncodingInfo {
-    explicit EncodingInfo(int id);
-    // Gets string key of EncodingInfo. With this method, we can use
-    // l10n_util::SortVectorWithStringKey to sort the encoding menu items
-    // by current locale character sequence. We need to keep the order within
-    // encoding category name, that's why we use category name as key.
-    const base::string16& GetStringKey() const { return encoding_category_name; }
-
-    // Encoding command id.
-    int encoding_id;
-    // Encoding display name.
-    base::string16 encoding_display_name;
-    // Encoding category name.
-    base::string16 encoding_category_name;
-  };
-
-  // Return canonical encoding name according to the command ID.
-  // THIS FUNCTION IS NOT THREADSAFE. You must run this function
-  // only in UI thread.
-  static std::string GetCanonicalEncodingNameByCommandId(int id);
-
-  // Return display name of canonical encoding according to the command
-  // ID. THIS FUNCTION IS NOT THREADSAFE. You must run this function
-  // only in UI thread.
-  static base::string16 GetCanonicalEncodingDisplayNameByCommandId(int id);
-
-  // Return count number of all supported canonical encoding.
-  static int GetSupportCanonicalEncodingCount();
-
-  // Return canonical encoding name according to the index, which starts
-  // from zero to GetSupportCanonicalEncodingCount() - 1. THIS FUNCTION
-  // IS NOT THREADSAFE. You must run this function only in UI thread.
-  static std::string GetCanonicalEncodingNameByIndex(int index);
-
-  // Return display name of canonical encoding according to the index,
-  // which starts from zero to GetSupportCanonicalEncodingCount() - 1.
-  // THIS FUNCTION IS NOT THREADSAFE. You must run this function
-  // only in UI thread.
-  static base::string16 GetCanonicalEncodingDisplayNameByIndex(int index);
-
-  // Return encoding command id according to the index, which starts from
-  // zero to GetSupportCanonicalEncodingCount() - 1. Otherwise returns 0.
-  static int GetEncodingCommandIdByIndex(int index);
-
-  // Return canonical encoding name according to the encoding alias name. THIS
-  // FUNCTION IS NOT THREADSAFE. You must run this function only in UI thread.
-  static std::string GetCanonicalEncodingNameByAliasName(
-      const std::string& alias_name);
-
-  // Returns the pointer of a vector of EncodingInfos corresponding to
-  // encodings to display in the encoding menu. The locale-dependent static
-  // encodings come at the top of the list and recently selected encodings
-  // come next. Finally, the rest of encodings are listed.
-  // The vector will be created and destroyed by CharacterEncoding.
-  // The returned std::vector is maintained by this class. The parameter
-  // |locale| points to the current application (UI) locale. The parameter
-  // |locale_encodings| is string of static encodings list which is from the
-  // corresponding string resource that is stored in the resource bundle.
-  // The parameter |recently_select_encodings| is string of encoding list which
-  // is from user recently selected. THIS FUNCTION IS NOT THREADSAFE. You must
-  // run this function only in UI thread.
-  static const std::vector<EncodingInfo>* GetCurrentDisplayEncodings(
-      const std::string& locale,
-      const std::string& locale_encodings,
-      const std::string& recently_select_encodings);
-
-  // This function is for updating |original_selected_encoding_list| with a
-  // |new_selected_encoding_id|. If the encoding is already in the original
-  // list, then returns false. Otherwise |selected_encoding_list| will return a
-  // new string for user selected encoding short list and function returns true.
-  static bool UpdateRecentlySelectedEncoding(
-      const std::string& original_selected_encodings,
-      int new_selected_encoding_id,
-      std::string* selected_encodings);
-
-  // Get encoding command id according to input encoding name. If the name is
-  // valid, return corresponding encoding command id. Otherwise return 0;
-  static int GetCommandIdByCanonicalEncodingName(
-      const std::string& encoding_name);
-
- private:
-  // Disallow instantiating it since this class only contains static methods.
-  DISALLOW_IMPLICIT_CONSTRUCTORS(CharacterEncoding);
-};
+// Return canonical encoding name according to the encoding alias name.
+std::string GetCanonicalEncodingNameByAliasName(const std::string& alias_name);
 
 #endif  // CHROME_BROWSER_CHARACTER_ENCODING_H_