[Media Galleries] Populate volume metadata in ChromeOS/Linux

chrome/browser/storage_monitor/removable_device_notifications_linux.h

Index: chrome/browser/storage_monitor/removable_device_notifications_linux.h
diff --git a/chrome/browser/storage_monitor/removable_device_notifications_linux.h b/chrome/browser/storage_monitor/removable_device_notifications_linux.h
index 2c4a75612fa9bc50c4194ae5c9448eaf2efb8b16..e2bed53f3d2660cc17fd95ad5f567c78b2db653f 100644
--- a/chrome/browser/storage_monitor/removable_device_notifications_linux.h
+++ b/chrome/browser/storage_monitor/removable_device_notifications_linux.h
@@ -29,13 +29,17 @@ namespace base {
 class FilePath;
 }
 
-// Gets device information given a |device_path|. On success, fills in
-// |unique_id|, |name|, |removable| and |partition_size_in_bytes|.
+// Gets device information given a |device_path| and |mount_point|.
+// On success, fills in metadata fields.
 typedef void (*GetDeviceInfoFunc)(const base::FilePath& device_path,

[vandebo (ex-Chrome), 2013/03/01 22:21:48]

So I think the signature of this function is pretty consisten across platforms. 
I'm not sure how I feel about changing that.  Also, this is a function that gets
changed out for tests.  So the less code we have in here the better.

[Greg Billock, 2013/03/05 19:20:55]

We are in the position now where we can use inheritance for the tests instead of
the swappable function. I've changed a couple to use the mocks, I think the rest
can follow, or perhaps delegate to a protected method (which we are also doing
in a couple spots...)

This really should take a StorageInfo*.

[vandebo (ex-Chrome), 2013/03/06 01:34:54]

Inheritance for the test sounds good.  The swappable functions are ugly.  But
every bit of code that is in a function with a OS specific APIs that we can't
mock is code that can't be tested.  As much as possible, I'd like to keep the OS
calls in isolated functions that do as little as possible in order to increase
our testability. 

[Greg Billock, 2013/03/06 23:24:25]

I think we have a plan for this now.

-                                  std::string* unique_id,
+                                  const base::FilePath& mount_point,
+                                  std::string* device_id,
                                   string16* name,
                                   bool* removable,
-                                  uint64* partition_size_in_bytes);
+                                  uint64* partition_size_in_bytes,
+                                  string16* volume_label,
+                                  string16* vendor_name,
+                                  string16* model_name);
 
 namespace chrome {
 
@@ -79,19 +83,10 @@ class RemovableDeviceNotificationsLinux
   friend struct content::BrowserThread::DeleteOnThread<
       content::BrowserThread::FILE>;
 
-  // Structure to save mounted device information such as device path, unique
-  // identifier, device name and partition size.
-  struct MountPointInfo {
-    MountPointInfo();
+  base::FilePath GetDeviceForMountPoint(const base::FilePath& mount_point);
 
-    base::FilePath mount_device;
-    std::string device_id;
-    string16 device_name;
-    uint64 partition_size_in_bytes;
-  };
-
-  // Mapping of mount points to MountPointInfo.
-  typedef std::map<base::FilePath, MountPointInfo> MountMap;
+  // Mapping of mount points to StorageInfo.
+  typedef std::map<base::FilePath, StorageInfo> MountMap;
 
   // (mount point, priority)
   // For devices that are mounted to multiple mount points, this helps us track