Split Autofill webdata

chrome/browser/webdata/web_database_service.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Chromium settings and storage represent user-selected preferences and
 // information and MUST not be extracted, overwritten or modified except
 // through Chromium defined APIs.
 
 #ifndef CHROME_BROWSER_WEBDATA_WEB_DATABASE_SERVICE_H_
 #define CHROME_BROWSER_WEBDATA_WEB_DATABASE_SERVICE_H_
@@ skipping 23 matching lines @@
 
 
 ////////////////////////////////////////////////////////////////////////////////
 //
 // WebDatabaseService defines the interface to a generic data repository
 // responsible for controlling access to the web database (metadata associated
 // with web pages).
 //
 ////////////////////////////////////////////////////////////////////////////////
 
-class WebDatabaseService {
+class WebDatabaseService : public base::RefCountedThreadSafe<WebDatabaseService,
+    content::BrowserThread::DeleteOnUIThread> {

[Jói, 2013/03/20 23:09:10]

nit: Indentation of the template parameters; would be nice if they could both
indent to the <, e.g. if you start the inheritance on the next line

[Cait (Slow), 2013/03/21 23:23:27]

Done.

  public:
   typedef base::Callback<scoped_ptr<WDTypedResult>(WebDatabase*)> ReadTask;
   typedef base::Callback<WebDatabase::State(WebDatabase*)> WriteTask;
   typedef base::Callback<void(sql::InitStatus)> InitCallback;
 
   // 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);
 
   // Initializes the web database service. Takes a callback which will return
   // the status of the DB after the init.
   virtual void LoadDatabase(const InitCallback& callback);
 
   // Unloads the database without actually shutting down the service.  This can
@@ skipping 17 matching lines @@
       const tracked_objects::Location& from_here,
       const ReadTask& task,
       WebDataServiceConsumer* consumer);
 
   // Cancel an existing request for a task on the DB thread.
   // TODO(caitkp): Think about moving the definition of the Handle type to
   // somewhere else.
   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
   // PostTask on DB thread may outlive us.
   scoped_refptr<WebDataServiceBackend> wds_backend_;
 };
 
 #endif  // CHROME_BROWSER_WEBDATA_WEB_DATABASE_SERVICE_H_