Remove ugly writable flags from IsolatedContext

For regular chrome centralize the read/write permission to ChildProcessSecurityPolicy and do nothing in fileapi layer.

For DRT/content_shell disable writes at SimpleFileSystem layer as well as we do in FileAPIMessageFilter for chrome.

BUG=none
TEST=existing tests
TEST=layout tests https://bugs.webkit.org/show_bug.cgi?id=89981

Review URL: https://chromiumcodereview.appspot.com/10692005

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145063 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 2 insertions, 14 deletions

diff --git a/webkit/fileapi/isolated_file_util.cc b/webkit/fileapi/isolated_file_util.cc
index bb145b6..6c0bf6a 100644
--- a/webkit/fileapi/isolated_file_util.cc
+++ b/webkit/fileapi/isolated_file_util.cc
@@ -280,7 +280,7 @@ PlatformFileError IsolatedFileUtil::Touch(
     const base::Time& last_access_time,
     const base::Time& last_modified_time) {
   FilePath platform_path;
-  if (!GetPlatformPathForWrite(url, &platform_path) || platform_path.empty())
+  if (!GetPlatformPath(url, &platform_path) || platform_path.empty())
     return base::PLATFORM_FILE_ERROR_SECURITY;
   return NativeFileUtil::Touch(
       platform_path, last_access_time, last_modified_time);
@@ -291,7 +291,7 @@ PlatformFileError IsolatedFileUtil::Truncate(
     const FileSystemURL& url,
     int64 length) {
   FilePath platform_path;
-  if (!GetPlatformPathForWrite(url, &platform_path) || platform_path.empty())
+  if (!GetPlatformPath(url, &platform_path) || platform_path.empty())
     return base::PLATFORM_FILE_ERROR_SECURITY;
   return NativeFileUtil::Truncate(platform_path, length);
 }
@@ -380,16 +380,4 @@ bool IsolatedFileUtil::GetPlatformPath(const FileSystemURL& url,
   return true;
 }
 
-bool IsolatedFileUtil::GetPlatformPathForWrite(
-    const FileSystemURL& url,
-    FilePath* platform_path) const {
-  DCHECK(platform_path);
-  std::string filesystem_id;
-  FilePath root_path;
-  if (!IsolatedContext::GetInstance()->CrackIsolatedPath(
-          url.path(), &filesystem_id, &root_path, platform_path))
-    return false;
-  return IsolatedContext::GetInstance()->IsWritable(filesystem_id);
-}
-
 }  // namespace