diff options
| author | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-22 17:56:51 +0000 |
|---|---|---|
| committer | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-22 17:56:51 +0000 |
| commit | 644090c5f177f71e739b51cbab579a4c62fd6da5 (patch) | |
| tree | 89d5b92696920c8be0550617e8fa241e199e3225 /chrome | |
| parent | 484e8231da8c65d935cf1f7f6b217fcebcef71f1 (diff) | |
| download | chromium_src-644090c5f177f71e739b51cbab579a4c62fd6da5.zip chromium_src-644090c5f177f71e739b51cbab579a4c62fd6da5.tar.gz chromium_src-644090c5f177f71e739b51cbab579a4c62fd6da5.tar.bz2 | |
AutoFill: Always check the result of FormField::Add().
CID=8477
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/2811064
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53356 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
| -rw-r--r-- | chrome/browser/autofill/credit_card_field.cc | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/chrome/browser/autofill/credit_card_field.cc b/chrome/browser/autofill/credit_card_field.cc index 770cff4..235bdf03 100644 --- a/chrome/browser/autofill/credit_card_field.cc +++ b/chrome/browser/autofill/credit_card_field.cc @@ -18,11 +18,12 @@ bool CreditCardField::GetFieldInfo(FieldTypeMap* field_type_map) const { // initial. if (cardholder_last_ == NULL) { // Add() will check if cardholder_ is != NULL. - Add(field_type_map, cardholder_, AutoFillType(CREDIT_CARD_NAME)); + ok = ok && Add(field_type_map, cardholder_, AutoFillType(CREDIT_CARD_NAME)); + DCHECK(ok); } - Add(field_type_map, type_, AutoFillType(CREDIT_CARD_TYPE)); - + ok = ok && Add(field_type_map, type_, AutoFillType(CREDIT_CARD_TYPE)); + DCHECK(ok); ok = ok && Add(field_type_map, expiration_month_, AutoFillType(CREDIT_CARD_EXP_MONTH)); DCHECK(ok); |