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

Unified Diff: components/autofill/core/browser/contact_info.h

Issue 1973873002: [Autofill] Improve the merging of two profiles' names. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments Created 4 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: components/autofill/core/browser/contact_info.h
diff --git a/components/autofill/core/browser/contact_info.h b/components/autofill/core/browser/contact_info.h
index d2e70ba13f323e51b61923def7eed41eec7b8b50..3f42902b37ef939565ea427627af262326c7889c 100644
--- a/components/autofill/core/browser/contact_info.h
+++ b/components/autofill/core/browser/contact_info.h
@@ -26,6 +26,14 @@ class NameInfo : public FormGroup {
// The comparison is case sensitive.
bool ParsedNamesAreEqual(const NameInfo& info) const;
+ // For every non-empty NameInfo part in |new_name|, the corresponding NameInfo
+ // part in | this | is overwritten.Special logic so that a middle initial may
+ // not overwrite a full middle name.
+ void OverwriteName(const NameInfo& new_name);
+
+ // Returns true if all the name parts (first, middle and last) are empty.
+ bool NamePartsAreEmpty() const;
+
// FormGroup:
base::string16 GetRawInfo(ServerFieldType type) const override;
void SetRawInfo(ServerFieldType type, const base::string16& value) override;
« no previous file with comments | « components/autofill/core/browser/autofill_profile_unittest.cc ('k') | components/autofill/core/browser/contact_info.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698