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

Side by Side Diff: webkit/fileapi/isolated_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 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 #include "webkit/fileapi/isolated_mount_point_provider.h" 5 #include "webkit/fileapi/isolated_mount_point_provider.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/file_path.h" 10 #include "base/file_path.h"
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
80 FilePath IsolatedMountPointProvider::GetFileSystemRootPathOnFileThread( 80 FilePath IsolatedMountPointProvider::GetFileSystemRootPathOnFileThread(
81 const GURL& origin_url, 81 const GURL& origin_url,
82 FileSystemType type, 82 FileSystemType type,
83 const FilePath& virtual_path, 83 const FilePath& virtual_path,
84 bool create) { 84 bool create) {
85 // This is not supposed to be used. 85 // This is not supposed to be used.
86 NOTREACHED(); 86 NOTREACHED();
87 return FilePath(); 87 return FilePath();
88 } 88 }
89 89
90 bool IsolatedMountPointProvider::IsAccessAllowed( 90 bool IsolatedMountPointProvider::IsAccessAllowed(const FileSystemURL& url) {
91 const GURL& origin_url, FileSystemType type, const FilePath& virtual_path) {
92 return true; 91 return true;
93 } 92 }
94 93
95 bool IsolatedMountPointProvider::IsRestrictedFileName( 94 bool IsolatedMountPointProvider::IsRestrictedFileName(
96 const FilePath& filename) const { 95 const FilePath& filename) const {
97 // TODO(kinuko): We need to check platform-specific restricted file names 96 // TODO(kinuko): We need to check platform-specific restricted file names
98 // before we actually start allowing file creation in isolated file systems. 97 // before we actually start allowing file creation in isolated file systems.
99 return false; 98 return false;
100 } 99 }
101 100
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
174 void IsolatedMountPointProvider::DeleteFileSystem( 173 void IsolatedMountPointProvider::DeleteFileSystem(
175 const GURL& origin_url, 174 const GURL& origin_url,
176 FileSystemType type, 175 FileSystemType type,
177 FileSystemContext* context, 176 FileSystemContext* context,
178 const DeleteFileSystemCallback& callback) { 177 const DeleteFileSystemCallback& callback) {
179 NOTREACHED(); 178 NOTREACHED();
180 callback.Run(base::PLATFORM_FILE_ERROR_INVALID_OPERATION); 179 callback.Run(base::PLATFORM_FILE_ERROR_INVALID_OPERATION);
181 } 180 }
182 181
183 } // namespace fileapi 182 } // namespace fileapi
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698