[Sync] Componentize ProfileSyncComponentsFactoryImpl

chrome/browser/sync/chrome_sync_client.h

Index: chrome/browser/sync/chrome_sync_client.h
diff --git a/chrome/browser/sync/chrome_sync_client.h b/chrome/browser/sync/chrome_sync_client.h
index 7d16e8a62293e8032993ba729913ef483a4926b7..eb7a3c375bd400d2b56dba6223e276f3e8d2ccd0 100644
--- a/chrome/browser/sync/chrome_sync_client.h
+++ b/chrome/browser/sync/chrome_sync_client.h
@@ -54,6 +54,22 @@ class ChromeSyncClient : public sync_driver::SyncClient {
   void SetBrowsingDataRemoverObserverForTesting(
       BrowsingDataRemover::Observer* observer);
 
+  // Register data types which are enabled on desktop platforms only.
+  // |disabled_types| and |enabled_types| correspond only to those types
+  // being explicitly enabled/disabled by the command line.

[blundell, 2015/11/10 15:24:24]

should this be "explicitly disabled/enabled"? I realize the comment was this way
before as well but I don't understand it :).

[Nicolas Zea, 2015/11/11 00:06:20]

Done.

+  static void RegisterDesktopDataTypes(Profile* profile,
+                                       syncer::ModelTypeSet disabled_types,
+                                       syncer::ModelTypeSet enabled_types,
+                                       sync_driver::SyncClient* sync_client);
+
+  // Register data types which are enabled on Android platforms only.
+  // |disabled_types| and |enabled_types| correspond only to those types
+  // being explicitly enabled/disabled by the command line.
+  static void RegisterAndroidDataTypes(Profile* profile,
+                                       syncer::ModelTypeSet disabled_types,
+                                       syncer::ModelTypeSet enabled_types,
+                                       sync_driver::SyncClient* sync_client);
+
  private:
   Profile* const profile_;