[downloads] Move platform specific code out of DownloadTargetDeterminer.

chrome/browser/download/download_path_reservation_tracker.h

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 "Download...Result" is the first
or last argument across all the callbacks.

[asanka, 2016/11/07 19:50:15]

Yeah. I made the result be the first argument since it dictates how to interpret
the path argument.

-                              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.