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

Unified Diff: chrome/browser/system_monitor/media_transfer_protocol_device_observer_chromeos.h

Issue 10918259: [Chrome OS]Implement MediaStorageUtil::GetDeviceInfoForPath function to support media gallery permi… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix win compile error Created 8 years, 3 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/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..825a70eb18f53528a280e43bba06e62114053fed 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,15 @@ 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.
+ 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.

Powered by Google App Engine
This is Rietveld 408576698