Retry: Make Isolated file system writable only if it is configured so

- CreateFileStreamWriter wiring in FSO
- Add yet another write-protection at IsolatedContext (in addition to ChildProcessSecurityPolicy, so that we can make each fs read-only or writable in DRT/testing)

BUG=none
TEST=existing tests should pass

Original review URL: https://chromiumcodereview.appspot.com/10540070

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

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

1 files changed, 16 insertions, 4 deletions

diff --git a/webkit/fileapi/file_system_operation.cc b/webkit/fileapi/file_system_operation.cc
index 1084fd8..5c77024 100644
--- a/webkit/fileapi/file_system_operation.cc
+++ b/webkit/fileapi/file_system_operation.cc
@@ -287,13 +287,25 @@ void FileSystemOperation::Write(
     delete this;
     return;
   }
+
+  FileSystemMountPointProvider* provider = file_system_context()->
+      GetMountPointProvider(src_path_.type());
+  DCHECK(provider);
+  scoped_ptr<FileStreamWriter> writer(provider->CreateFileStreamWriter(
+          path_url, offset, file_system_context()));
+
+  if (!writer.get()) {
+    // Write is not supported.
+    callback.Run(base::PLATFORM_FILE_ERROR_SECURITY, 0, false);
+    delete this;
+    return;
+  }
+
   DCHECK(blob_url.is_valid());
   file_writer_delegate_.reset(new FileWriterDelegate(
       base::Bind(&FileSystemOperation::DidWrite, weak_factory_.GetWeakPtr()),
-      scoped_ptr<FileStreamWriter>(
-          new SandboxFileStreamWriter(file_system_context(),
-                                      path_url,
-                                      offset))));
+      writer.Pass()));
+
   set_write_callback(callback);
   scoped_ptr<net::URLRequest> blob_request(
       new net::URLRequest(blob_url, file_writer_delegate_.get()));