Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(9)

Unified Diff: webkit/fileapi/file_system_mount_point_provider.h

Issue 11648027: Extract external file systems handling from isolated context. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 7e2ef117acc4e1c8dd9696d062614a4554ca1bde..c4f88e2e538c70b624c621ccb072a31d72f637b2 100644
--- a/webkit/fileapi/file_system_mount_point_provider.h
+++ b/webkit/fileapi/file_system_mount_point_provider.h
@@ -128,7 +128,8 @@ class ExternalFileSystemMountPointProvider
// 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.
- virtual std::vector<FilePath> GetRootDirectories() const = 0;
+ virtual std::vector<FilePath> GetRootDirectories(
+ bool include_system_mount_points) const = 0;
// Grants access to all external file system from extension identified with
// |extension_id|.
virtual void GrantFullAccessToExtension(const std::string& extension_id) = 0;
@@ -140,14 +141,14 @@ class ExternalFileSystemMountPointProvider
virtual void RevokeAccessForExtension(
const std::string& extension_id) = 0;
// Checks if a given |mount_point| already exists.
- virtual bool HasMountPoint(const FilePath& mount_point) = 0;
+ virtual bool HasMountPoint(const FilePath& mount_point) const = 0;
// Adds a new local mount point.
- virtual void AddLocalMountPoint(const FilePath& mount_point) = 0;
+ virtual bool AddLocalMountPoint(const FilePath& mount_point) = 0;
// Adds a new local mount point that will be accessible only by extensions
// that have been granted full acess for all external file systems.
- virtual void AddRestrictedLocalMountPoint(const FilePath& mount_point) = 0;
+ virtual bool AddRestrictedLocalMountPoint(const FilePath& mount_point) = 0;
// Adds a new remote mount point.
- virtual void AddRemoteMountPoint(
+ virtual bool AddRemoteMountPoint(
const FilePath& mount_point,
RemoteFileSystemProxyInterface* remote_proxy) = 0;
// Removes a mount point.

Powered by Google App Engine
This is Rietveld 408576698