Invoke AttachmentUploader and update AttachmentIds.

components/sync_driver/generic_change_processor_factory.h

Index: components/sync_driver/generic_change_processor_factory.h
diff --git a/components/sync_driver/generic_change_processor_factory.h b/components/sync_driver/generic_change_processor_factory.h
index 978d313f9834eee8f59e71942c3b5efe8112c6df..1fe8754830b6859ce5302a060924f4a0cc5d731e 100644
--- a/components/sync_driver/generic_change_processor_factory.h
+++ b/components/sync_driver/generic_change_processor_factory.h
@@ -18,6 +18,7 @@ namespace browser_sync {
 
 class DataTypeErrorHandler;
 class GenericChangeProcessor;
+class SyncApiComponentFactory;
 
 // Because GenericChangeProcessors are created and used only from the model
 // thread, their lifetime is strictly shorter than other components like
@@ -35,7 +36,8 @@ class GenericChangeProcessorFactory {
       browser_sync::DataTypeErrorHandler* error_handler,
       const base::WeakPtr<syncer::SyncableService>& local_service,
       const base::WeakPtr<syncer::SyncMergeResult>& merge_result,
-      scoped_ptr<syncer::AttachmentService> attachment_service);
+      SyncApiComponentFactory* sync_factory);

[maniscalco, 2014/05/15 20:09:28]

Can you document the sync_factory parameter?  I think it's worth calling out
that GCP won't own or manage the sync_factory and that it will only use it in
its ctor (that way, the caller doesn't have to guarantee the sync_factory
outlives the GCP).

[pavely, 2014/05/16 00:19:10]

Done.

+
  private:
   DISALLOW_COPY_AND_ASSIGN(GenericChangeProcessorFactory);
 };