Deprecate MountPointProvider::IsAccessAllowed in favor of GetPermissionPolicy

webkit/fileapi/file_system_mount_point_provider.h

Index: webkit/fileapi/file_system_mount_point_provider.h
diff --git a/webkit/fileapi/file_system_mount_point_provider.h b/webkit/fileapi/file_system_mount_point_provider.h
index 2e1937278671234377f542078b145516b72fd8b4..b93d282303325f7aea9b51384a02d473c91df9e4 100644
--- a/webkit/fileapi/file_system_mount_point_provider.h
+++ b/webkit/fileapi/file_system_mount_point_provider.h
@@ -60,13 +60,6 @@ class WEBKIT_STORAGE_EXPORT FileSystemMountPointProvider {
       const FileSystemURL& url,
       bool create) = 0;
 
-  // Checks if access to |virtual_path| is allowed from |origin_url|.
-  virtual bool IsAccessAllowed(const FileSystemURL& url) = 0;
-
-  // Checks if a given |name| contains any restricted names/chars in it.
-  // Callable on any thread.
-  virtual bool IsRestrictedFileName(const base::FilePath& filename) const = 0;
-
   // Returns the specialized FileSystemFileUtil for this mount point.
   // It is ok to return NULL if the filesystem doesn't support synchronous
   // version of FileUtil.
@@ -132,6 +125,8 @@ class WEBKIT_STORAGE_EXPORT FileSystemMountPointProvider {
 class ExternalFileSystemMountPointProvider
     : public FileSystemMountPointProvider {
  public:
+  // Returns true if |url| is allowed to be accessed.
+  virtual bool IsAccessAllowed(const fileapi::FileSystemURL& url) const = 0;

[ericu, 2013/02/13 01:45:56]

Given the other methods, please add a comment explaining how this one differs. 
"is allowed" by whom, and if this should override other checks or if it's an
additional requirement.

If this will only ever be used by implementers of this interface, we could also
make it protected to clarify that.

[kinuko, 2013/02/13 03:36:30]

Done.

   // Returns the list of top level directories that are exposed by this
   // provider. This list is used to set appropriate child process file access
   // permissions.