USS SyncContextProxy / data type activation refactoring

sync/internal_api/public/sync_context.h

Index: sync/internal_api/public/sync_context.h
diff --git a/sync/internal_api/public/sync_context.h b/sync/internal_api/public/sync_context.h
index 2ca15d41f5c294b325f246eb4c6fc737af5e5a53..f3f0fcd9b49f24c2c544bb56df4d1a16bb5ec6ed 100644
--- a/sync/internal_api/public/sync_context.h
+++ b/sync/internal_api/public/sync_context.h
@@ -5,16 +5,11 @@
 #ifndef SYNC_INTERNAL_API_PUBLIC_SYNC_CONTEXT_H_
 #define SYNC_INTERNAL_API_PUBLIC_SYNC_CONTEXT_H_
 
-#include "base/basictypes.h"
-#include "base/memory/weak_ptr.h"
-#include "base/sequenced_task_runner.h"
-#include "sync/base/sync_export.h"
+#include "base/memory/scoped_ptr.h"
+#include "sync/internal_api/public/activation_context.h"
 #include "sync/internal_api/public/base/model_type.h"
-#include "sync/internal_api/public/non_blocking_sync_common.h"
 
 namespace syncer_v2 {
-class ModelTypeProcessor;
-
 // An interface of the core parts of sync.
 //
 // In theory, this is the component that provides off-thread sync types with
@@ -33,10 +28,7 @@ class SYNC_EXPORT_PRIVATE SyncContext {
   // type's thread.
   virtual void ConnectSyncTypeToWorker(
       syncer::ModelType type,
-      const DataTypeState& data_type_state,
-      const UpdateResponseDataList& saved_pending_updates,
-      const scoped_refptr<base::SequencedTaskRunner>& datatype_task_runner,
-      const base::WeakPtr<ModelTypeProcessor>& type_processor) = 0;
+      scoped_ptr<ActivationContext> activation_context) = 0;

[pavely, 2015/09/25 18:22:39]

nit: When using scoped_ptr it is enough to forward declare enclosed class, you
don't need to include header with class declaration.

[stanisc, 2015/09/26 00:16:01]

I wanted to avoid having to include activation_context.h everywhere
sync_context.h and sync_context_proxy.h are included.
But OK, I've changed this.

 
   // Disconnects the syncer from the model and stops syncing the type.
   //
@@ -50,6 +42,6 @@ class SYNC_EXPORT_PRIVATE SyncContext {
   virtual void DisconnectSyncWorker(syncer::ModelType type) = 0;
 };
 
-}  // namespace syncer
+}  // namespace syncer_v2
 
 #endif  // SYNC_INTERNAL_API_PUBLIC_SYNC_CONTEXT_H_