[Autofill] Dedupe similar profiles on insertion.

components/autofill/core/browser/autofill_profile.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 <algorithm>
 #include <functional>
 #include <map>
 #include <memory>
@@ skipping 414 matching lines @@
 
 bool AutofillProfile::operator==(const AutofillProfile& profile) const {
   return guid() == profile.guid() && EqualsSansGuid(profile);
 }
 
 bool AutofillProfile::operator!=(const AutofillProfile& profile) const {
   return !operator==(profile);
 }
 
 const base::string16 AutofillProfile::PrimaryValue() const {
-  return GetRawInfo(NAME_FIRST) + GetRawInfo(NAME_LAST) +
-         GetRawInfo(ADDRESS_HOME_LINE1) + GetRawInfo(ADDRESS_HOME_CITY);
+  std::vector<std::string> primary_values{
+      base::UTF16ToUTF8(GetRawInfo(NAME_FIRST)),
+      base::UTF16ToUTF8(GetRawInfo(NAME_LAST)),
+      base::UTF16ToUTF8(GetRawInfo(ADDRESS_HOME_LINE1)),
+      base::UTF16ToUTF8(GetRawInfo(ADDRESS_HOME_CITY))};

[Roger McFarlane (Chromium), 2016/06/08 18:55:38]

GetInfo() instead of GetRawInfo()?

[sebsg, 2016/06/08 21:58:46]

Done.

+  return base::UTF8ToUTF16(base::JoinString(primary_values, " "));

[Mathieu, 2016/06/08 18:32:24]

can't you use the string16 version of JoinString? Right now going from 16->8->16

[sebsg, 2016/06/08 21:58:46]

Done.

 }
 
 bool AutofillProfile::IsSubsetOf(const AutofillProfile& profile,
                                  const std::string& app_locale) const {
   ServerFieldTypeSet types;
   GetSupportedTypes(&types);
   return IsSubsetOfForFieldSet(profile, app_locale, types);
 }
 
 bool AutofillProfile::IsSubsetOfForFieldSet(
@@ skipping 637 matching lines @@
             << " " << UTF16ToUTF8(profile.GetRawInfo(ADDRESS_HOME_CITY)) << " "
             << UTF16ToUTF8(profile.GetRawInfo(ADDRESS_HOME_STATE)) << " "
             << UTF16ToUTF8(profile.GetRawInfo(ADDRESS_HOME_ZIP)) << " "
             << UTF16ToUTF8(profile.GetRawInfo(ADDRESS_HOME_SORTING_CODE)) << " "
             << UTF16ToUTF8(profile.GetRawInfo(ADDRESS_HOME_COUNTRY)) << " "
             << profile.language_code() << " "
             << UTF16ToUTF8(profile.GetRawInfo(PHONE_HOME_WHOLE_NUMBER));
 }
 
 }  // namespace autofill