[fsp] Create a RequestManager per a ProvidedFileSystem instance.

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 dedb17f43deefd761b2a9e384be9d392e8f292c5..97f09c22ce4540a1efa85c0ed02f7efdf34cc967 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
@@ -30,6 +30,9 @@ class ProvidedFileSystemInterface {
 
   // Returns a provided file system info for this file system.
   virtual const ProvidedFileSystemInfo& GetFileSystemInfo() const = 0;
+
+  // Returns a request manager for the file system if exists, or NULL.

[hashimoto, 2014/04/16 04:14:13]

Since the real implementation always returns a non-NULL pointer, how about
removing the "if exists, or NULL" part in sake of less ambiguity?
IIUC GetRequestManager() should not be called for the fake implementation, so
that we can put NOTREACHED() in the fake's method definition.

Another solution would be to add FullfillRequest() and RejectRequest() to this
class which simply calls corresponding methods of RequestManager.
In the fake, you can leave the method body empty.

[mtomasz, 2014/04/16 05:08:10]

NOTREACHED() sounds good to me. Done.

+  virtual RequestManager* GetRequestManager() = 0;
 };
 
 }  // namespace file_system_provider