| Index: components/autofill/core/browser/webdata/autofill_change.cc
|
| diff --git a/components/autofill/core/browser/webdata/autofill_change.cc b/components/autofill/core/browser/webdata/autofill_change.cc
|
| index 2d2ff6742b0a622c3ddf9702ea7be4414beb6244..c02bdd8464d8476d974f1cfd0a04e2c60f55af10 100644
|
| --- a/components/autofill/core/browser/webdata/autofill_change.cc
|
| +++ b/components/autofill/core/browser/webdata/autofill_change.cc
|
| @@ -17,11 +17,13 @@
|
| AutofillChange::~AutofillChange() {
|
| }
|
|
|
| -AutofillProfileChange::AutofillProfileChange(Type type,
|
| - const std::string& key,
|
| - const AutofillProfile* profile)
|
| - : GenericAutofillChange<std::string>(type, key), profile_(profile) {
|
| - DCHECK(type == REMOVE ? !profile : profile && profile->guid() == key);
|
| +AutofillProfileChange::AutofillProfileChange(
|
| + Type type, const std::string& key, const AutofillProfile* profile)
|
| + : GenericAutofillChange<std::string>(type, key),
|
| + profile_(profile) {
|
| + DCHECK(type == ADD ? (profile && profile->guid() == key) : true);
|
| + DCHECK(type == UPDATE ? (profile && profile->guid() == key) : true);
|
| + DCHECK(type == REMOVE ? !profile : true);
|
| }
|
|
|
| AutofillProfileChange::~AutofillProfileChange() {
|
| @@ -29,23 +31,9 @@
|
|
|
| bool AutofillProfileChange::operator==(
|
| const AutofillProfileChange& change) const {
|
| - return type() == change.type() && key() == change.key() &&
|
| - (type() == REMOVE || *profile() == *change.profile());
|
| -}
|
| -
|
| -CreditCardChange::CreditCardChange(Type type,
|
| - const std::string& key,
|
| - const CreditCard* card)
|
| - : GenericAutofillChange<std::string>(type, key), card_(card) {
|
| - DCHECK(type == REMOVE ? !card : card && card->guid() == key);
|
| -}
|
| -
|
| -CreditCardChange::~CreditCardChange() {
|
| -}
|
| -
|
| -bool CreditCardChange::operator==(const CreditCardChange& change) const {
|
| - return type() == change.type() && key() == change.key() &&
|
| - (type() == REMOVE || *card() == *change.card());
|
| + return type() == change.type() &&
|
| + key() == change.key() &&
|
| + (type() != REMOVE) ? *profile() == *change.profile() : true;
|
| }
|
|
|
| } // namespace autofill
|
|
|