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

Unified Diff: chrome/browser/webdata/web_data_service.h

Issue 12494020: Remove knowledge of Chrome-specific SyncableService classes from WebDataService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 9 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/web_data_service.h
diff --git a/chrome/browser/webdata/web_data_service.h b/chrome/browser/webdata/web_data_service.h
index 638278e8fe1f69d7239337d53857ff08cbaa8a25..4ba7f370c684078a0fb9710eeb0aaf54c354eb35 100644
--- a/chrome/browser/webdata/web_data_service.h
+++ b/chrome/browser/webdata/web_data_service.h
@@ -111,7 +111,6 @@ class WebDataService
// WebDataServiceBase implementation.
virtual void ShutdownOnUIThread() OVERRIDE;
- virtual void Init(const base::FilePath& path) OVERRIDE;
// Notifies listeners on the UI thread that multiple changes have been made to
// to Autofill records of the database.
@@ -308,17 +307,12 @@ class WebDataService
virtual ~WebDataService();
+ private:
//////////////////////////////////////////////////////////////////////////////
//
// The following methods are only invoked in the web data service thread.
//
//////////////////////////////////////////////////////////////////////////////
- private:
- // Initialize any syncable services.
- void InitializeSyncableServices();
-
- // Deletes the syncable services.
- void ShutdownSyncableServices();
//////////////////////////////////////////////////////////////////////////////
//
@@ -438,14 +432,6 @@ class WebDataService
void DestroyAutofillProfileResult(const WDTypedResult* result);
void DestroyAutofillCreditCardResult(const WDTypedResult* result);
- // Syncable services for the database data. We own the services, but don't
- // use |scoped_ptr|s because the lifetimes must be managed on the database
- // thread.
- // Currently only Autocomplete and Autofill profiles use the new Sync API, but
- // all the database data should migrate to this API over time.
- AutocompleteSyncableService* autocomplete_syncable_service_;
- AutofillProfileSyncableService* autofill_profile_syncable_service_;
-
DISALLOW_COPY_AND_ASSIGN(WebDataService);
};

Powered by Google App Engine
This is Rietveld 408576698