diff options
| author | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-03 01:57:07 +0000 |
|---|---|---|
| committer | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-03 01:57:07 +0000 |
| commit | 877535175434bbf768234d10cd6df8085e707381 (patch) | |
| tree | 0c08a136830a80ec63dbe6a3368cb88de4224781 /chrome/browser/autofill/contact_info.cc | |
| parent | 1314544e05b07de2bfba8c91eff32fa127dca815 (diff) | |
| download | chromium_src-877535175434bbf768234d10cd6df8085e707381.zip chromium_src-877535175434bbf768234d10cd6df8085e707381.tar.gz chromium_src-877535175434bbf768234d10cd6df8085e707381.tar.bz2 | |
Where possible, use return-by-const-ref for accessors in AutoFill data structures. In addition, I fixed the access levels of many of the modified methods.
BUG=31775
TEST=none
Review URL: http://codereview.chromium.org/661421
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@40477 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/autofill/contact_info.cc')
| -rw-r--r-- | chrome/browser/autofill/contact_info.cc | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/chrome/browser/autofill/contact_info.cc b/chrome/browser/autofill/contact_info.cc index 37c5714..f77652a 100644 --- a/chrome/browser/autofill/contact_info.cc +++ b/chrome/browser/autofill/contact_info.cc @@ -120,6 +120,19 @@ void ContactInfo::SetInfo(const AutoFillType& type, const string16& value) { NOTREACHED(); } +ContactInfo::ContactInfo(const ContactInfo& contact_info) + : FormGroup(), + first_tokens_(contact_info.first_tokens_), + middle_tokens_(contact_info.middle_tokens_), + last_tokens_(contact_info.last_tokens_), + first_(contact_info.first_), + middle_(contact_info.middle_), + last_(contact_info.last_), + suffix_(contact_info.suffix_), + email_(contact_info.email_), + company_name_(contact_info.company_name_) { +} + string16 ContactInfo::FullName() const { if (first_.empty()) return string16(); @@ -149,19 +162,6 @@ string16 ContactInfo::MiddleInitial() const { return initial; } -ContactInfo::ContactInfo(const ContactInfo& contact_info) - : FormGroup(), - first_tokens_(contact_info.first_tokens_), - middle_tokens_(contact_info.middle_tokens_), - last_tokens_(contact_info.last_tokens_), - first_(contact_info.first_), - middle_(contact_info.middle_), - last_(contact_info.last_), - suffix_(contact_info.suffix_), - email_(contact_info.email_), - company_name_(contact_info.company_name_) { -} - bool ContactInfo::FindInfoMatchesHelper(const AutoFillFieldType& field_type, const string16& info, string16* match) const { |