[Sync] Remove the last datatype-specific deps from sync_driver.

components/autofill/core/browser/webdata/autofill_data_type_controller.h

Index: components/autofill/core/browser/webdata/autofill_data_type_controller.h
diff --git a/components/autofill/core/browser/webdata/autofill_data_type_controller.h b/components/autofill/core/browser/webdata/autofill_data_type_controller.h
index 9755d7a6f95aa847816b27572d77f5cb1f797feb..5f90e2f9e08f1c70c56dc2214e0271764c670a23 100644
--- a/components/autofill/core/browser/webdata/autofill_data_type_controller.h
+++ b/components/autofill/core/browser/webdata/autofill_data_type_controller.h
@@ -26,7 +26,8 @@ class AutofillDataTypeController : public sync_driver::NonUIDataTypeController {
       const scoped_refptr<base::SingleThreadTaskRunner>& ui_thread,
       const scoped_refptr<base::SingleThreadTaskRunner>& db_thread,
       const base::Closure& error_callback,
-      sync_driver::SyncClient* sync_client);
+      sync_driver::SyncClient* sync_client,
+      const scoped_refptr<autofill::AutofillWebDataService>& web_data_service);

[vabr (Chromium), 2015/12/08 10:14:28]

nit: Just autofill::AutofillWebDataService*? See my reasoning in
components/password_manager/sync/browser/password_data_type_controller.h.

[maxbogue, 2015/12/08 19:08:05]

Acknowledged.

 
   // NonUIDataTypeController implementation.
   syncer::ModelType type() const override;
@@ -52,11 +53,15 @@ class AutofillDataTypeController : public sync_driver::NonUIDataTypeController {
   // Callback once WebDatabase has loaded.
   void WebDatabaseLoaded();
 
+  // A pointer to the sync client.
   sync_driver::SyncClient* const sync_client_;
 
   // A reference to the DB thread's task runner.
   const scoped_refptr<base::SingleThreadTaskRunner> db_thread_;
 
+  // A reference to the AutofillWebDataService for this controller.
+  scoped_refptr<autofill::AutofillWebDataService> web_data_service_;
+
   DISALLOW_COPY_AND_ASSIGN(AutofillDataTypeController);
 };