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

Unified Diff: chrome/browser/download/download_path_reservation_tracker.h

Issue 2453633006: [downloads] Move platform specific code out of DownloadTargetDeterminer. (Closed)
Patch Set: . Created 4 years, 2 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/download/download_path_reservation_tracker.h
diff --git a/chrome/browser/download/download_path_reservation_tracker.h b/chrome/browser/download/download_path_reservation_tracker.h
index 494253d5188d0ce44ecc1c9d0b8974b926949f25..93793027554048527a61c651dbf254dad6c26993 100644
--- a/chrome/browser/download/download_path_reservation_tracker.h
+++ b/chrome/browser/download/download_path_reservation_tracker.h
@@ -6,6 +6,7 @@
#define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_PATH_RESERVATION_TRACKER_H_
#include "base/callback_forward.h"
+#include "chrome/browser/download/download_target_result.h"
namespace base {
class FilePath;
@@ -33,7 +34,8 @@ class DownloadPathReservationTracker {
// If |requested_target_path| was not writeable, then the parent directory of
// |target_path| may be different from that of |requested_target_path|.
typedef base::Callback<void(const base::FilePath& target_path,
svaldez 2016/10/28 17:29:35 Might want to be consistent about whether the "Dow
asanka 2016/11/07 19:50:15 Yeah. I made the result be the first argument sinc
- bool target_path_verified)> ReservedPathCallback;
+ DownloadTargetResult result)>
+ ReservedPathCallback;
// The largest index for the uniquification suffix that we will try while
// attempting to come up with a unique path.

Powered by Google App Engine
This is Rietveld 408576698