GDataFileSystem::CloseFile needs not modify GDataEntry.

chrome/browser/chromeos/gdata/gdata_file_system.h

 // Copyright (c) 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 CHROME_BROWSER_CHROMEOS_GDATA_GDATA_FILE_SYSTEM_H_
 #define CHROME_BROWSER_CHROMEOS_GDATA_GDATA_FILE_SYSTEM_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 249 matching lines @@
   // Invoked at the last step of OpenFile. It removes |file_path| from the
   // current set of opened files if |result| is an error, and then invokes the
   // |callback| function.
   void OnOpenFileFinished(const FilePath& file_path,
                           const OpenFileCallback& callback,
                           GDataFileError result,
                           const FilePath& cache_file_path);
 
   // Invoked during the process of CloseFile. What is done here is as follows:
   // 1) Gets resource_id and md5 of the entry at |file_path|.
-  // 2) Gets the local path of the cache file from resource_id and md5.
-  // 3) Gets PlatformFileInfo of the modified local cache file.
-  // 4) Gets GDataEntry for |file_path|.
-  // 5) Modifies GDataEntry using the new PlatformFileInfo.
-  // 6) Commits the modification to the cache system.
-  // 7) Invokes the user-supplied |callback|.
+  // 2) Commits the modification to the cache system.
+  // 3) Removes the |file_path| from the remembered set of opened files.
+  // 4) Invokes the user-supplied |callback|.
   // |callback| must not be null.
-  void OnGetEntryInfoCompleteForCloseFile(
+  void CloseFileOnUIThreadAfterGetEntryInfo(
       const FilePath& file_path,
       const FileOperationCallback& callback,
       GDataFileError error,
       scoped_ptr<GDataEntryProto> entry_proto);
-  void OnGetCacheFilePathCompleteForCloseFile(
-      const FilePath& file_path,
-      const FileOperationCallback& callback,
-      GDataFileError error,
-      const std::string& resource_id,
-      const std::string& md5,
-      const FilePath& local_cache_path);
-  void OnGetModifiedFileInfoCompleteForCloseFile(
-      const FilePath& file_path,
-      base::PlatformFileInfo* file_info,
-      bool* get_file_info_result,
-      const FileOperationCallback& callback);
-  void OnGetEntryCompleteForCloseFile(
-      const base::PlatformFileInfo& file_info,
-      const FileOperationCallback& callback,
-      GDataFileError error,
-      GDataEntry* entry);
-  void OnCommitDirtyInCacheCompleteForCloseFile(
+  void CloseFileOnUIThreadAfterCommitDirtyInCache(
       const FileOperationCallback& callback,
       GDataFileError error,
       const std::string& resource_id,
       const std::string& md5);
-  void OnCloseFileFinished(const FilePath& file_path,
-                           const FileOperationCallback& callback,
-                           GDataFileError result);
+  void CloseFileOnUIThreadFinalize(const FilePath& file_path,
+                                   const FileOperationCallback& callback,
+                                   GDataFileError result);
 
   // Invoked upon completion of GetFileByPath initiated by Copy. If
   // GetFileByPath reports no error, calls TransferRegularFile to transfer
   // |local_file_path| to |remote_dest_file_path|.
   //
   // Can be called from UI thread. |callback| is run on the calling thread.
   void OnGetFileCompleteForCopy(const FilePath& remote_dest_file_path,
                                 const FileOperationCallback& callback,
                                 GDataFileError error,
                                 const FilePath& local_file_path,
@@ skipping 549 matching lines @@
   // invalidate the weak pointers before any other members are destroyed.
   base::WeakPtrFactory<GDataFileSystem> ui_weak_ptr_factory_;
   // Unlike other classes, we need this as we need this to redirect a task
   // from IO thread to UI thread.
   base::WeakPtr<GDataFileSystem> ui_weak_ptr_;
 };
 
 }  // namespace gdata
 
 #endif  // CHROME_BROWSER_CHROMEOS_GDATA_GDATA_FILE_SYSTEM_H_