SyncFS: Introduce SyncFileSystemBackend

webkit/browser/fileapi/file_system_mount_point_provider.h

Index: webkit/browser/fileapi/file_system_mount_point_provider.h
diff --git a/webkit/browser/fileapi/file_system_mount_point_provider.h b/webkit/browser/fileapi/file_system_mount_point_provider.h
index 3de8793819b5cfac6ba8daaf40763d8293a44184..cfc8e5e300d7004ff979860d327c9f511c9b0066 100644
--- a/webkit/browser/fileapi/file_system_mount_point_provider.h
+++ b/webkit/browser/fileapi/file_system_mount_point_provider.h
@@ -14,6 +14,7 @@
 #include "base/platform_file.h"
 #include "webkit/browser/fileapi/file_permission_policy.h"
 #include "webkit/browser/fileapi/open_file_system_mode.h"
+#include "webkit/browser/fileapi/task_runner_bound_observer_list.h"
 #include "webkit/browser/webkit_storage_browser_export.h"
 #include "webkit/common/fileapi/file_system_types.h"
 
@@ -52,6 +53,11 @@ class WEBKIT_STORAGE_BROWSER_EXPORT FileSystemMountPointProvider {
   // One mount point provider may be able to handle multiple filesystem types.
   virtual bool CanHandleType(FileSystemType type) const = 0;
 
+  // Returns the root URI for the given |origin_url| and |type|. It is ok to
+  // return an empty GURL if this mount point provider cannot handle |type|.
+  virtual GURL GetRootURI(const GURL& origin_url,
+                          FileSystemType type) const = 0;

[kinuko, 2013/07/08 12:31:20]

I don't think this is necessary here. (Please see the comment in
FileSystemContext::OpenFileSystem)

[nhiroki, 2013/07/22 04:34:14]

Removed.

+
   // Initializes the filesystem for the given |origin_url| and |type|.
   // This verifies if it is allowed to request (or create) the filesystem
   // and if it can access (or create) the root directory of the mount point.
@@ -115,6 +121,11 @@ class WEBKIT_STORAGE_BROWSER_EXPORT FileSystemMountPointProvider {
   // Returns the specialized FileSystemQuotaUtil for this mount point.
   // This could return NULL if this mount point does not support quota.
   virtual FileSystemQuotaUtil* GetQuotaUtil() = 0;
+
+  // Returns the update observers for this mount point. This could return NULL
+  // if update observer is not available in this mount point.
+  virtual const UpdateObserverList* GetUpdateObservers(
+      FileSystemType type) const = 0;

[kinuko, 2013/07/08 12:31:20]

Can we move this method out of FileSystemMountPointProvider but into QuotaUtil? 
This one's meaningful only in the sandboxed context (at least now) and I'm
trying to move all those sandbox-fs only methods to QuotaUtil. (So that
non-sandbox FS modules don't need to care about them)

Also can you move this change into a separate CL?

[nhiroki, 2013/07/22 04:34:14]

Done in http://crrev.com/211332/

 };
 
 // An interface to control external file system access permissions.