| Index: components/autofill/core/browser/personal_data_manager.cc
|
| diff --git a/components/autofill/core/browser/personal_data_manager.cc b/components/autofill/core/browser/personal_data_manager.cc
|
| index 0e10852ef13b8ca07f890a31b40b2bfddc7b57e2..b7a0a46e0cfea0ac59b7cce98dd670f788ead666 100644
|
| --- a/components/autofill/core/browser/personal_data_manager.cc
|
| +++ b/components/autofill/core/browser/personal_data_manager.cc
|
| @@ -517,16 +517,10 @@ void PersonalDataManager::RecordUseOf(const AutofillDataModel& data_model) {
|
| if (credit_card) {
|
| credit_card->RecordUse();
|
|
|
| - if (credit_card->record_type() == CreditCard::LOCAL_CARD) {
|
| + if (credit_card->record_type() == CreditCard::LOCAL_CARD)
|
| database_->UpdateCreditCard(*credit_card);
|
| - } else if (credit_card->record_type() == CreditCard::FULL_SERVER_CARD) {
|
| - database_->UpdateUnmaskedCardUsageStats(*credit_card);
|
| - } else {
|
| - // It's possible to get a masked server card here if the user decides not
|
| - // to store a card while verifying it. We don't currently track usage
|
| - // of masked cards, so no-op.
|
| - return;
|
| - }
|
| + else
|
| + database_->UpdateServerCardUsageStats(*credit_card);
|
|
|
| Refresh();
|
| return;
|
| @@ -535,7 +529,12 @@ void PersonalDataManager::RecordUseOf(const AutofillDataModel& data_model) {
|
| AutofillProfile* profile = GetProfileByGUID(data_model.guid());
|
| if (profile) {
|
| profile->RecordUse();
|
| - database_->UpdateAutofillProfile(*profile);
|
| +
|
| + if (profile->record_type() == AutofillProfile::LOCAL_PROFILE)
|
| + database_->UpdateAutofillProfile(*profile);
|
| + else if (profile->record_type() == AutofillProfile::SERVER_PROFILE)
|
| + database_->UpdateServerAddressUsageStats(*profile);
|
| +
|
| Refresh();
|
| }
|
| }
|
| @@ -673,7 +672,7 @@ void PersonalDataManager::UpdateServerCreditCard(
|
| DCHECK_NE(existing_credit_card->record_type(), credit_card.record_type());
|
| DCHECK_EQ(existing_credit_card->Label(), credit_card.Label());
|
| if (existing_credit_card->record_type() == CreditCard::MASKED_SERVER_CARD) {
|
| - database_->UnmaskServerCreditCard(credit_card.server_id(),
|
| + database_->UnmaskServerCreditCard(credit_card,
|
| credit_card.number());
|
| } else {
|
| database_->MaskServerCreditCard(credit_card.server_id());
|
|
|