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

Unified Diff: chrome/browser/media_galleries/linux/snapshot_file_details.h

Issue 377383002: Media Galleries: Access MTP devices by file ids rather than file paths. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 5 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_galleries/linux/snapshot_file_details.h
diff --git a/chrome/browser/media_galleries/linux/snapshot_file_details.h b/chrome/browser/media_galleries/linux/snapshot_file_details.h
index c36d6e3fc51adf694ebd174c03014127600b8fa5..557646bf8cc79586dc2c9bffd03acf27ab8e2d38 100644
--- a/chrome/browser/media_galleries/linux/snapshot_file_details.h
+++ b/chrome/browser/media_galleries/linux/snapshot_file_details.h
@@ -16,15 +16,15 @@
// Used to represent snapshot file request params.
struct SnapshotRequestInfo {
SnapshotRequestInfo(
- const std::string& device_file_path,
+ uint32 file_id,
const base::FilePath& snapshot_file_path,
const MTPDeviceAsyncDelegate::CreateSnapshotFileSuccessCallback&
success_callback,
const MTPDeviceAsyncDelegate::ErrorCallback& error_callback);
~SnapshotRequestInfo();
- // MTP device file path.
- const std::string device_file_path;
+ // MTP device file id.
+ const uint32 file_id;
// Local platform path of the snapshot file.
const base::FilePath snapshot_file_path;
@@ -47,8 +47,8 @@ class SnapshotFileDetails {
~SnapshotFileDetails();
- std::string device_file_path() const {
- return request_info_.device_file_path;
+ uint32 file_id() const {
+ return request_info_.file_id;
}
base::FilePath snapshot_file_path() const {

Powered by Google App Engine
This is Rietveld 408576698