Change access mode of disk devices when mounting based on config.

chrome/browser/chromeos/extensions/file_manager/private_api_mount.cc

 // Copyright 2013 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/extensions/file_manager/private_api_mount.h"
 
 #include <string>
 
 #include "base/files/file_util.h"
 #include "base/format_macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/chromeos/drive/file_system_util.h"
 #include "chrome/browser/chromeos/extensions/file_manager/private_api_util.h"
 #include "chrome/browser/chromeos/file_manager/fileapi_util.h"
 #include "chrome/browser/chromeos/file_manager/volume_manager.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/extensions/api/file_manager_private.h"
+#include "chrome/common/pref_names.h"
 #include "chromeos/disks/disk_mount_manager.h"
 #include "components/drive/chromeos/file_system_interface.h"
 #include "components/drive/event_logger.h"
+#include "components/prefs/pref_service.h"
 #include "content/public/browser/browser_thread.h"
 #include "google_apis/drive/task_util.h"
 #include "ui/shell_dialogs/selected_file_info.h"
 
 using chromeos::disks::DiskMountManager;
 using content::BrowserThread;
 namespace file_manager_private = extensions::api::file_manager_private;
 
 namespace extensions {
 
@@ skipping 121 matching lines @@
     SendResponse(false);
     return;
   }
 
   // Pass back the actual source path of the mount point.
   SetResult(base::MakeUnique<base::StringValue>(file_path.AsUTF8Unsafe()));
   SendResponse(true);
 
   // MountPath() takes a std::string.
   DiskMountManager* disk_mount_manager = DiskMountManager::GetInstance();
+  bool read_only =
+      GetProfile()->GetPrefs()->GetBoolean(prefs::kExternalStorageReadOnly);
+  chromeos::MountAccessMode access_mode =
+      read_only ? chromeos::MOUNT_ACCESS_MODE_READ_ONLY
+                : chromeos::MOUNT_ACCESS_MODE_READ_WRITE;
   disk_mount_manager->MountPath(
       file_path.AsUTF8Unsafe(),
       base::FilePath(display_name.Extension()).AsUTF8Unsafe(),
-      display_name.AsUTF8Unsafe(),
-      chromeos::MOUNT_TYPE_ARCHIVE);
+      display_name.AsUTF8Unsafe(), chromeos::MOUNT_TYPE_ARCHIVE, access_mode);

[fukino, 2016/08/03 07:18:19]

Should we specify access mode for mounted archive files?

[yamaguchi, 2016/08/03 10:22:14]

Done.
Mounted archives are treated as read-only by itself now.

 }
 
 bool FileManagerPrivateRemoveMountFunction::RunAsync() {
   using file_manager_private::RemoveMount::Params;
   const std::unique_ptr<Params> params(Params::Create(*args_));
   EXTENSION_FUNCTION_VALIDATE(params);
 
   drive::EventLogger* logger = file_manager::util::GetLogger(GetProfile());
   if (logger) {
     logger->Log(logging::LOG_INFO, "%s[%d] called. (volume_id: '%s')", name(),
@@ skipping 68 matching lines @@
                 name(), request_id(), log_string.c_str(), result.size());
   }
 
   results_ =
       file_manager_private::GetVolumeMetadataList::Results::Create(result);
   SendResponse(true);
   return true;
 }
 
 }  // namespace extensions