Remove base version of GetCanonicalLocale() function.

base/i18n/rtl.h

 // Copyright (c) 2011 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 BASE_I18N_RTL_H_
 #define BASE_I18N_RTL_H_
 
 #include <string>
 
 #include "base/compiler_specific.h"
 #include "base/i18n/base_i18n_export.h"
 #include "base/strings/string16.h"
 #include "build/build_config.h"
+#include "third_party/icu/source/common/unicode/locid.h"

[tfarina, 2014/05/05 17:54:32]

I was not able to do:

namespace icu {
class Locale;
}

It gave me an error about redefinition or something :/

[jungshik at Google, 2014/05/05 21:49:23]

What happens if you try 'namespace icu46'? Not that I want you to do that, but
....

 
 namespace base {
 
 class FilePath;
 
 namespace i18n {
 
 const char16 kRightToLeftMark = 0x200F;
 const char16 kLeftToRightMark = 0x200E;
 const char16 kLeftToRightEmbeddingMark = 0x202A;
 const char16 kRightToLeftEmbeddingMark = 0x202B;
 const char16 kPopDirectionalFormatting = 0x202C;
 const char16 kLeftToRightOverride = 0x202D;
 const char16 kRightToLeftOverride = 0x202E;
 
 // Locale.java mirrored this enum TextDirection. Please keep in sync.
 enum TextDirection {
   UNKNOWN_DIRECTION = 0,
   RIGHT_TO_LEFT = 1,
   LEFT_TO_RIGHT = 2,
   TEXT_DIRECTION_NUM_DIRECTIONS = 3,
 };
 
+BASE_I18N_EXPORT std::string GetLocaleString(const icu::Locale& locale);
+
 // Get the locale that the currently running process has been configured to use.
 // The return value is of the form language[-country] (e.g., en-US) where the
 // language is the 2 or 3 letter code from ISO-639.
 BASE_I18N_EXPORT std::string GetConfiguredLocale();
 
-// Canonicalize a string (eg. a POSIX locale string) to a Chrome locale name.
-BASE_I18N_EXPORT std::string GetCanonicalLocale(const char* locale);
-
 // Sets the default locale of ICU.
 // Once the application locale of Chrome in GetApplicationLocale is determined,
 // the default locale of ICU need to be changed to match the application locale
 // so that ICU functions work correctly in a locale-dependent manner.
 // This is handy in that we don't have to call GetApplicationLocale()
 // everytime we call locale-dependent ICU APIs as long as we make sure
 // that this is called before any locale-dependent API is called.
 BASE_I18N_EXPORT void SetICUDefaultLocale(const std::string& locale_string);
 
 // Returns true if the application text direction is right-to-left.
@@ skipping 85 matching lines @@
 // return the text itself. Explicit bidi control characters display and have
 // semantic effect. They can be deleted so they might not always appear in a
 // pair.
 BASE_I18N_EXPORT string16 StripWrappingBidiControlCharacters(
     const string16& text) WARN_UNUSED_RESULT;
 
 }  // namespace i18n
 }  // namespace base
 
 #endif  // BASE_I18N_RTL_H_