Propagate the read_only_hardware flag of volumes to js.

chrome/browser/chromeos/file_manager/volume_manager.h

Index: chrome/browser/chromeos/file_manager/volume_manager.h
diff --git a/chrome/browser/chromeos/file_manager/volume_manager.h b/chrome/browser/chromeos/file_manager/volume_manager.h
index cc4ae549c50a89f43747aa34f45edff7cc55dc43..376bdc06db6c1fe2dd3b22cc2243f6cb3b769c32 100644
--- a/chrome/browser/chromeos/file_manager/volume_manager.h
+++ b/chrome/browser/chromeos/file_manager/volume_manager.h
@@ -116,7 +116,13 @@ class Volume : public base::SupportsWeakPtr<Volume> {
   }
   const std::string& volume_label() const { return volume_label_; }
   bool is_parent() const { return is_parent_; }
+  // Whether the device is writable by applications.
   bool is_read_only() const { return is_read_only_; }
+  // Whether the device is write-protected by hardware. This field is valid
+  // only when device_type is VOLUME_TYPE_REMOVABLE_DISK_PARTITION and
+  // source is SOURCE_DEVICE.
+  // When this value is true, is_read_only() is also true.

[fukino, 2016/10/26 07:23:31]

Could you mention an example in which is_read_only_ is true but
is_read_only_hardware_ is false.

In CL description or code comment, could you explain why we need to have these
two separated fields.

[yamaguchi, 2016/10/26 11:14:20]

Done.
Described for what this change is, in the CL description.

+  bool is_read_only_hardware() const { return is_read_only_hardware_; }
   bool has_media() const { return has_media_; }
   bool configurable() const { return configurable_; }
   bool watchable() const { return watchable_; }
@@ -175,9 +181,13 @@ class Volume : public base::SupportsWeakPtr<Volume> {
   // Is the device is a parent device (i.e. sdb rather than sdb1).
   bool is_parent_;
 
-  // True if the volume is read only.
+  // True if the volume is not writable by applications.
   bool is_read_only_;
 
+  // True if the volume is made read_only due to its hardware.
+  // This implies is_read_only_.
+  bool is_read_only_hardware_;
+
   // True if the volume contains media.
   bool has_media_;