Invoke AttachmentUploader and update AttachmentIds.

sync/internal_api/public/write_transaction.h

Index: sync/internal_api/public/write_transaction.h
diff --git a/sync/internal_api/public/write_transaction.h b/sync/internal_api/public/write_transaction.h
index b0865ecc1de898c078c8fa6e5d4b79245867ab73..beeb6a5d2424c4f3d86ca6cb94ccac0219e88fcf 100644
--- a/sync/internal_api/public/write_transaction.h
+++ b/sync/internal_api/public/write_transaction.h
@@ -52,6 +52,10 @@ class SYNC_EXPORT WriteTransaction : public BaseTransaction {
       syncer::SyncChangeProcessor::ContextRefreshStatus refresh_status,
       const std::string& context);
 
+  // Attachment id was just updated. Propagate this change to all entries that
+  // refer this attachment id.
+  void UpdateEntriesWithAttachmentId(const AttachmentId& attachment_id);

[maniscalco, 2014/05/09 18:12:11]

How about a unit test for this method?  directory_unittest.cc might be a good
place for it.

[pavely, 2014/05/13 23:06:44]

directory_unittest tests the other WriteTransaction. This one is not accessible
from there.
Method is straightforward so I decided not to test it.
On 2014/05/09 18:12:11, maniscalco wrote:

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

You are right, directory_unittest is for the other WriteTransaction.  I'm OK
with omitting a test for this method.

On 2014/05/13 23:06:44, pavely wrote:

+
  protected:
   WriteTransaction() {}