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

Unified Diff: webkit/fileapi/sandbox_mount_point_provider.cc

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, 4 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/sandbox_mount_point_provider.cc
diff --git a/webkit/fileapi/sandbox_mount_point_provider.cc b/webkit/fileapi/sandbox_mount_point_provider.cc
index 854d264a93a5be3cf1ede2e474d3c72b5dc18b27..4a909ee265919dde9658f624f30e847d0968a4f1 100644
--- a/webkit/fileapi/sandbox_mount_point_provider.cc
+++ b/webkit/fileapi/sandbox_mount_point_provider.cc
@@ -384,14 +384,13 @@ SandboxMountPointProvider::GetFileSystemRootPathOnFileThread(
return GetBaseDirectoryForOriginAndType(origin_url, type, create);
}
-bool SandboxMountPointProvider::IsAccessAllowed(const GURL& origin_url,
- FileSystemType type,
- const FilePath& unused) {
+bool SandboxMountPointProvider::IsAccessAllowed(const FileSystemURL& url) {
+ FileSystemType type = url.type();
if (type != kFileSystemTypeTemporary && type != kFileSystemTypePersistent)
return false;
// We essentially depend on quota to do our access controls, so here
// we only check if the requested scheme is allowed or not.
- return IsAllowedScheme(origin_url);
+ return IsAllowedScheme(url.origin());
}
bool SandboxMountPointProvider::IsRestrictedFileName(const FilePath& filename)

Powered by Google App Engine
This is Rietveld 408576698