[fsp] Group arguments for mounting into a struct.

chrome/browser/chromeos/file_system_provider/operations/move_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/move_entry.h"
 
 #include <string>
 #include <vector>
 
 #include "base/files/file.h"
@@ skipping 20 matching lines @@
 const base::FilePath::CharType kTargetPath[] = "/kitty/and/puppy/happy";
 
 }  // namespace
 
 class FileSystemProviderOperationsMoveEntryTest : public testing::Test {
  protected:
   FileSystemProviderOperationsMoveEntryTest() {}
   virtual ~FileSystemProviderOperationsMoveEntryTest() {}
 
   virtual void SetUp() override {
+    MountOptions mount_options(kFileSystemId, "" /* display_name */);
+    mount_options.writable = true;
     file_system_info_ =
-        ProvidedFileSystemInfo(kExtensionId,
-                               kFileSystemId,
-                               "" /* file_system_name */,
-                               true /* writable */,
-                               false /* supports_notify_tag */,
-                               base::FilePath() /* mount_path */);
+        ProvidedFileSystemInfo(kExtensionId, mount_options, base::FilePath());
   }
 
   ProvidedFileSystemInfo file_system_info_;
 };
 
 TEST_F(FileSystemProviderOperationsMoveEntryTest, Execute) {
   using extensions::api::file_system_provider::MoveEntryRequestedOptions;
 
   util::LoggingDispatchEventImpl dispatcher(true /* dispatch_reply */);
   util::StatusCallbackLog callback_log;
@@ skipping 43 matching lines @@
 
   EXPECT_FALSE(move_entry.Execute(kRequestId));
 }
 
 TEST_F(FileSystemProviderOperationsMoveEntryTest, 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 */,
-      false /* supports_notify_tag */,
+      MountOptions(kFileSystemId, "" /* display_name */),
       base::FilePath() /* mount_path */);
 
   MoveEntry move_entry(NULL,
                        read_only_file_system_info,
                        base::FilePath::FromUTF8Unsafe(kSourcePath),
                        base::FilePath::FromUTF8Unsafe(kTargetPath),
                        base::Bind(&util::LogStatusCallback, &callback_log));
   move_entry.SetDispatchEventImplForTesting(
       base::Bind(&util::LoggingDispatchEventImpl::OnDispatchEventImpl,
                  base::Unretained(&dispatcher)));
@@ skipping 41 matching lines @@
   move_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