[fsp] Refactor handling operations.

chrome/browser/chromeos/file_system_provider/provided_file_system_interface.h

Index: chrome/browser/chromeos/file_system_provider/provided_file_system_interface.h
diff --git a/chrome/browser/chromeos/file_system_provider/provided_file_system_interface.h b/chrome/browser/chromeos/file_system_provider/provided_file_system_interface.h
index 3a552d84ca4a94efcf3c49d6e432433a9ba58a5b..722ee3befe83713a509708d155b3c96398817c89 100644
--- a/chrome/browser/chromeos/file_system_provider/provided_file_system_interface.h
+++ b/chrome/browser/chromeos/file_system_provider/provided_file_system_interface.h
@@ -5,10 +5,15 @@
 #ifndef CHROME_BROWSER_CHROMEOS_FILE_SYSTEM_PROVIDER_PROVIDED_FILE_SYSTEM_INTERFACE_H_
 #define CHROME_BROWSER_CHROMEOS_FILE_SYSTEM_PROVIDER_PROVIDED_FILE_SYSTEM_INTERFACE_H_
 
+#include "base/memory/weak_ptr.h"
 #include "webkit/browser/fileapi/async_file_util.h"
 
 class EventRouter;
 
+namespace base {
+class FilePath;
+}  // namespace base
+
 namespace chromeos {
 namespace file_system_provider {
 
@@ -23,9 +28,8 @@ class ProvidedFileSystemInterface {
   virtual ~ProvidedFileSystemInterface() {}
 
   // Requests unmounting of the file system. The callback is called when the
-  // request is accepted or rejected, with an error code. Returns false if the
-  // request could not been created, true otherwise.
-  virtual bool RequestUnmount(
+  // request is accepted or rejected, with an error code.
+  virtual void RequestUnmount(
       const fileapi::AsyncFileUtil::StatusCallback& callback) = 0;
 
   // Returns a provided file system info for this file system.
@@ -33,6 +37,9 @@ class ProvidedFileSystemInterface {
 
   // Returns a request manager for the file system.
   virtual RequestManager* GetRequestManager() = 0;
+
+  // Returns a weak pointer for the file system class.
+  virtual base::WeakPtr<ProvidedFileSystemInterface> GetWeakPtr() = 0;

[kinaba, 2014/04/25 06:35:06]

Do you have specific reason for this method to be public?

[mtomasz, 2014/04/28 00:42:47]

This CL is a split of a big CL. I just removed this, since it should be added
later. Done.

 };
 
 }  // namespace file_system_provider