Do not update AttachmentIds after uploading attachments to sync server.

sync/internal_api/attachments/fake_attachment_uploader.cc

Index: sync/internal_api/attachments/fake_attachment_uploader.cc
diff --git a/sync/internal_api/attachments/fake_attachment_uploader.cc b/sync/internal_api/attachments/fake_attachment_uploader.cc
index 743b534e839a3695137f308fb7433e8d23108a6f..1ffbe5b0539f690c75f11ac15a548edfac94323e 100644
--- a/sync/internal_api/attachments/fake_attachment_uploader.cc
+++ b/sync/internal_api/attachments/fake_attachment_uploader.cc
@@ -25,11 +25,9 @@ void FakeAttachmentUploader::UploadAttachment(const Attachment& attachment,
   DCHECK(!attachment.GetId().GetProto().unique_id().empty());
 
   UploadResult result = UPLOAD_SUCCESS;
-  AttachmentId updated_id = attachment.GetId();
-  // TODO(maniscalco): Update the attachment id with server address information
-  // before passing it to the callback.
-  base::MessageLoop::current()->PostTask(
-      FROM_HERE, base::Bind(callback, result, updated_id));
+  AttachmentId id = attachment.GetId();
+  base::MessageLoop::current()->PostTask(FROM_HERE,
+                                         base::Bind(callback, result, id));
 }
 
 }  // namespace syncer