[Sync] Componentize synced_tab_delegate

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 6839c78d4d9ee6d8f43dd08ecd90a2e3d40d3ffe..4e6de467e8dba017259b0dd327c421cf49b8d9c7 100644
--- a/chrome/browser/sync/chrome_sync_client.h
+++ b/chrome/browser/sync/chrome_sync_client.h
@@ -6,6 +6,7 @@
 #define CHROME_BROWSER_SYNC_CHROME_SYNC_CLIENT_H__
 
 #include "components/sync_driver/sync_client.h"
+#include "components/sync_sessions/sync_sessions_client.h"
 
 class Profile;
 
@@ -16,6 +17,8 @@ class SyncService;
 
 namespace browser_sync {
 
+class SyncSessionsClientImpl;
+
 class ChromeSyncClient : public sync_driver::SyncClient {
  public:
   ChromeSyncClient(
@@ -36,6 +39,7 @@ class ChromeSyncClient : public sync_driver::SyncClient {
   autofill::PersonalDataManager* GetPersonalDataManager() override;
   scoped_refptr<autofill::AutofillWebDataService> GetWebDataService() override;
   BookmarkUndoService* GetBookmarkUndoServiceIfExists() override;
+  sync_sessions::SyncSessionsClient* GetSyncSessionsClient() override;
   base::WeakPtr<syncer::SyncableService> GetSyncableServiceForType(
       syncer::ModelType type) override;
 
@@ -52,6 +56,8 @@ class ChromeSyncClient : public sync_driver::SyncClient {
   scoped_refptr<autofill::AutofillWebDataService> web_data_service_;
   scoped_refptr<password_manager::PasswordStore> password_store_;
 
+  scoped_ptr<SyncSessionsClientImpl> sync_sessions_client_;

[pavely, 2015/10/16 21:55:20]

You can hold pointer to SyncSessionClient, don't have to hold
SyncSessionClientImpl. In this case you won't need to forward declare Impl.

[Nicolas Zea, 2015/10/20 23:14:41]

Good call, done.

+
   // TODO(zea): this is a member only because Typed URLs needs access to
   // the UserShare and Cryptographer outside of the UI thread. Remove this
   // once that's no longer the case.