[Sync] Replace ProfileSyncComponentsFactory with SyncClient

components/sync_driver/non_ui_data_type_controller.h

Index: components/sync_driver/non_ui_data_type_controller.h
diff --git a/components/sync_driver/non_ui_data_type_controller.h b/components/sync_driver/non_ui_data_type_controller.h
index 2ea7dc8ec3f5bf606614be4a3f08193bff851280..09bee6a6fe830bce20ef6d2dbb2409750c276b8b 100644
--- a/components/sync_driver/non_ui_data_type_controller.h
+++ b/components/sync_driver/non_ui_data_type_controller.h
@@ -20,14 +20,14 @@ class SyncableService;
 
 namespace sync_driver {
 
-class SyncApiComponentFactory;
+class SyncClient;
 
 class NonUIDataTypeController : public DataTypeController {
  public:
   NonUIDataTypeController(
       scoped_refptr<base::SingleThreadTaskRunner> ui_thread,
       const base::Closure& error_callback,
-      SyncApiComponentFactory* sync_factory);
+      SyncClient* sync_client);
 
   // DataTypeController interface.
   void LoadModels(const ModelLoadCallback& model_load_callback) override;
@@ -124,7 +124,7 @@ class NonUIDataTypeController : public DataTypeController {
   // Note: this is performed on the UI thread.
   void DisableImpl(const syncer::SyncError& error);
 
-  SyncApiComponentFactory* const sync_factory_;
+  SyncClient* const sync_client_;

[stanisc, 2015/08/27 22:14:04]

Since DTC classes now have SyncClient it should be unnecessary in the base DTC
to have user_share() and OnUserShareReady(...).
Instead user share could be accessed via sync_client_ ->
GetSyncService()->GetUserShare(). That would also simplify changing the base DTC
to be used with USS data types. We could do that in a separate change.

[Nicolas Zea, 2015/08/27 23:37:16]

Acknowledged.

 
   // State of this datatype controller.
   State state_;