Deprecate MountPointProvider::IsAccessAllowed in favor of GetPermissionPolicy

webkit/fileapi/local_file_system_operation.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/local_file_system_operation.h"
 
 #include "base/bind.h"
 #include "base/single_thread_task_runner.h"
 #include "base/time.h"
 #include "base/utf_string_conversions.h"
@@ skipping 817 matching lines @@
     file_ref = ShareableFileReference::GetOrCreate(
         platform_path, ShareableFileReference::DELETE_ON_FINAL_RELEASE,
         file_system_context()->task_runners()->file_task_runner());
   }
   callback.Run(result, file_info, platform_path, file_ref);
 }
 
 base::PlatformFileError LocalFileSystemOperation::SetUp(
     const FileSystemURL& url,
     SetUpMode mode) {
-  if (!url.is_valid())
-    return base::PLATFORM_FILE_ERROR_INVALID_URL;
-
-  // Restricted file system is read-only.
-  if (url.type() == fileapi::kFileSystemTypeRestrictedNativeLocal &&
-      mode != SETUP_FOR_READ)
-    return base::PLATFORM_FILE_ERROR_SECURITY;
+  DCHECK(url.is_valid());
 
   async_file_util_ = file_system_context()->GetAsyncFileUtil(url.type());
   if (!async_file_util_)
     return base::PLATFORM_FILE_ERROR_SECURITY;
 
   // If this operation is created for recursive sub-operations (i.e.
   // operation context is overridden from another operation) we skip
-  // some duplicated security checks.
+  // some duplicated notifications.
   if (overriding_operation_context_)
     return base::PLATFORM_FILE_OK;
 
-  if (!file_system_context()->GetMountPointProvider(
-          url.type())->IsAccessAllowed(url))
-    return base::PLATFORM_FILE_ERROR_SECURITY;
-
+  // Notify / set up observers.
   if (mode == SETUP_FOR_READ) {
     operation_context()->access_observers()->Notify(
         &FileAccessObserver::OnAccess, MakeTuple(url));
-    return base::PLATFORM_FILE_OK;
-  }
-
-  DCHECK(mode == SETUP_FOR_WRITE || mode == SETUP_FOR_CREATE);
-
-  scoped_update_notifiers_.push_back(new ScopedUpdateNotifier(
-      operation_context(), url));
-
-  // Any write access is disallowed on the root path.
-  if (url.path().value().length() == 0 ||
-      url.path().DirName().value() == url.path().value())
-    return base::PLATFORM_FILE_ERROR_SECURITY;
-
-  if (mode == SETUP_FOR_CREATE) {
-    FileSystemMountPointProvider* provider = file_system_context()->
-        GetMountPointProvider(url.type());
-
-    // Check if the cracked file name looks good to create.
-    if (provider->IsRestrictedFileName(VirtualPath::BaseName(url.path())))
-      return base::PLATFORM_FILE_ERROR_SECURITY;
+  } else {
+    DCHECK(mode == SETUP_FOR_WRITE || mode == SETUP_FOR_CREATE);
+    scoped_update_notifiers_.push_back(new ScopedUpdateNotifier(
+        operation_context(), url));
   }
 
   return base::PLATFORM_FILE_OK;
 }
 
 bool LocalFileSystemOperation::SetPendingOperationType(OperationType type) {
   if (pending_operation_ != kOperationNone)
     return false;
   pending_operation_ = type;
   return true;
 }
 
 }  // namespace fileapi