Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(330)

Unified Diff: ui/base/l10n/l10n_util.cc

Issue 10380018: Revert 135484 - Use Android API for GetDisplayNameForLocale(). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ui/base/l10n/l10n_util.h ('k') | ui/base/l10n/l10n_util_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/base/l10n/l10n_util.cc
===================================================================
--- ui/base/l10n/l10n_util.cc (revision 135489)
+++ ui/base/l10n/l10n_util.cc (working copy)
@@ -27,10 +27,6 @@
#include "unicode/rbbi.h"
#include "unicode/uloc.h"
-#if defined(OS_ANDROID)
-#include "base/android/locale_utils.h"
-#endif
-
#if defined(OS_LINUX)
#include <glib.h>
#endif
@@ -478,49 +474,26 @@
// #1 and #2 wouldn't work if display_locale != current UI locale although
// we can think of additional hack to work around the problem.
// #3 can be potentially expensive.
- bool is_zh = false;
- if (locale_code == "zh-CN") {
+ if (locale_code == "zh-CN")
locale_code = "zh-Hans";
- is_zh = true;
- } else if (locale_code == "zh-TW") {
+ else if (locale_code == "zh-TW")
locale_code = "zh-Hant";
- is_zh = true;
- }
- string16 display_name;
-#if defined(OS_ANDROID)
- // Use Java API to get locale display name so that we can remove most of
- // the lang data from icu data to reduce binary size, except for zh-Hans and
- // zh-Hant because the current Android Java API doesn't support scripts.
- // TODO(wangxianzhu): remove the special handling of zh-Hans and zh-Hant once
- // Android Java API supports scripts.
- if (!is_zh) {
- display_name = base::android::GetDisplayNameForLocale(locale_code,
- display_locale);
- } else
-#endif
- {
- UErrorCode error = U_ZERO_ERROR;
- const int kBufferSize = 1024;
+ UErrorCode error = U_ZERO_ERROR;
+ const int kBufferSize = 1024;
- int actual_size = uloc_getDisplayName(locale_code.c_str(),
- display_locale.c_str(),
- WriteInto(&display_name, kBufferSize), kBufferSize - 1, &error);
- DCHECK(U_SUCCESS(error));
- display_name.resize(actual_size);
- }
-
+ string16 display_name;
+ int actual_size = uloc_getDisplayName(locale_code.c_str(),
+ display_locale.c_str(),
+ WriteInto(&display_name, kBufferSize), kBufferSize - 1, &error);
+ DCHECK(U_SUCCESS(error));
+ display_name.resize(actual_size);
// Add an RTL mark so parentheses are properly placed.
if (is_for_ui && base::i18n::IsRTL())
display_name.push_back(static_cast<char16>(base::i18n::kRightToLeftMark));
return display_name;
}
-string16 GetDisplayNameForCountry(const std::string& country_code,
- const std::string& display_locale) {
- return GetDisplayNameForLocale("_" + country_code, display_locale, false);
-}
-
std::string NormalizeLocale(const std::string& locale) {
std::string normalized_locale(locale);
std::replace(normalized_locale.begin(), normalized_locale.end(), '-', '_');
« no previous file with comments | « ui/base/l10n/l10n_util.h ('k') | ui/base/l10n/l10n_util_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698