[fsp] Add support for aborting running operations.

chrome/browser/chromeos/file_system_provider/fake_provided_file_system.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_FILE_SYSTEM_PROVIDER_FAKE_PROVIDED_FILE_SYSTEM_H_
 #define CHROME_BROWSER_CHROMEOS_FILE_SYSTEM_PROVIDER_FAKE_PROVIDED_FILE_SYSTEM_H_
 
 #include <map>
 #include <string>
+#include <vector>
 
 #include "base/memory/weak_ptr.h"
+#include "base/task/cancelable_task_tracker.h"
 #include "chrome/browser/chromeos/file_system_provider/provided_file_system_info.h"
 #include "chrome/browser/chromeos/file_system_provider/provided_file_system_interface.h"
 
 class Profile;
 
 namespace base {
 class Time;
 }  // namespace base
 
 namespace net {
@@ skipping 38 matching lines @@
                 base::Time modification_time,
                 std::string mime_type,
                 std::string contents);
 
   // Fetches a pointer to a fake entry registered in the fake file system. If
   // found, then the result is written to |fake_entry| and true is returned.
   // Otherwise, false is returned. |fake_entry| must not be NULL.
   bool GetEntry(const base::FilePath& entry_path, FakeEntry* fake_entry) const;
 
   // ProvidedFileSystemInterface overrides.
-  virtual void RequestUnmount(
+  virtual AbortCallback RequestUnmount(
       const fileapi::AsyncFileUtil::StatusCallback& callback) OVERRIDE;
-  virtual void GetMetadata(
+  virtual AbortCallback GetMetadata(
       const base::FilePath& entry_path,
       const ProvidedFileSystemInterface::GetMetadataCallback& callback)
       OVERRIDE;
-  virtual void ReadDirectory(
+  virtual AbortCallback ReadDirectory(
       const base::FilePath& directory_path,
       const fileapi::AsyncFileUtil::ReadDirectoryCallback& callback) OVERRIDE;
-  virtual void OpenFile(const base::FilePath& file_path,
-                        OpenFileMode mode,
-                        const OpenFileCallback& callback) OVERRIDE;
-  virtual void CloseFile(
+  virtual AbortCallback OpenFile(const base::FilePath& file_path,

[hirono, 2014/08/06 08:33:50]

nit: Make it the rule of breaking lines consistent with others.

[mtomasz, 2014/08/08 06:29:40]

It is done by git cl format. AFAIR it is correct. First we should align to the
first asgument, but if doesn't fit then use 4-space ident.

[hirono, 2014/08/08 07:23:38]

Acknowledged.

+                                 OpenFileMode mode,
+                                 const OpenFileCallback& callback) OVERRIDE;
+  virtual AbortCallback CloseFile(
       int file_handle,
       const fileapi::AsyncFileUtil::StatusCallback& callback) OVERRIDE;
-  virtual void ReadFile(int file_handle,
-                        net::IOBuffer* buffer,
-                        int64 offset,
-                        int length,
-                        const ReadChunkReceivedCallback& callback) OVERRIDE;
-  virtual void CreateDirectory(
+  virtual AbortCallback ReadFile(
+      int file_handle,
+      net::IOBuffer* buffer,
+      int64 offset,
+      int length,
+      const ReadChunkReceivedCallback& callback) OVERRIDE;
+  virtual AbortCallback CreateDirectory(
       const base::FilePath& directory_path,
       bool exclusive,
       bool recursive,
       const fileapi::AsyncFileUtil::StatusCallback& callback) OVERRIDE;
-  virtual void DeleteEntry(
+  virtual AbortCallback DeleteEntry(
       const base::FilePath& entry_path,
       bool recursive,
       const fileapi::AsyncFileUtil::StatusCallback& callback) OVERRIDE;
-  virtual void CreateFile(
+  virtual AbortCallback CreateFile(
       const base::FilePath& file_path,
       const fileapi::AsyncFileUtil::StatusCallback& callback) OVERRIDE;
-  virtual void CopyEntry(
+  virtual AbortCallback CopyEntry(
       const base::FilePath& source_path,
       const base::FilePath& target_path,
       const fileapi::AsyncFileUtil::StatusCallback& callback) OVERRIDE;
-  virtual void MoveEntry(
+  virtual AbortCallback MoveEntry(
       const base::FilePath& source_path,
       const base::FilePath& target_path,
       const fileapi::AsyncFileUtil::StatusCallback& callback) OVERRIDE;
-  virtual void Truncate(
+  virtual AbortCallback Truncate(
       const base::FilePath& file_path,
       int64 length,
       const fileapi::AsyncFileUtil::StatusCallback& callback) OVERRIDE;
-  virtual void WriteFile(
+  virtual AbortCallback WriteFile(
       int file_handle,
       net::IOBuffer* buffer,
       int64 offset,
       int length,
       const fileapi::AsyncFileUtil::StatusCallback& callback) OVERRIDE;
   virtual const ProvidedFileSystemInfo& GetFileSystemInfo() const OVERRIDE;
   virtual RequestManager* GetRequestManager() OVERRIDE;
   virtual base::WeakPtr<ProvidedFileSystemInterface> GetWeakPtr() OVERRIDE;
 
   // Factory callback, to be used in Service::SetFileSystemFactory(). The
   // |event_router| argument can be NULL.
   static ProvidedFileSystemInterface* Create(
       Profile* profile,
       const ProvidedFileSystemInfo& file_system_info);
 
  private:
   typedef std::map<base::FilePath, FakeEntry> Entries;
   typedef std::map<int, base::FilePath> OpenedFilesMap;
 
+  // Aborts a request. |task_id| refers to a posted callback returning a
+  // response for the operation, which will be cancelled, hence not called.
+  void Abort(int task_id,
+             const fileapi::AsyncFileUtil::StatusCallback& callback);
+
+  // Aborts a request. |task_ids| refers to a vector of posted callbacks
+  // returning a response for the operation, which will be cancelled, hence not
+  // called.
+  void AbortMany(const std::vector<int>& task_ids,
+                 const fileapi::AsyncFileUtil::StatusCallback& callback);
+
   ProvidedFileSystemInfo file_system_info_;
   Entries entries_;
   OpenedFilesMap opened_files_;
   int last_file_handle_;
+  base::CancelableTaskTracker tracker_;
 
-  base::WeakPtrFactory<ProvidedFileSystemInterface> weak_ptr_factory_;
+  base::WeakPtrFactory<FakeProvidedFileSystem> weak_ptr_factory_;
   DISALLOW_COPY_AND_ASSIGN(FakeProvidedFileSystem);
 };
 
 }  // namespace file_system_provider
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_FILE_SYSTEM_PROVIDER_FAKE_PROVIDED_FILE_SYSTEM_H_