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

Unified Diff: chrome/browser/translate/translate_manager_unittest.cc

Issue 3013010: Adding alpha supported languages to Chrome. (Closed) Base URL: http://src.chromium.org/git/chromium.git
Patch Set: Created 10 years, 5 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 | « chrome/browser/translate/translate_manager.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/translate/translate_manager_unittest.cc
diff --git a/chrome/browser/translate/translate_manager_unittest.cc b/chrome/browser/translate/translate_manager_unittest.cc
index 88710a9d3029a7bb7d91c8ebbd5f43371326d0d0..47c83b722de754701a967cd3e7f4aa56c2b7b7ab 100644
--- a/chrome/browser/translate/translate_manager_unittest.cc
+++ b/chrome/browser/translate/translate_manager_unittest.cc
@@ -429,7 +429,7 @@ TEST_F(TranslateManagerTest, TranslateUnknownLanguage) {
TEST_F(TranslateManagerTest, TestAllLanguages) {
// The index in kExpectation are the Language enum (see languages.pb.h).
// true if we expect a translate infobar for that language.
- // Note the supported languages are in translation_service.cc, see
+ // Note the supported languages are in translation_manager.cc, see
// kSupportedLanguages.
bool kExpectations[] = {
// 0-9
@@ -443,15 +443,15 @@ TEST_F(TranslateManagerTest, TestAllLanguages) {
// 40-49
true, false, true, false, false, true, false, true, false, false,
// 50-59
- false, false, false, true, true, true, false, false, false, false,
+ true, false, false, true, true, true, false, true, false, false,
// 60-69
false, false, true, true, false, true, true, false, true, true,
// 70-79
- false, false, false, false, false, false, false, true, false, false,
+ false, false, false, false, true, true, false, true, false, false,
// 80-89
false, false, false, false, false, false, false, false, false, false,
// 90-99
- false, true, false, false, false, false, false, false, false, false,
+ false, true, false, false, false, false, false, true, false, false,
// 100-109
false, true, false, false, false, false, false, false, false, false,
// 110-119
@@ -459,7 +459,7 @@ TEST_F(TranslateManagerTest, TestAllLanguages) {
// 120-129
false, false, false, false, false, false, false, false, false, false,
// 130-139
- false, false, false, false, false, false, false, false, false, false,
+ false, false, false, false, false, false, false, false, false, true,
// 140-149
false, false, false, false, false, false, false, false, false, false,
// 150-159
« no previous file with comments | « chrome/browser/translate/translate_manager.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698