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

Unified Diff: chrome/browser/webdata/autofill_profile_syncable_service.cc

Issue 14081043: Hook up Autofill Backend interface to SyncableServices (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Pure merge Created 7 years, 7 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: chrome/browser/webdata/autofill_profile_syncable_service.cc
diff --git a/chrome/browser/webdata/autofill_profile_syncable_service.cc b/chrome/browser/webdata/autofill_profile_syncable_service.cc
index 787d56a65b3506574c51202e69ab6ef483a9c57c..3610492c9607f2a475eba2e1e5ffc721c5e3418f 100644
--- a/chrome/browser/webdata/autofill_profile_syncable_service.cc
+++ b/chrome/browser/webdata/autofill_profile_syncable_service.cc
@@ -48,15 +48,15 @@ void* UserDataKey() {
const char kAutofillProfileTag[] = "google_chrome_autofill_profiles";
AutofillProfileSyncableService::AutofillProfileSyncableService(
- AutofillWebDataService* web_data_service,
+ autofill::AutofillWebDataBackend* webdata_backend,
const std::string& app_locale)
- : web_data_service_(web_data_service),
+ : webdata_backend_(webdata_backend),
app_locale_(app_locale),
scoped_observer_(this) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB));
- DCHECK(web_data_service_);
+ DCHECK(webdata_backend_);
- scoped_observer_.Add(web_data_service_);
+ scoped_observer_.Add(webdata_backend_);
}
AutofillProfileSyncableService::~AutofillProfileSyncableService() {
@@ -64,12 +64,13 @@ AutofillProfileSyncableService::~AutofillProfileSyncableService() {
}
// static
-void AutofillProfileSyncableService::CreateForWebDataService(
+void AutofillProfileSyncableService::CreateForWebDataServiceAndBackend(
AutofillWebDataService* web_data_service,
+ autofill::AutofillWebDataBackend* webdata_backend,
const std::string& app_locale) {
web_data_service->GetDBUserData()->SetUserData(
UserDataKey(),
- new AutofillProfileSyncableService(web_data_service, app_locale));
+ new AutofillProfileSyncableService(webdata_backend, app_locale));
}
// static
@@ -81,7 +82,7 @@ AutofillProfileSyncableService::FromWebDataService(
}
AutofillProfileSyncableService::AutofillProfileSyncableService()
- : web_data_service_(NULL),
+ : webdata_backend_(NULL),
scoped_observer_(this) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB));
}
@@ -191,7 +192,8 @@ AutofillProfileSyncableService::MergeDataAndStartSyncing(
sync_processor_->ProcessSyncChanges(FROM_HERE, new_changes));
}
- AutofillWebDataService::NotifyOfMultipleAutofillChanges(web_data_service_);
+ if (webdata_backend_)
+ webdata_backend_->NotifyOfMultipleAutofillChanges();
return merge_result;
}
@@ -262,7 +264,8 @@ syncer::SyncError AutofillProfileSyncableService::ProcessSyncChanges(
"Failed to update webdata.");
}
- AutofillWebDataService::NotifyOfMultipleAutofillChanges(web_data_service_);
+ if (webdata_backend_)
+ webdata_backend_->NotifyOfMultipleAutofillChanges();
return syncer::SyncError();
}
@@ -583,7 +586,7 @@ bool AutofillProfileSyncableService::MergeProfile(
}
AutofillTable* AutofillProfileSyncableService::GetAutofillTable() const {
- return AutofillTable::FromWebDatabase(web_data_service_->GetDatabase());
+ return AutofillTable::FromWebDatabase(webdata_backend_->GetDatabase());
}
void AutofillProfileSyncableService::InjectStartSyncFlare(
« no previous file with comments | « chrome/browser/webdata/autofill_profile_syncable_service.h ('k') | chrome/browser/webdata/web_data_service_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698