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

Unified Diff: components/autofill/core/browser/contact_info_unittest.cc

Issue 1931123002: [Autofill] Make PersonalDataManager::OverwriteWith case sensitive. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 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: components/autofill/core/browser/contact_info_unittest.cc
diff --git a/components/autofill/core/browser/contact_info_unittest.cc b/components/autofill/core/browser/contact_info_unittest.cc
index 1330e5387498ecb662e5cc5cde5de8f1480a49c4..c04f148892b3c5011e05f3086e0ed0fafa4e99fc 100644
--- a/components/autofill/core/browser/contact_info_unittest.cc
+++ b/components/autofill/core/browser/contact_info_unittest.cc
@@ -188,22 +188,22 @@ TEST(NameInfoTest, ParsedNamesAreEqual) {
{"Marion", "Mitchell", "Morrison"},
true},
- // Case-insensative comparisons.
+ // Case-sensitive comparisons.
{{"Marion", "Mitchell", "Morrison"},
{"Marion", "Mitchell", "MORRISON"},
- true},
+ false},
{{"Marion", "Mitchell", "Morrison"},
{"MARION", "Mitchell", "MORRISON"},
- true},
+ false},
{{"Marion", "Mitchell", "Morrison"},
{"MARION", "MITCHELL", "MORRISON"},
- true},
+ false},
{{"Marion", "", "Mitchell Morrison"},
{"MARION", "", "MITCHELL MORRISON"},
- true},
+ false},
{{"Marion Mitchell", "", "Morrison"},
{"MARION MITCHELL", "", "MORRISON"},
- true},
+ false},
// Identical full names but different canonical forms.
{{"Marion", "Mitchell", "Morrison"},
@@ -222,7 +222,7 @@ TEST(NameInfoTest, ParsedNamesAreEqual) {
// Non-ASCII characters.
{{"M\xc3\xa1rion Mitchell", "", "Morrison"},
- {"M\xc3\x81RION MITCHELL", "", "MORRISON"},
+ {"M\xc3\xa1rion Mitchell", "", "Morrison"},
true},
};
« no previous file with comments | « components/autofill/core/browser/contact_info.cc ('k') | components/test/data/autofill/merge/input/addressLinesLessInfo.in » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698