Improve the merging of two profiles' names. |
||
Issue descriptionTwo profiles, one with a middle name and the other with the corresponding middle name initial should be merged. At the same time it would be good to improve the way we overwrite names when merging profiles as to avoid possible loss of information. For example saving a profile with NAME_FIRST = "Marion" NAME_MIDDLE = "Mitchell" NAME_LAST = "Morrison" NAME_FULL = "" merged with NAME_FIRST = "" NAME_MIDDLE = "" NAME_LAST = "" NAME_FULL = "Marion Mitchell Morrison" will result in a profile with NAME_FIRST = "" NAME_MIDDLE = "" NAME_LAST = "" NAME_FULL = "Marion Mitchell Morrison"
,
May 16 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d989eb5dfc869b9e97c797e15370d6feb30fd230 commit d989eb5dfc869b9e97c797e15370d6feb30fd230 Author: sebsg <sebsg@chromium.org> Date: Mon May 16 22:53:23 2016 Make sure a profile with a complete middle name gets merged with a profile with a corresponding middle name initial. Make sure no loss of information is possible when merging names. BUG= 611282 TEST=NameInfoTest, PersonalDataManagerTest Review-Url: https://codereview.chromium.org/1973873002 Cr-Commit-Position: refs/heads/master@{#393963} [modify] https://crrev.com/d989eb5dfc869b9e97c797e15370d6feb30fd230/components/autofill/core/browser/autofill_profile.cc [modify] https://crrev.com/d989eb5dfc869b9e97c797e15370d6feb30fd230/components/autofill/core/browser/autofill_profile_unittest.cc [modify] https://crrev.com/d989eb5dfc869b9e97c797e15370d6feb30fd230/components/autofill/core/browser/contact_info.cc [modify] https://crrev.com/d989eb5dfc869b9e97c797e15370d6feb30fd230/components/autofill/core/browser/contact_info.h [modify] https://crrev.com/d989eb5dfc869b9e97c797e15370d6feb30fd230/components/autofill/core/browser/contact_info_unittest.cc [modify] https://crrev.com/d989eb5dfc869b9e97c797e15370d6feb30fd230/components/autofill/core/browser/personal_data_manager_unittest.cc
,
May 16 2016
|
||
►
Sign in to add a comment |
||
Comment 1 by se...@chromium.org
, May 12 2016