diff options
author | thomasvl@chromium.org <thomasvl@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-17 12:30:34 +0000 |
---|---|---|
committer | thomasvl@chromium.org <thomasvl@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-17 12:30:34 +0000 |
commit | 4f4863750ceccaa845e03eaf9768451c0db7e805 (patch) | |
tree | a4ca9e2271a4479c8b260ad2c6c065a522acbc07 /chrome/browser/autofill | |
parent | 60a0af936fe4d0254a721de500e50b5ffa798b4d (diff) | |
download | chromium_src-4f4863750ceccaa845e03eaf9768451c0db7e805.zip chromium_src-4f4863750ceccaa845e03eaf9768451c0db7e805.tar.gz chromium_src-4f4863750ceccaa845e03eaf9768451c0db7e805.tar.bz2 |
[Mac] more -Wextra support:
- autofill: explicitly initialize in the copy constructor
- make some loops clearly use {} for empty bodies
- bump up app mode version number to 1.0 so we don't check for < 0 on the unsigned version number.
BUG=34160
TEST=none
Review URL: http://codereview.chromium.org/2856006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50095 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/autofill')
-rw-r--r-- | chrome/browser/autofill/fax_field.cc | 5 | ||||
-rw-r--r-- | chrome/browser/autofill/phone_field.cc | 3 |
2 files changed, 6 insertions, 2 deletions
diff --git a/chrome/browser/autofill/fax_field.cc b/chrome/browser/autofill/fax_field.cc index e7b475e..de86e30 100644 --- a/chrome/browser/autofill/fax_field.cc +++ b/chrome/browser/autofill/fax_field.cc @@ -24,4 +24,7 @@ bool FaxField::GetFieldInfo(FieldTypeMap* field_type_map) const { FaxField::FaxField() : number_(NULL) {} -FaxField::FaxField(const FaxField& fax_field) : number_(fax_field.number_) {} +FaxField::FaxField(const FaxField& fax_field) + : FormField(), + number_(fax_field.number_) { +} diff --git a/chrome/browser/autofill/phone_field.cc b/chrome/browser/autofill/phone_field.cc index 5b4494d..f67d6e2 100644 --- a/chrome/browser/autofill/phone_field.cc +++ b/chrome/browser/autofill/phone_field.cc @@ -132,7 +132,8 @@ PhoneField::PhoneField() } PhoneField::PhoneField(const PhoneField& phone_field) - : phone_(phone_field.phone_), + : FormField(), + phone_(phone_field.phone_), area_code_(phone_field.area_code_), prefix_(phone_field.prefix_), extension_(phone_field.extension_) { |