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

Unified Diff: third_party/libaddressinput/chromium/cpp/test/rule_test.cc

Issue 237693002: requestAutocomplete should not unlatinize administrative region names. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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
Index: third_party/libaddressinput/chromium/cpp/test/rule_test.cc
diff --git a/third_party/libaddressinput/chromium/cpp/test/rule_test.cc b/third_party/libaddressinput/chromium/cpp/test/rule_test.cc
index ae3cfc359427cb2174276d502d7b5b378e5116a3..860c1a56e65415b74156e9d761a2ada76f1d7882 100644
--- a/third_party/libaddressinput/chromium/cpp/test/rule_test.cc
+++ b/third_party/libaddressinput/chromium/cpp/test/rule_test.cc
@@ -329,19 +329,25 @@ TEST(RuleTest, ParseSubKeyTest) {
EXPECT_EQ(3U, rule.GetSubKeys().size());
std::string sub_key;
- EXPECT_TRUE(rule.CanonicalizeSubKey("BAR", &sub_key));
+ EXPECT_TRUE(rule.CanonicalizeSubKey("BAR", true, &sub_key));
EXPECT_EQ("BAR", sub_key);
sub_key.clear();
- EXPECT_TRUE(rule.CanonicalizeSubKey("Bartopolis", &sub_key));
+ EXPECT_TRUE(rule.CanonicalizeSubKey("Bartopolis", true, &sub_key));
EXPECT_EQ("BAR", sub_key);
sub_key.clear();
- EXPECT_TRUE(rule.CanonicalizeSubKey("Bartopolis2", &sub_key));
+ // Unlatinize.
+ EXPECT_TRUE(rule.CanonicalizeSubKey("Bartopolis2", true, &sub_key));
EXPECT_EQ("BAR", sub_key);
sub_key.clear();
- EXPECT_FALSE(rule.CanonicalizeSubKey("Beertopia", &sub_key));
+ // Should not unlatinize.
+ EXPECT_TRUE(rule.CanonicalizeSubKey("Bartopolis2", false, &sub_key));
+ EXPECT_EQ("", sub_key);
+ sub_key.clear();
+
+ EXPECT_FALSE(rule.CanonicalizeSubKey("Beertopia", true, &sub_key));
EXPECT_EQ("", sub_key);
}

Powered by Google App Engine
This is Rietveld 408576698