[sync] Remove a few TODOs

After careful consideration, there is no obvious solution to revert back
to SyncApiComponentFactory without a big amount of refactoring that
doesn't seem justified because of this small TODO.

Change-Id: I3e1d6fbe473c07b492fbdb2ce4e15ec3706aa53e
Fixed: 915154
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2480084
Auto-Submit: Mikel Astiz <mastiz@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Commit-Queue: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#818388}

chrome/browser/sync/chrome_sync_client.h

@@ -89,8 +89,6 @@ class ChromeSyncClient : public browser_sync::BrowserSyncClient {
   Profile* const profile_;
 
   // The sync api component factory in use by this client.
-  // TODO(crbug.com/915154): Revert to SyncApiComponentFactory once common
-  // controller creation is moved elsewhere.
   std::unique_ptr<browser_sync::ProfileSyncComponentsFactoryImpl>
       component_factory_;
 

ios/chrome/browser/sync/ios_chrome_sync_client.h

@@ -62,8 +62,6 @@ class IOSChromeSyncClient : public browser_sync::BrowserSyncClient {
   ChromeBrowserState* const browser_state_;
 
   // The sync api component factory in use by this client.
-  // TODO(crbug.com/915154): Revert to SyncApiComponentFactory once common
-  // controller creation is moved elsewhere.
   std::unique_ptr<browser_sync::ProfileSyncComponentsFactoryImpl>
       component_factory_;
 

ios/web_view/internal/sync/web_view_sync_client.h

@@ -74,8 +74,6 @@ class WebViewSyncClient : public browser_sync::BrowserSyncClient {
   invalidation::InvalidationService* invalidation_service_;
   syncer::SyncInvalidationsService* sync_invalidations_service_;
 
-  // TODO(crbug.com/915154): Revert to SyncApiComponentFactory once common
-  // controller creation is moved elsewhere.
   std::unique_ptr<browser_sync::ProfileSyncComponentsFactoryImpl>
       component_factory_;