[Sync] Componentize ProfileSyncComponentsFactoryImpl

chrome/browser/sync/chrome_sync_client.h

 // Copyright 2015 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_SYNC_CHROME_SYNC_CLIENT_H__
 #define CHROME_BROWSER_SYNC_CHROME_SYNC_CLIENT_H__
 
 #include "base/macros.h"
 #include "chrome/browser/browsing_data/browsing_data_remover.h"
 #include "chrome/browser/sync/glue/extensions_activity_monitor.h"
@@ skipping 36 matching lines @@
       syncer::ModelType type) override;
   scoped_refptr<syncer::ModelSafeWorker> CreateModelWorkerForGroup(
       syncer::ModelSafeGroup group,
       syncer::WorkerLoopDestructionObserver* observer) override;
   sync_driver::SyncApiComponentFactory* GetSyncApiComponentFactory() override;
 
   // Helper for testing rollback.
   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_;
 
   void ClearBrowsingData(base::Time start, base::Time end);
 
   // The sync api component factory in use by this client.
   scoped_ptr<sync_driver::SyncApiComponentFactory> component_factory_;
 
   // Members that must be fetched on the UI thread but accessed on their
   // respective backend threads.
@@ skipping 13 matching lines @@
 
   // Used in integration tests.
   BrowsingDataRemover::Observer* browsing_data_remover_observer_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeSyncClient);
 };
 
 }  // namespace browser_sync
 
 #endif  // CHROME_BROWSER_SYNC_CHROME_SYNC_CLIENT_H__