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

Unified Diff: chrome/browser/chromeos/drive/search_metadata_unittest.cc

Issue 15945004: Case-insensitive search for non-ASCII characters in auto-complete of Drive files. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 7 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: chrome/browser/chromeos/drive/search_metadata_unittest.cc
diff --git a/chrome/browser/chromeos/drive/search_metadata_unittest.cc b/chrome/browser/chromeos/drive/search_metadata_unittest.cc
index 3730f64286e17d3b81976805682e872d77b04dd2..e3c12efb1042e786ebed60355a16cc7a648ede70 100644
--- a/chrome/browser/chromeos/drive/search_metadata_unittest.cc
+++ b/chrome/browser/chromeos/drive/search_metadata_unittest.cc
@@ -574,7 +574,8 @@ TEST(SearchMetadataSimpleTest, FindAndHighlight_InTheMiddle) {
TEST(SearchMetadataSimpleTest, FindAndHighlight_MultipeMatches) {
std::string highlighted_text;
EXPECT_TRUE(FindAndHighlight("yoyoyoyoy", "yoy", &highlighted_text));
- EXPECT_EQ("<b>yoy</b>o<b>yoy</b>oy", highlighted_text);
+ // Only the first match is highlighted.
+ EXPECT_EQ("<b>yoy</b>oyoyoy", highlighted_text);
}
TEST(SearchMetadataSimpleTest, FindAndHighlight_IgnoreCase) {
@@ -583,6 +584,17 @@ TEST(SearchMetadataSimpleTest, FindAndHighlight_IgnoreCase) {
EXPECT_EQ("<b>HeLLo</b>", highlighted_text);
}
+TEST(SearchMetadataSimpleTest, FindAndHighlight_IgnoreCaseNonASCII) {
+ std::string highlighted_text;
+ // Find "socra" in "Socra'tes", in Greek.
+ EXPECT_TRUE(FindAndHighlight(
+ "\xCE\xA3\xCF\x89\xCE\xBA\xCF\x81\xCE\xAC\xCF\x84\xCE\xB7\xCF\x82",
+ "\xCF\x83\xCF\x89\xCE\xBA\xCF\x81\xCE\xB1", &highlighted_text));
+ EXPECT_EQ(
+ "<b>\xCE\xA3\xCF\x89\xCE\xBA\xCF\x81\xCE\xAC</b>\xCF\x84\xCE\xB7\xCF\x82",
+ highlighted_text);
+}
+
TEST(SearchMetadataSimpleTest, FindAndHighlight_MetaChars) {
std::string highlighted_text;
EXPECT_TRUE(FindAndHighlight("<hello>", "hello", &highlighted_text));

Powered by Google App Engine
This is Rietveld 408576698