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

Unified Diff: chrome/browser/media_gallery/media_storage_util.cc

Issue 10882039: Make the Linux System Monitor implementation track all devices (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix CrOS build Created 8 years, 4 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/media_gallery/media_storage_util.cc
diff --git a/chrome/browser/media_gallery/media_storage_util.cc b/chrome/browser/media_gallery/media_storage_util.cc
index 14dece03451875cea15cc1093f5a2cf61ec7ddad..ec9c4770819feab87c12b71eceb30608e80a21ae 100644
--- a/chrome/browser/media_gallery/media_storage_util.cc
+++ b/chrome/browser/media_gallery/media_storage_util.cc
@@ -156,7 +156,7 @@ void MediaStorageUtil::IsDeviceAttached(const std::string& device_id,
callback.Run(!FindRemovableStorageLocationById(device_id).empty());
break;
case REMOVABLE_MASS_STORAGE_NO_DCIM:
- FindUSBDeviceById(unique_id,
+ FindUSBDeviceById(device_id,
base::Bind(&EmptyPathIsFalseCallback, callback));
break;
case FIXED_MASS_STORAGE:
@@ -202,7 +202,7 @@ void MediaStorageUtil::FindDevicePathById(const std::string& device_id,
callback.Run(FilePath());
break;
case REMOVABLE_MASS_STORAGE_NO_DCIM:
- FindUSBDeviceById(unique_id, callback);
+ FindUSBDeviceById(device_id, callback);
break;
case FIXED_MASS_STORAGE:
// For this type, the unique_id is the path.
@@ -222,7 +222,7 @@ void MediaStorageUtil::FindDevicePathById(const std::string& device_id,
MediaStorageUtil::MediaStorageUtil() {}
// static
-void MediaStorageUtil::FindUSBDeviceById(const std::string& unique_id,
+void MediaStorageUtil::FindUSBDeviceById(const std::string& device_id,
const FilePathCallback& callback) {
// TODO(vandebo) This needs to be implemented per platform.
// Type is REMOVABLE_MASS_STORAGE_NO_DCIM, so it's a device possibly mounted
« no previous file with comments | « chrome/browser/media_gallery/media_storage_util.h ('k') | chrome/browser/media_gallery/removable_device_notifications_linux.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698