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

Side by Side Diff: ui/base/l10n/l10n_util_unittest.cc

Issue 7086017: Fix the loading of ca@valencia.pak, which is available in (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: one less line Created 9 years, 6 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 unified diff | Download patch | Annotate | Revision Log
« base/i18n/rtl.cc ('K') | « ui/base/l10n/l10n_util.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "build/build_config.h" 5 #include "build/build_config.h"
6 6
7 #if defined(OS_POSIX) && !defined(OS_MACOSX) 7 #if defined(OS_POSIX) && !defined(OS_MACOSX)
8 #include <cstdlib> 8 #include <cstdlib>
9 #endif 9 #endif
10 10
(...skipping 123 matching lines...) Expand 10 before | Expand all | Expand 10 after
134 "en-GB", 134 "en-GB",
135 "fr", 135 "fr",
136 "es-419", 136 "es-419",
137 "es", 137 "es",
138 "zh-TW", 138 "zh-TW",
139 "zh-CN", 139 "zh-CN",
140 "he", 140 "he",
141 "fil", 141 "fil",
142 "nb", 142 "nb",
143 "am", 143 "am",
144 "ca",
145 "ca@valencia",
144 }; 146 };
145 147
146 #if defined(OS_WIN) 148 #if defined(OS_WIN)
147 static const char kLocaleFileExtension[] = ".dll"; 149 static const char kLocaleFileExtension[] = ".dll";
148 #elif defined(OS_POSIX) 150 #elif defined(OS_POSIX)
149 static const char kLocaleFileExtension[] = ".pak"; 151 static const char kLocaleFileExtension[] = ".pak";
150 #endif 152 #endif
151 for (size_t i = 0; i < arraysize(filenames); ++i) { 153 for (size_t i = 0; i < arraysize(filenames); ++i) {
152 FilePath filename = new_locale_dir.AppendASCII( 154 FilePath filename = new_locale_dir.AppendASCII(
153 filenames[i] + kLocaleFileExtension); 155 filenames[i] + kLocaleFileExtension);
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
187 env->SetVar("LC_MESSAGES", "he"); 189 env->SetVar("LC_MESSAGES", "he");
188 env->SetVar("LANG", "nb"); 190 env->SetVar("LANG", "nb");
189 EXPECT_EQ("fr", l10n_util::GetApplicationLocale("")); 191 EXPECT_EQ("fr", l10n_util::GetApplicationLocale(""));
190 env->UnSetVar("LANGUAGE"); 192 env->UnSetVar("LANGUAGE");
191 EXPECT_EQ("es", l10n_util::GetApplicationLocale("")); 193 EXPECT_EQ("es", l10n_util::GetApplicationLocale(""));
192 env->UnSetVar("LC_ALL"); 194 env->UnSetVar("LC_ALL");
193 EXPECT_EQ("he", l10n_util::GetApplicationLocale("")); 195 EXPECT_EQ("he", l10n_util::GetApplicationLocale(""));
194 env->UnSetVar("LC_MESSAGES"); 196 env->UnSetVar("LC_MESSAGES");
195 EXPECT_EQ("nb", l10n_util::GetApplicationLocale("")); 197 EXPECT_EQ("nb", l10n_util::GetApplicationLocale(""));
196 env->UnSetVar("LANG"); 198 env->UnSetVar("LANG");
199
200 SetDefaultLocaleForTest("ca", env.get());
201 EXPECT_EQ("ca", l10n_util::GetApplicationLocale(""));
202
203 SetDefaultLocaleForTest("ca-ES", env.get());
204 EXPECT_EQ("ca", l10n_util::GetApplicationLocale(""));
205
206 SetDefaultLocaleForTest("ca@valencia", env.get());
207 EXPECT_EQ("ca@valencia", l10n_util::GetApplicationLocale(""));
208
209 SetDefaultLocaleForTest("ca_ES@valencia", env.get());
210 EXPECT_EQ("ca@valencia", l10n_util::GetApplicationLocale(""));
211
212 SetDefaultLocaleForTest("ca_ES.UTF8@valencia", env.get());
213 EXPECT_EQ("ca@valencia", l10n_util::GetApplicationLocale(""));
197 #endif // defined(OS_POSIX) && !defined(OS_CHROMEOS) 214 #endif // defined(OS_POSIX) && !defined(OS_CHROMEOS)
198 215
199 SetDefaultLocaleForTest("en-US", env.get()); 216 SetDefaultLocaleForTest("en-US", env.get());
200 EXPECT_EQ("en-US", l10n_util::GetApplicationLocale("")); 217 EXPECT_EQ("en-US", l10n_util::GetApplicationLocale(""));
201 218
202 SetDefaultLocaleForTest("xx", env.get()); 219 SetDefaultLocaleForTest("xx", env.get());
203 EXPECT_EQ("en-US", l10n_util::GetApplicationLocale("")); 220 EXPECT_EQ("en-US", l10n_util::GetApplicationLocale(""));
204 221
205 #if defined(OS_CHROMEOS) 222 #if defined(OS_CHROMEOS)
206 // ChromeOS honors preferred locale first in GetApplicationLocale(), 223 // ChromeOS honors preferred locale first in GetApplicationLocale(),
(...skipping 186 matching lines...) Expand 10 before | Expand all | Expand 10 after
393 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("Latin")); 410 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("Latin"));
394 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("German")); 411 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("German"));
395 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("pt--BR")); 412 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("pt--BR"));
396 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("sl-macedonia")); 413 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("sl-macedonia"));
397 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("@")); 414 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("@"));
398 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("en-US@")); 415 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("en-US@"));
399 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("en-US@x")); 416 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("en-US@x"));
400 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("en-US@x=")); 417 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("en-US@x="));
401 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("en-US@=y")); 418 EXPECT_FALSE(l10n_util::IsValidLocaleSyntax("en-US@=y"));
402 } 419 }
OLDNEW
« base/i18n/rtl.cc ('K') | « ui/base/l10n/l10n_util.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698