Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(706)

Unified Diff: components/autofill/core/browser/webdata/autofill_webdata_backend_impl.cc

Issue 2703673002: [Merge-57] Add billing_address_id and has_converted to autofill_table (Closed)
Patch Set: Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/autofill/core/browser/webdata/autofill_webdata_backend_impl.cc
diff --git a/components/autofill/core/browser/webdata/autofill_webdata_backend_impl.cc b/components/autofill/core/browser/webdata/autofill_webdata_backend_impl.cc
index bcb9cf542f4dcbc02baa127eaa0539ed1f38417a..2129da0abddab02f59536141351ea0ee3fd779c4 100644
--- a/components/autofill/core/browser/webdata/autofill_webdata_backend_impl.cc
+++ b/components/autofill/core/browser/webdata/autofill_webdata_backend_impl.cc
@@ -370,11 +370,11 @@ WebDatabase::State
return WebDatabase::COMMIT_NOT_NEEDED;
}
-WebDatabase::State AutofillWebDataBackendImpl::UpdateServerCardUsageStats(
+WebDatabase::State AutofillWebDataBackendImpl::UpdateServerCardMetadata(
const CreditCard& card,
WebDatabase* db) {
DCHECK(db_thread_->BelongsToCurrentThread());
- if (!AutofillTable::FromWebDatabase(db)->UpdateServerCardUsageStats(card))
+ if (!AutofillTable::FromWebDatabase(db)->UpdateServerCardMetadata(card))
return WebDatabase::COMMIT_NOT_NEEDED;
for (auto& db_observer : db_observer_list_) {
@@ -385,11 +385,11 @@ WebDatabase::State AutofillWebDataBackendImpl::UpdateServerCardUsageStats(
return WebDatabase::COMMIT_NEEDED;
}
-WebDatabase::State AutofillWebDataBackendImpl::UpdateServerAddressUsageStats(
+WebDatabase::State AutofillWebDataBackendImpl::UpdateServerAddressMetadata(
const AutofillProfile& profile,
WebDatabase* db) {
DCHECK(db_thread_->BelongsToCurrentThread());
- if (!AutofillTable::FromWebDatabase(db)->UpdateServerAddressUsageStats(
+ if (!AutofillTable::FromWebDatabase(db)->UpdateServerAddressMetadata(
profile)) {
return WebDatabase::COMMIT_NOT_NEEDED;
}
@@ -402,23 +402,6 @@ WebDatabase::State AutofillWebDataBackendImpl::UpdateServerAddressUsageStats(
return WebDatabase::COMMIT_NEEDED;
}
-WebDatabase::State AutofillWebDataBackendImpl::UpdateServerCardBillingAddress(
- const CreditCard& card,
- WebDatabase* db) {
- DCHECK(db_thread_->BelongsToCurrentThread());
- if (!AutofillTable::FromWebDatabase(db)->UpdateServerCardBillingAddress(
- card)) {
- return WebDatabase::COMMIT_NOT_NEEDED;
- }
-
- for (auto& db_observer : db_observer_list_) {
- db_observer.CreditCardChanged(
- CreditCardChange(CreditCardChange::UPDATE, card.guid(), &card));
- }
-
- return WebDatabase::COMMIT_NEEDED;
-}
-
WebDatabase::State AutofillWebDataBackendImpl::ClearAllServerData(
WebDatabase* db) {
DCHECK(db_thread_->BelongsToCurrentThread());

Powered by Google App Engine
This is Rietveld 408576698