Do not update AttachmentIds after uploading attachments to sync server.

sync/internal_api/attachments/attachment_uploader_impl.cc

 // Copyright 2014 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.
 
 #include "sync/internal_api/public/attachments/attachment_uploader_impl.h"
 
 #include "base/bind.h"
 #include "base/message_loop/message_loop.h"
 #include "base/threading/non_thread_safe.h"
 #include "google_apis/gaia/gaia_constants.h"
@@ skipping 119 matching lines @@
   return attachment_;
 }
 
 void AttachmentUploaderImpl::UploadState::OnURLFetchComplete(
     const net::URLFetcher* source) {
   DCHECK(CalledOnValidThread());
   UploadResult result = UPLOAD_UNSPECIFIED_ERROR;
   AttachmentId attachment_id = attachment_.GetId();
   if (source->GetResponseCode() == net::HTTP_OK) {
     result = UPLOAD_SUCCESS;
-    // TODO(maniscalco): Update the attachment id with server address
-    // information before passing it to the callback (bug 371522).
   } else if (source->GetResponseCode() == net::HTTP_UNAUTHORIZED) {
     // TODO(maniscalco): One possibility is that we received a 401 because our
     // access token has expired.  We should probably fetch a new access token
     // and retry this upload before giving up and reporting failure to our
     // caller (bug 380437).
     OAuth2TokenServiceRequest::InvalidateToken(
         token_service_provider_, account_id_, scopes_, access_token_);
   } else {
     // TODO(maniscalco): Once the protocol is better defined, deal with the
     // various HTTP response codes we may encounter.
@@ skipping 119 matching lines @@
   GURL::Replacements replacements;
   replacements.SetPathStr(path);
   return sync_service_url.ReplaceComponents(replacements);
 }
 
 void AttachmentUploaderImpl::DeleteUploadStateFor(const UniqueId& unique_id) {
   state_map_.erase(unique_id);
 }
 
 }  // namespace syncer