Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(827)

Unified Diff: chrome/browser/chromeos/file_system_provider/provided_file_system_info.cc

Issue 624903002: [fsp] Group arguments for mounting into a struct. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed a bug. Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/file_system_provider/provided_file_system_info.cc
diff --git a/chrome/browser/chromeos/file_system_provider/provided_file_system_info.cc b/chrome/browser/chromeos/file_system_provider/provided_file_system_info.cc
index 58f8407bd31b86d52ebfd9f612fccc70e27e745c..6e9dd9c9260e49bcd86e48d642b9b573bf3bb93d 100644
--- a/chrome/browser/chromeos/file_system_provider/provided_file_system_info.cc
+++ b/chrome/browser/chromeos/file_system_provider/provided_file_system_info.cc
@@ -7,20 +7,30 @@
namespace chromeos {
namespace file_system_provider {
-ProvidedFileSystemInfo::ProvidedFileSystemInfo() {}
+MountOptions::MountOptions() : writable(false), supports_notify_tag(false) {
+}
+
+MountOptions::MountOptions(const std::string& file_system_id,
+ const std::string& display_name)
+ : file_system_id(file_system_id),
+ display_name(display_name),
+ writable(false),
+ supports_notify_tag(false) {
+}
+
+ProvidedFileSystemInfo::ProvidedFileSystemInfo()
+ : writable_(false), supports_notify_tag_(false) {
+}
ProvidedFileSystemInfo::ProvidedFileSystemInfo(
const std::string& extension_id,
- const std::string& file_system_id,
- const std::string& display_name,
- bool writable,
- bool supports_notify_tag,
+ const MountOptions& mount_options,
const base::FilePath& mount_path)
: extension_id_(extension_id),
- file_system_id_(file_system_id),
- display_name_(display_name),
- writable_(writable),
- supports_notify_tag_(supports_notify_tag),
+ file_system_id_(mount_options.file_system_id),
+ display_name_(mount_options.display_name),
+ writable_(mount_options.writable),
+ supports_notify_tag_(mount_options.supports_notify_tag),
mount_path_(mount_path) {
}

Powered by Google App Engine
This is Rietveld 408576698