Remove CloseFile from FileSystem.

chrome/browser/chromeos/drive/file_system/open_file_operation.h

 // Copyright 2013 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_DRIVE_FILE_SYSTEM_OPEN_FILE_OPERATION_H_
 #define CHROME_BROWSER_CHROMEOS_DRIVE_FILE_SYSTEM_OPEN_FILE_OPERATION_H_
 
 #include <map>
 
 #include "base/basictypes.h"
@@ skipping 24 matching lines @@
 class DownloadOperation;
 class OperationObserver;
 
 class OpenFileOperation {
  public:
   OpenFileOperation(base::SequencedTaskRunner* blocking_task_runner,
                     OperationObserver* observer,
                     JobScheduler* scheduler,
                     internal::ResourceMetadata* metadata,
                     internal::FileCache* cache,
-                    const base::FilePath& temporary_file_directory,
-                    std::map<base::FilePath, int>* open_files);
+                    const base::FilePath& temporary_file_directory);
   ~OpenFileOperation();
 
   // Opens the file at |file_path|.
   // If the file is not actually downloaded, this method starts
   // to download it to the cache, and then runs |callback| upon the
   // completation with the path to the local cache file.
   // See also the definition of OpenMode for its meaning.
   // |callback| must not be null.
   void OpenFile(const base::FilePath& file_path,
                 OpenMode open_mode,
                 const OpenFileCallback& callback);
 
  private:
   // Part of OpenFile(). Called after file creation is completed.
   void OpenFileAfterCreateFile(const base::FilePath& file_path,
                                const OpenFileCallback& callback,
                                FileError error);
 
   // Part of OpenFile(). Called after file downloading is completed.
-  void OpenFileAfterFileDownloaded(const base::FilePath& file_path,
-                                   const OpenFileCallback& callback,
+  void OpenFileAfterFileDownloaded(const OpenFileCallback& callback,
                                    FileError error,
                                    const base::FilePath& local_file_path,
                                    scoped_ptr<ResourceEntry> entry);
 
   // Part of OpenFile(). Called after the updating of the local state.
-  void OpenFileAfterUpdateLocalState(const base::FilePath& file_path,
+  void OpenFileAfterUpdateLocalState(const std::string& resource_id,
                                      const OpenFileCallback& callback,
                                      const base::FilePath* local_file_path,
                                      FileError error);
 
+  // Closes the file with |resource_id|.
+  void CloseFile(const std::string& resource_id);
+
   scoped_refptr<base::SequencedTaskRunner> blocking_task_runner_;
+  OperationObserver* observer_;
   internal::FileCache* cache_;
 
   scoped_ptr<CreateFileOperation> create_file_operation_;
   scoped_ptr<DownloadOperation> download_operation_;
 
-  // The map from paths for opened file to the number how many the file is
-  // opened. The instance is owned by FileSystem and shared with
-  // CloseFileOperation.
-  std::map<base::FilePath, int>* open_files_;
+  // The map from resource id for an opened file to the number how many times
+  // the file is opened.
+  std::map<std::string, int> open_files_;
 
   // Note: This should remain the last member so it'll be destroyed and
   // invalidate its weak pointers before any other members are destroyed.
   base::WeakPtrFactory<OpenFileOperation> weak_ptr_factory_;
   DISALLOW_COPY_AND_ASSIGN(OpenFileOperation);
 };
 
 }  // namespace file_system
 }  // namespace drive
 
 #endif  // CHROME_BROWSER_CHROMEOS_DRIVE_FILE_SYSTEM_OPEN_FILE_OPERATION_H_