Isolated FS for media devices.

chrome/browser/media_gallery/media_file_system_registry.h

Index: chrome/browser/media_gallery/media_file_system_registry.h
diff --git a/chrome/browser/media_gallery/media_file_system_registry.h b/chrome/browser/media_gallery/media_file_system_registry.h
index a4055710de1bb16d1555360f1d2503b8d1fa8fe9..f4d1f8518aed11976d3f2b419fb7228ac59398a3 100644
--- a/chrome/browser/media_gallery/media_file_system_registry.h
+++ b/chrome/browser/media_gallery/media_file_system_registry.h
@@ -72,8 +72,9 @@ class MediaFileSystemRegistry
   typedef std::map<const content::RenderProcessHost*,
                    MediaPathToFSIDMap> ChildIdToMediaFSMap;
 
-  // Mapping of device id to mount path.
-  typedef std::map<std::string, FilePath> DeviceIdToMediaPathMap;
+  // Mapping of device id to media device info.
+  typedef std::map<std::string, base::SystemMonitor::MediaDeviceInfo>
+      DeviceIdToInfoMap;
 
   // Obtain an instance of this class via GetInstance().
   MediaFileSystemRegistry();
@@ -85,16 +86,21 @@ class MediaFileSystemRegistry
   void UnregisterForRPHGoneNotifications(const content::RenderProcessHost* rph);
 
   // Registers a path as a media file system and return the filesystem id.
-  std::string RegisterPathAsFileSystem(const FilePath& path);
+  std::string RegisterPathAsFileSystem(
+      const base::SystemMonitor::MediaDeviceType& type,

[Lei Zhang, 2012/08/02 20:57:19]

nit: Please change this and RevokeMediaFileSystem()'s |type| parameter to
|device_type| so the .h and .cc files match.

[kmadhusu, 2012/08/02 22:05:39]

Done.

+      const FilePath& path);
+
 
   // Revoke a media file system with a given |path|.
-  void RevokeMediaFileSystem(const FilePath& path);
+  void RevokeMediaFileSystem(
+      const base::SystemMonitor::MediaDeviceType& type,
+      const FilePath& path);
 
   // Only accessed on the UI thread.
   ChildIdToMediaFSMap media_fs_map_;
 
   // Only accessed on the UI thread.
-  DeviceIdToMediaPathMap device_id_map_;
+  DeviceIdToInfoMap device_id_map_;
 
   // Is only used on the UI thread.
   content::NotificationRegistrar registrar_;