Invoke AttachmentUploader and update AttachmentIds.

sync/syncable/mutable_entry.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_SYNCABLE_MUTABLE_ENTRY_H_
 #define SYNC_SYNCABLE_MUTABLE_ENTRY_H_
 
 #include "sync/base/sync_export.h"
 #include "sync/internal_api/public/base/model_type.h"
 #include "sync/syncable/entry.h"
@@ skipping 43 matching lines @@
   void PutUniquePosition(const UniquePosition& value);
 
   // Sets the position of this item, and updates the entry kernels of the
   // adjacent siblings so that list invariants are maintained.  Returns false
   // and fails if |predecessor_id| does not identify a sibling.  Pass the root
   // ID to put the node in first position.
   bool PutPredecessor(const Id& predecessor_id);
 
   void PutAttachmentMetadata(
       const sync_pb::AttachmentMetadata& attachment_metadata);
+  // Update attachment metadata, replace all records matching attachment id's

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

nit: there should be a blank line above this comment.

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

Done.

+  // unique id with updated attachment id.
+  void UpdateAttachmentId(const sync_pb::AttachmentIdProto& attachment_id);

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

Consider renaming attachment_id to updated_attachment_id so it's even more clear
that this method isn't updating attachment_id.

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

Done.

 
  private:
   // Kind of redundant. We should reduce the number of pointers
   // floating around if at all possible. Could we store this in Directory?
   // Scope: Set on construction, never changed after that.
   WriteTransaction* const write_transaction_;
 
   DISALLOW_COPY_AND_ASSIGN(MutableEntry);
 };
 
 // This function sets only the flags needed to get this entry to sync.
 bool MarkForSyncing(syncable::MutableEntry* e);
 
 }  // namespace syncable
 }  // namespace syncer
 
 #endif  // SYNC_SYNCABLE_MUTABLE_ENTRY_H_