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

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

Issue 12695015: Split Autofill webdata (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Pure merge to 12853004 (move table creation to factory) 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_database_service.h
diff --git a/chrome/browser/webdata/web_database_service.h b/chrome/browser/webdata/web_database_service.h
index e7587e680af97891465f28a45c02209d051ac27f..9d85ee4058b0ffa0192cfe5670c624ade29c8c03 100644
--- a/chrome/browser/webdata/web_database_service.h
+++ b/chrome/browser/webdata/web_database_service.h
@@ -41,7 +41,8 @@ class WebDataServiceConsumer;
//
////////////////////////////////////////////////////////////////////////////////
-class WebDatabaseService {
+class WebDatabaseService : public base::RefCountedThreadSafe<WebDatabaseService,
+ content::BrowserThread::DeleteOnUIThread> {
public:
typedef base::Callback<scoped_ptr<WDTypedResult>(WebDatabase*)> ReadTask;
typedef base::Callback<WebDatabase::State(WebDatabase*)> WriteTask;
@@ -50,7 +51,10 @@ class WebDatabaseService {
// Takes the path to the WebDatabase file.
explicit WebDatabaseService(const base::FilePath& path);
- virtual ~WebDatabaseService();
+ // Adds |table| as a WebDatabaseTable that will participate in
+ // managing the database, transferring ownership. All calls to this
+ // method must be made before |LoadDatabase| is called.
+ virtual void AddTable(scoped_ptr<WebDatabaseTable> table);
// Adds |table| as a WebDatabaseTable that will participate in
// managing the database, transferring ownership. All calls to this
@@ -89,6 +93,12 @@ class WebDatabaseService {
virtual void CancelRequest(WebDataServiceBase::Handle h);
private:
+ friend struct content::BrowserThread::DeleteOnThread<
+ content::BrowserThread::UI>;
+ friend class base::DeleteHelper<WebDatabaseService>;
+
+ virtual ~WebDatabaseService();
+
base::FilePath path_;
// The primary owner is |WebDatabaseService| but is refcounted because

Powered by Google App Engine
This is Rietveld 408576698