Deprecate MountPointProvider::IsAccessAllowed in favor of GetPermissionPolicy

webkit/fileapi/isolated_mount_point_provider.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "webkit/fileapi/isolated_mount_point_provider.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/file_path.h"
@@ skipping 61 matching lines @@
 }
 
 base::FilePath IsolatedMountPointProvider::GetFileSystemRootPathOnFileThread(
     const FileSystemURL& url,
     bool create) {
   // This is not supposed to be used.
   NOTREACHED();
   return base::FilePath();
 }
 
-bool IsolatedMountPointProvider::IsAccessAllowed(const FileSystemURL& url) {
-  return true;
-}
-
-bool IsolatedMountPointProvider::IsRestrictedFileName(
-    const base::FilePath& filename) const {
-  // TODO(kinuko): We need to check platform-specific restricted file names
-  // before we actually start allowing file creation in isolated file systems.
-  return false;
-}
-
 FileSystemFileUtil* IsolatedMountPointProvider::GetFileUtil(
     FileSystemType type) {
   switch (type) {
     case kFileSystemTypeNativeLocal:
       return isolated_file_util_->sync_file_util();
     case kFileSystemTypeDragged:
       return dragged_file_util_->sync_file_util();
     case kFileSystemTypeNativeMedia:
       return native_media_file_util_->sync_file_util();
     case kFileSystemTypeDeviceMedia:
@@ skipping 26 matching lines @@
     default:
       NOTREACHED();
   }
   return NULL;
 }
 
 FilePermissionPolicy IsolatedMountPointProvider::GetPermissionPolicy(
     const FileSystemURL& url, int permissions) const {
   if (url.type() == kFileSystemTypeDragged && url.path().empty()) {
     // The root directory of the dragged filesystem must be always read-only.
-    if (permissions != kReadFilePermissions)
+    if (permissions & ~fileapi::kReadFilePermissions)
       return FILE_PERMISSION_ALWAYS_DENY;
   }
   // Access to isolated file systems should be checked using per-filesystem
   // access permission.
   return FILE_PERMISSION_USE_FILESYSTEM_PERMISSION;
 }
 
 FileSystemOperation* IsolatedMountPointProvider::CreateFileSystemOperation(
     const FileSystemURL& url,
     FileSystemContext* context,
@@ skipping 41 matching lines @@
 void IsolatedMountPointProvider::DeleteFileSystem(
     const GURL& origin_url,
     FileSystemType type,
     FileSystemContext* context,
     const DeleteFileSystemCallback& callback) {
   NOTREACHED();
   callback.Run(base::PLATFORM_FILE_ERROR_INVALID_OPERATION);
 }
 
 }  // namespace fileapi