[Chrome OS]Implement MediaStorageUtil::GetDeviceInfoForPath function to support media gallery permi…

chrome/browser/system_monitor/media_transfer_protocol_device_observer_chromeos.h

Index: chrome/browser/system_monitor/media_transfer_protocol_device_observer_chromeos.h
diff --git a/chrome/browser/system_monitor/media_transfer_protocol_device_observer_chromeos.h b/chrome/browser/system_monitor/media_transfer_protocol_device_observer_chromeos.h
index ab252eca6f157e08ecc7dd9b0b0c5de5a03a70b9..440a9327e84aba1063549893c22c38ed51afd703 100644
--- a/chrome/browser/system_monitor/media_transfer_protocol_device_observer_chromeos.h
+++ b/chrome/browser/system_monitor/media_transfer_protocol_device_observer_chromeos.h
@@ -9,8 +9,11 @@
 #include <string>
 
 #include "base/string16.h"
+#include "base/system_monitor/system_monitor.h"
 #include "chrome/browser/chromeos/mtp/media_transfer_protocol_manager.h"
 
+class FilePath;
+
 namespace chromeos {
 namespace mtp {
 
@@ -26,9 +29,18 @@ typedef void (*GetStorageInfoFunc)(const std::string& storage_name,
 class MediaTransferProtocolDeviceObserverCros
     : public MediaTransferProtocolManager::Observer {
  public:
+  // Should only be called by browser start up code. Use GetInstance() instead.
   MediaTransferProtocolDeviceObserverCros();
   virtual ~MediaTransferProtocolDeviceObserverCros();
 
+  static MediaTransferProtocolDeviceObserverCros* GetInstance();
+
+  // Finds the storage that contains |path| and populates |storage_info|.
+  // Returns false if unable to find the storage.
+  bool GetStorageInfoForPath(
+      const FilePath& path,
+      base::SystemMonitor::RemovableStorageInfo* storage_info) const;
+
  protected:
   // Only used in unit tests.
   explicit MediaTransferProtocolDeviceObserverCros(
@@ -40,14 +52,16 @@ class MediaTransferProtocolDeviceObserverCros
                               const std::string& storage_name) OVERRIDE;
 
  private:
-  // Mapping of storage name and device id.
-  typedef std::map<std::string, std::string> StorageNameToIdMap;
+  // Mapping of storage location and mtp storage info object.

[vandebo (ex-Chrome), 2012/09/16 23:03:19]

Looks like you could continue using storage name as the key... though there's
not much difference.

+  typedef std::map<std::string,
+                   base::SystemMonitor::RemovableStorageInfo>
+      StorageLocationToInfoMap;
 
   // Enumerate existing mtp storage devices.
   void EnumerateStorages();
 
   // Map of all attached mtp devices.
-  StorageNameToIdMap storage_map_;
+  StorageLocationToInfoMap storage_map_;
 
   // Function handler to get storage information. This is useful to set a mock
   // handler for unit testing.