Split Autofill webdata

chrome/browser/api/webdata/autofill_web_data_service.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_API_WEBDATA_AUTOFILL_WEB_DATA_SERVICE_H_
 #define CHROME_BROWSER_API_WEBDATA_AUTOFILL_WEB_DATA_SERVICE_H_
 
 #include "chrome/browser/api/webdata/autofill_web_data.h"
 
+class WebDatabaseService;
+
 namespace content {
 class BrowserContext;
 }
 
 // API for Autofill web data.
 class AutofillWebDataService

[Jói, 2013/03/19 20:53:58]

I think there is no need for this interface any more, we can collapse it into
the subclass and rename the subclass from AutofillWebDataServiceImpl to
AutofillWebDataService.

[Cait (Slow), 2013/03/20 22:21:10]

I'll wait for a later CL to do this (as well as pull all autofill parts out of
WebDataService), if it's ok with you.

On 2013/03/19 20:53:58, Jói wrote:

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

Yep, that's fine.

     : public AutofillWebData,
       public WebDataServiceBase {
  public:
+  // TODO(joi): This should take a ProfileErrorCallback once this

[Jói, 2013/03/19 20:53:58]

this TODO seems to be addressed 2 lines below?

[Cait (Slow), 2013/03/20 22:21:10]

Done.

+  // class doesn't simply delegate to WebDataService.
+  AutofillWebDataService(scoped_refptr<WebDatabaseService> wdbs,
+      const ProfileErrorCallback& callback);
+
   // Retrieve an AutofillWebDataService for the given context.
   //
   // Can return NULL in some contexts.
   static scoped_refptr<AutofillWebDataService> FromBrowserContext(
       content::BrowserContext* context);
 
  protected:
   virtual ~AutofillWebDataService() {}
 };
 
 #endif  // CHROME_BROWSER_API_WEBDATA_AUTOFILL_WEB_DATA_SERVICE_H_