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

Unified Diff: chrome/browser/chromeos/login/language_switch_menu.cc

Issue 5990008: Remove wstring from l10n_util. Part 1.... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 10 years 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
Index: chrome/browser/chromeos/login/language_switch_menu.cc
===================================================================
--- chrome/browser/chromeos/login/language_switch_menu.cc (revision 70069)
+++ chrome/browser/chromeos/login/language_switch_menu.cc (working copy)
@@ -47,8 +47,7 @@
// Fill menu items with updated items.
for (int line = 0; line != kLanguageMainMenuSize; line++) {
- menu_model_.AddItem(
- line, WideToUTF16(language_list_->GetLanguageNameAt(line)));
+ menu_model_.AddItem(line, language_list_->GetLanguageNameAt(line));
}
menu_model_.AddSeparator();
menu_model_.AddSubMenuWithStringId(kMoreLanguagesSubMenu,
@@ -57,14 +56,14 @@
for (int line = kLanguageMainMenuSize;
line != language_list_->get_languages_count(); line++) {
menu_model_submenu_.AddItem(
- line, WideToUTF16(language_list_->GetLanguageNameAt(line)));
+ line, language_list_->GetLanguageNameAt(line));
}
// Initialize menu here so it appears fast when called.
menu_.reset(new views::Menu2(&menu_model_));
}
-std::wstring LanguageSwitchMenu::GetCurrentLocaleName() const {
+string16 LanguageSwitchMenu::GetCurrentLocaleName() const {
DCHECK(g_browser_process);
const std::string locale = g_browser_process->GetApplicationLocale();
int index = language_list_->GetIndexFromLocale(locale);

Powered by Google App Engine
This is Rietveld 408576698