Do not update AttachmentIds after uploading attachments to sync server.

sync/internal_api/public/write_transaction.h

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef SYNC_INTERNAL_API_PUBLIC_WRITE_TRANSACTION_H_
 #define SYNC_INTERNAL_API_PUBLIC_WRITE_TRANSACTION_H_
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "sync/api/sync_change_processor.h"
@@ skipping 34 matching lines @@
   syncable::WriteTransaction* GetWrappedWriteTrans() { return transaction_; }
 
   // Set's a |type|'s local context. |refresh_status| controls whether
   // a datatype refresh is performed (clearing the progress marker token and
   // setting the version of all synced entities to 1).
   void SetDataTypeContext(
       ModelType type,
       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 and set is_on_server for corresponding records.
-  void UpdateEntriesWithAttachmentId(const AttachmentId& attachment_id);
+  // Update all entries that refer to |attachment_id| indicating that
+  // |attachment_id| has been uploaded to the sync server.
+  void UpdateEntriesMarkAttachmentAsOnServer(const AttachmentId& attachment_id);
 
  protected:
   WriteTransaction() {}
 
   void SetTransaction(syncable::WriteTransaction* trans) {
     transaction_ = trans;
   }
 
  private:
   void* operator new(size_t size);  // Transaction is meant for stack use only.
 
   // The underlying syncable object which this class wraps.
   syncable::WriteTransaction* transaction_;
 
   DISALLOW_COPY_AND_ASSIGN(WriteTransaction);
 };
 
 }  // namespace syncer
 
 #endif  // SYNC_INTERNAL_API_PUBLIC_WRITE_TRANSACTION_H_