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

Side by Side Diff: webkit/fileapi/file_system_mount_point_provider.h

Issue 10873055: Changed FileSystemPointProvider::IsAcccessAllowed() to take a single FileSystemURL instead of a tri… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix for typos from Kinuko's review. Sorry, I will look more closely at my own diff next time Created 8 years, 3 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef WEBKIT_FILEAPI_FILE_SYSTEM_MOUNT_POINT_PROVIDER_H_ 5 #ifndef WEBKIT_FILEAPI_FILE_SYSTEM_MOUNT_POINT_PROVIDER_H_
6 #define WEBKIT_FILEAPI_FILE_SYSTEM_MOUNT_POINT_PROVIDER_H_ 6 #define WEBKIT_FILEAPI_FILE_SYSTEM_MOUNT_POINT_PROVIDER_H_
7 7
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
54 // |origin_url| and |type| on the file thread. 54 // |origin_url| and |type| on the file thread.
55 // If |create| is true this may also create the root directory for 55 // If |create| is true this may also create the root directory for
56 // the filesystem if it doesn't exist. 56 // the filesystem if it doesn't exist.
57 virtual FilePath GetFileSystemRootPathOnFileThread( 57 virtual FilePath GetFileSystemRootPathOnFileThread(
58 const GURL& origin_url, 58 const GURL& origin_url,
59 FileSystemType type, 59 FileSystemType type,
60 const FilePath& virtual_path, 60 const FilePath& virtual_path,
61 bool create) = 0; 61 bool create) = 0;
62 62
63 // Checks if access to |virtual_path| is allowed from |origin_url|. 63 // Checks if access to |virtual_path| is allowed from |origin_url|.
64 virtual bool IsAccessAllowed(const GURL& origin_url, 64 virtual bool IsAccessAllowed(const FileSystemURL& url) = 0;
65 FileSystemType type,
66 const FilePath& virtual_path) = 0;
67 65
68 // Checks if a given |name| contains any restricted names/chars in it. 66 // Checks if a given |name| contains any restricted names/chars in it.
69 // Callable on any thread. 67 // Callable on any thread.
70 virtual bool IsRestrictedFileName(const FilePath& filename) const = 0; 68 virtual bool IsRestrictedFileName(const FilePath& filename) const = 0;
71 69
72 // Returns the specialized FileSystemFileUtil for this mount point. 70 // Returns the specialized FileSystemFileUtil for this mount point.
73 virtual FileSystemFileUtil* GetFileUtil(FileSystemType type) = 0; 71 virtual FileSystemFileUtil* GetFileUtil(FileSystemType type) = 0;
74 72
75 // Returns file path we should use to check access permissions for 73 // Returns file path we should use to check access permissions for
76 // |virtual_path|. 74 // |virtual_path|.
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
148 virtual void RemoveMountPoint(const FilePath& mount_point) = 0; 146 virtual void RemoveMountPoint(const FilePath& mount_point) = 0;
149 // Gets virtual path by known filesystem path. Returns false when filesystem 147 // Gets virtual path by known filesystem path. Returns false when filesystem
150 // path is not exposed by this provider. 148 // path is not exposed by this provider.
151 virtual bool GetVirtualPath(const FilePath& file_system_path, 149 virtual bool GetVirtualPath(const FilePath& file_system_path,
152 FilePath* virtual_path) = 0; 150 FilePath* virtual_path) = 0;
153 }; 151 };
154 152
155 } // namespace fileapi 153 } // namespace fileapi
156 154
157 #endif // WEBKIT_FILEAPI_FILE_SYSTEM_MOUNT_POINT_PROVIDER_H_ 155 #endif // WEBKIT_FILEAPI_FILE_SYSTEM_MOUNT_POINT_PROVIDER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698