Use AutofillProfileComparator in place of ad-hoc merge logic.

components/autofill/core/browser/autofill_profile_unittest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/autofill/core/browser/autofill_profile.h"
 
 #include <stddef.h>
 
 #include <memory>
 
@@ skipping 825 matching lines @@
       new AutofillProfile(base::GenerateGUID(), "https://www.example.com/"));
   test::SetProfileInfo(a.get(), "Thomas", NULL, "Jefferson",
       "declaration_guy@gmail.com", NULL, NULL, NULL, NULL, NULL, NULL, NULL,
       NULL);
   test::SetProfileInfo(a.get(), "Thomas", NULL, "Adams",
       "declaration_guy@gmail.com", NULL, NULL, NULL, NULL, NULL, NULL, NULL,
       NULL);
   EXPECT_FALSE(a->IsSubsetOf(*b, "en-US"));
 }
 
-TEST(AutofillProfileTest, OverwriteWith_DifferentProfile) {
+TEST(AutofillProfileTest, MergeDataFrom_DifferentProfile) {
   AutofillProfile a;
   SetupTestProfile(a);
 
   // Create an identical profile except that the new profile:
   //   (1) Has a different origin,
   //   (2) Has a different address line 2,
   //   (3) Lacks a company name,
   //   (4) Has a different full name, and
   //   (5) Has a language code.
   AutofillProfile b = a;
   b.set_guid(base::GenerateGUID());
   b.set_origin(kSettingsOrigin);
-  b.SetRawInfo(ADDRESS_HOME_LINE2, ASCIIToUTF16("area 51"));
+  b.SetRawInfo(ADDRESS_HOME_LINE2, ASCIIToUTF16("Unit 5, area 51"));
   b.SetRawInfo(COMPANY_NAME, base::string16());
 
+  b.SetRawInfo(NAME_MIDDLE, ASCIIToUTF16("M."));
   b.SetRawInfo(NAME_FULL, ASCIIToUTF16("Marion M. Morrison"));
   b.set_language_code("en");
 
-  EXPECT_TRUE(a.OverwriteWith(b, "en-US"));
+  EXPECT_TRUE(a.MergeDataFrom(b, "en-US"));
   EXPECT_EQ(kSettingsOrigin, a.origin());
-  EXPECT_EQ(ASCIIToUTF16("area 51"), a.GetRawInfo(ADDRESS_HOME_LINE2));
+  EXPECT_EQ(ASCIIToUTF16("Unit 5, area 51"), a.GetRawInfo(ADDRESS_HOME_LINE2));
   EXPECT_EQ(ASCIIToUTF16("Fox"), a.GetRawInfo(COMPANY_NAME));
   base::string16 name = a.GetInfo(AutofillType(NAME_FULL), "en-US");
-  EXPECT_EQ(ASCIIToUTF16("Marion M. Morrison"), name);
+  EXPECT_EQ(ASCIIToUTF16("Marion Mitchell Morrison"), name);
   EXPECT_EQ("en", a.language_code());
 }
 
-TEST(AutofillProfileTest, OverwriteWith_SameProfile) {
+TEST(AutofillProfileTest, MergeDataFrom_SameProfile) {
   AutofillProfile a;
   SetupTestProfile(a);
 
+  // The profile has no full name yet. Merge will add it.
   AutofillProfile b = a;
   b.set_guid(base::GenerateGUID());
-  EXPECT_FALSE(a.OverwriteWith(b, "en-US"));
-}
+  EXPECT_TRUE(a.MergeDataFrom(b, "en-US"));
+  EXPECT_EQ(2u, a.use_count());
 
-TEST(AutofillProfileTest, OverwriteWith_DifferentName) {
-  AutofillProfile a;
-  SetupTestProfile(a);
-
-  AutofillProfile b = a;
-  b.SetRawInfo(NAME_FIRST, ASCIIToUTF16("Mario"));
-  EXPECT_TRUE(a.OverwriteWith(b, "en-US"));
-  base::string16 name_full = a.GetInfo(AutofillType(NAME_FULL), "en-US");
-  EXPECT_EQ(ASCIIToUTF16("Mario Mitchell Morrison"), name_full);
-}
-
-TEST(AutofillProfileTest, OverwriteWith_DifferentAddress) {
-  AutofillProfile a;
-  SetupTestProfile(a);
-
-  AutofillProfile b = a;
-  b.SetRawInfo(ADDRESS_HOME_LINE1, ASCIIToUTF16("123 Aquarium St."));
-  EXPECT_TRUE(a.OverwriteWith(b, "en-US"));
-  base::string16 address = a.GetRawInfo(ADDRESS_HOME_LINE1);
-  EXPECT_EQ(ASCIIToUTF16("123 Aquarium St."), address);
+  // Now the profile is fully populated. Merging it again has no effect (except
+  // for usage statistics).
+  AutofillProfile c = a;
+  c.set_guid(base::GenerateGUID());
+  c.set_use_count(3);
+  EXPECT_FALSE(a.MergeDataFrom(c, "en-US"));
+  EXPECT_EQ(5u, a.use_count());
 }
 
 TEST(AutofillProfileTest, OverwriteName_AddNameFull) {
   AutofillProfile a;
 
   a.SetRawInfo(NAME_FIRST, base::ASCIIToUTF16("Marion"));
   a.SetRawInfo(NAME_MIDDLE, base::ASCIIToUTF16("Mitchell"));
   a.SetRawInfo(NAME_LAST, base::ASCIIToUTF16("Morrison"));
 
   AutofillProfile b = a;
   b.SetRawInfo(NAME_FULL, base::ASCIIToUTF16("Marion Mitchell Morrison"));
 
-  EXPECT_TRUE(a.OverwriteWith(b, "en-US"));
+  EXPECT_TRUE(a.MergeDataFrom(b, "en-US"));
   EXPECT_EQ(base::ASCIIToUTF16("Marion"), a.GetRawInfo(NAME_FIRST));
   EXPECT_EQ(base::ASCIIToUTF16("Mitchell"), a.GetRawInfo(NAME_MIDDLE));
   EXPECT_EQ(base::ASCIIToUTF16("Morrison"), a.GetRawInfo(NAME_LAST));
   EXPECT_EQ(base::ASCIIToUTF16("Marion Mitchell Morrison"),
             a.GetRawInfo(NAME_FULL));
 }
 
 // Tests that OverwriteName overwrites the name parts if they have different
 // case.
 TEST(AutofillProfileTest, OverwriteName_DifferentCase) {
   AutofillProfile a;
 
   a.SetRawInfo(NAME_FIRST, base::ASCIIToUTF16("marion"));
   a.SetRawInfo(NAME_MIDDLE, base::ASCIIToUTF16("mitchell"));
   a.SetRawInfo(NAME_LAST, base::ASCIIToUTF16("morrison"));
 
   AutofillProfile b = a;
   b.SetRawInfo(NAME_FIRST, base::ASCIIToUTF16("Marion"));
   b.SetRawInfo(NAME_MIDDLE, base::ASCIIToUTF16("Mitchell"));
   b.SetRawInfo(NAME_LAST, base::ASCIIToUTF16("Morrison"));
 
-  EXPECT_TRUE(a.OverwriteWith(b, "en-US"));
+  EXPECT_TRUE(a.MergeDataFrom(b, "en-US"));
   EXPECT_EQ(base::ASCIIToUTF16("Marion"), a.GetRawInfo(NAME_FIRST));
   EXPECT_EQ(base::ASCIIToUTF16("Mitchell"), a.GetRawInfo(NAME_MIDDLE));
   EXPECT_EQ(base::ASCIIToUTF16("Morrison"), a.GetRawInfo(NAME_LAST));
 }
 
 TEST(AutofillProfileTest, AssignmentOperator) {
   AutofillProfile a(base::GenerateGUID(), "https://www.example.com/");
   test::SetProfileInfo(&a, "Marion", "Mitchell", "Morrison",
                        "marion@me.xyz", "Fox", "123 Zoo St.", "unit 5",
                        "Hollywood", "CA", "91601", "US",
@@ skipping 179 matching lines @@
   profile.SetInfo(AutofillType(ADDRESS_HOME_STATE),
                                ASCIIToUTF16("CA"),
                                "en-US");
   EXPECT_FALSE(profile.GetInfo(full_address, "en-US").empty());
   profile.SetInfo(AutofillType(ADDRESS_HOME_COUNTRY),
                                base::string16(),
                                "en-US");
   EXPECT_TRUE(profile.GetInfo(full_address, "en-US").empty());
 }
 
-TEST(AutofillProfileTest, CanonicalizeProfileString) {
-  // NOP.
-  EXPECT_EQ(base::string16(),
-            AutofillProfile::CanonicalizeProfileString(base::string16()));
-
-  // Simple punctuation removed.
-  EXPECT_EQ(ASCIIToUTF16("1600 amphitheatre pkwy"),
-            AutofillProfile::CanonicalizeProfileString(ASCIIToUTF16(
-                "1600 Amphitheatre, Pkwy.")));
-
-  // Unicode punctuation (hyphen and space), multiple spaces collapsed.
-  EXPECT_EQ(ASCIIToUTF16("mid island plaza"),
-            AutofillProfile::CanonicalizeProfileString(base::WideToUTF16(
-                L"Mid\x2013Island\x2003 Plaza")));
-
-  // Newline character removed.
-  EXPECT_EQ(ASCIIToUTF16("1600 amphitheatre pkwy app 2"),
-            AutofillProfile::CanonicalizeProfileString(
-                ASCIIToUTF16("1600 amphitheatre pkwy \n App. 2")));
-
-  // Diacritics removed.
-  EXPECT_EQ(UTF8ToUTF16("まeoa정"),
-            AutofillProfile::CanonicalizeProfileString(UTF8ToUTF16("まéÖä정")));
-}
-
 TEST(AutofillProfileTest, SaveAdditionalInfo_Name_AddingNameFull) {
   AutofillProfile a;
 
   a.SetRawInfo(NAME_FIRST, base::ASCIIToUTF16("Marion"));
   a.SetRawInfo(NAME_MIDDLE, base::ASCIIToUTF16("Mitchell"));
   a.SetRawInfo(NAME_LAST, base::ASCIIToUTF16("Morrison"));
 
   AutofillProfile b = a;
   b.SetRawInfo(NAME_FULL, base::ASCIIToUTF16("Marion Mitchell Morrison"));
 
@@ skipping 43 matching lines @@
   b.SetRawInfo(NAME_LAST,UTF8ToUTF16("Morrison"));
   b.SetRawInfo(NAME_FULL, UTF8ToUTF16(""));
 
   EXPECT_TRUE(a.SaveAdditionalInfo(b, "en-US"));
 
   // The first, middle and last names should have their first letter in
   // uppercase and have acquired diacritics.
   EXPECT_EQ(UTF8ToUTF16("Märion"), a.GetRawInfo(NAME_FIRST));
   EXPECT_EQ(UTF8ToUTF16("Mitchéll"), a.GetRawInfo(NAME_MIDDLE));
   EXPECT_EQ(UTF8ToUTF16("Morrison"), a.GetRawInfo(NAME_LAST));
-  EXPECT_EQ(UTF8ToUTF16("marion mitchell morrison"),
+  EXPECT_EQ(UTF8ToUTF16("Märion Mitchéll Morrison"),
             a.GetRawInfo(NAME_FULL));
 }
 
 // Tests that no loss of information happens when SavingAdditionalInfo with a
 // profile with an empty name part.
 TEST(AutofillProfileTest, SaveAdditionalInfo_Name_LossOfInformation) {
   AutofillProfile a;
 
   a.SetRawInfo(NAME_FIRST, base::ASCIIToUTF16("Marion"));
   a.SetRawInfo(NAME_MIDDLE, base::ASCIIToUTF16("Mitchell"));
@@ skipping 25 matching lines @@
 
   // The first, middle and last names should be kept and name full should be
   // added.
   EXPECT_EQ(base::ASCIIToUTF16("Marion"), a.GetRawInfo(NAME_FIRST));
   EXPECT_EQ(base::ASCIIToUTF16("Mitchell"), a.GetRawInfo(NAME_MIDDLE));
   EXPECT_EQ(base::ASCIIToUTF16("Morrison"), a.GetRawInfo(NAME_LAST));
   EXPECT_EQ(base::ASCIIToUTF16("Marion Mitchell Morrison"),
             a.GetRawInfo(NAME_FULL));
 }
 
-// Tests that PrimaryValues of two similar profiles with different
-// punctuation and case are equal.
-TEST(AutofillProfileTest, PrimaryValue_SimilarProfiles) {
-  AutofillProfile profile1(base::GenerateGUID(), "https://www.example.com/");
-  test::SetProfileInfo(&profile1, "Marion", "Mitchell", "Morrison",
-                       "marion@me.xyz", "Fox", "123 Zoo St.", "unit 5",
-                       "Hollywood", "CA", "91601", "US", "12345678910");
-
-  AutofillProfile profile2(base::GenerateGUID(), "https://www.example.com/");
-  test::SetProfileInfo(&profile2, "marion", "mitchell", "morrison",
-                       "marion@me.xyz", "Fox", "123, Zoo St", "unit 5",
-                       "hollywood", "CA", "91601", "US", "12345678910");
-
-  EXPECT_EQ(profile1.PrimaryValue("en-US"), profile2.PrimaryValue("en-US"));
-}
-
 }  // namespace autofill