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

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

Issue 3024005: Google translate supports some new languages, makes Chrome aware of them. Tha... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
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
===================================================================
--- chrome/browser/translate/translate_manager_unittest.cc (revision 52842)
+++ chrome/browser/translate/translate_manager_unittest.cc (working copy)
@@ -429,7 +429,7 @@
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 @@
// 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 @@
// 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