[fsp] Do not allow write operations on read only file systems.

chrome/browser/chromeos/file_system_provider/operations/delete_entry_unittest.cc

 // Copyright 2014 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 "chrome/browser/chromeos/file_system_provider/operations/delete_entry.h"
 
 #include <string>
 #include <vector>
 
 #include "base/files/file.h"
@@ skipping 23 matching lines @@
 class FileSystemProviderOperationsDeleteEntryTest : public testing::Test {
  protected:
   FileSystemProviderOperationsDeleteEntryTest() {}
   virtual ~FileSystemProviderOperationsDeleteEntryTest() {}
 
   virtual void SetUp() OVERRIDE {
     file_system_info_ =
         ProvidedFileSystemInfo(kExtensionId,
                                kFileSystemId,
                                "" /* file_system_name */,
-                               false /* writable */,
+                               true /* writable */,
                                base::FilePath() /* mount_path */);
   }
 
   ProvidedFileSystemInfo file_system_info_;
 };
 
 TEST_F(FileSystemProviderOperationsDeleteEntryTest, Execute) {
   util::LoggingDispatchEventImpl dispatcher(true /* dispatch_reply */);
   util::StatusCallbackLog callback_log;
 
@@ skipping 45 matching lines @@
                            base::FilePath::FromUTF8Unsafe(kEntryPath),
                            true /* recursive */,
                            base::Bind(&util::LogStatusCallback, &callback_log));
   delete_entry.SetDispatchEventImplForTesting(
       base::Bind(&util::LoggingDispatchEventImpl::OnDispatchEventImpl,
                  base::Unretained(&dispatcher)));
 
   EXPECT_FALSE(delete_entry.Execute(kRequestId));
 }
 
+TEST_F(FileSystemProviderOperationsDeleteEntryTest, Execute_ReadOnly) {
+  util::LoggingDispatchEventImpl dispatcher(true /* dispatch_reply */);
+  util::StatusCallbackLog callback_log;
+
+  const ProvidedFileSystemInfo read_only_file_system_info(
+      kExtensionId,
+      kFileSystemId,
+      "" /* file_system_name */,
+      false /* writable */,
+      base::FilePath() /* mount_path */);
+
+  DeleteEntry delete_entry(NULL,
+                           read_only_file_system_info,
+                           base::FilePath::FromUTF8Unsafe(kEntryPath),
+                           true /* recursive */,
+                           base::Bind(&util::LogStatusCallback, &callback_log));
+  delete_entry.SetDispatchEventImplForTesting(
+      base::Bind(&util::LoggingDispatchEventImpl::OnDispatchEventImpl,
+                 base::Unretained(&dispatcher)));
+
+  EXPECT_FALSE(delete_entry.Execute(kRequestId));
+}
+
 TEST_F(FileSystemProviderOperationsDeleteEntryTest, OnSuccess) {
   util::LoggingDispatchEventImpl dispatcher(true /* dispatch_reply */);
   util::StatusCallbackLog callback_log;
 
   DeleteEntry delete_entry(NULL,
                            file_system_info_,
                            base::FilePath::FromUTF8Unsafe(kEntryPath),
                            true /* recursive */,
                            base::Bind(&util::LogStatusCallback, &callback_log));
   delete_entry.SetDispatchEventImplForTesting(
@@ skipping 27 matching lines @@
   delete_entry.OnError(kRequestId,
                        scoped_ptr<RequestValue>(new RequestValue()),
                        base::File::FILE_ERROR_TOO_MANY_OPENED);
   ASSERT_EQ(1u, callback_log.size());
   EXPECT_EQ(base::File::FILE_ERROR_TOO_MANY_OPENED, callback_log[0]);
 }
 
 }  // namespace operations
 }  // namespace file_system_provider
 }  // namespace chromeos