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

Unified Diff: chrome/browser/download/download_target_result.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_target_result.h
diff --git a/chrome/browser/download/download_target_result.h b/chrome/browser/download/download_target_result.h
new file mode 100644
index 0000000000000000000000000000000000000000..e9e847391a02a048424bced5aa6c42b6cf9e80e8
--- /dev/null
+++ b/chrome/browser/download/download_target_result.h
@@ -0,0 +1,37 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_DOWNLOAD_DOWNLOAD_TARGET_RESULT_H_
+#define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_TARGET_RESULT_H_
+
+enum class DownloadTargetResult {
+ // Successfully reserved the path. If there were conflicts, they were
+ // resolved by uniquifying the path. The returned path is writeable and
+ // unique (at least when last observed). No further user interaction is
+ // required.
+ SUCCESS,
+
+ // Cancelled by user.
+ USER_CANCELED,
+
+ // Unexpected error.
+ UNEXPECTED,
+
+ // The suggested name was too long and could not be truncated to fit within
+ // the limits of the target file system. The returned path may not be valid
+ // for use on the filesystem.
+ NAME_TOO_LONG,
+
+ // There were unresolved conflicts with existing files or files that are in
+ // the process of being downloaded to the target path. The returned path
+ // should not be considered to be unique.
+ CONFLICT,
+
+ // The suggested download path was not writeable. A possibly writeable
+ // alternate path will be returned. The returned path may be empty if no
+ // suitable alternate was found.
+ PATH_NOT_WRITEABLE
svaldez 2016/10/28 17:29:36 *writable
asanka 2016/11/07 19:50:15 Done. Apparently "writable" is much more common in
+};
+
+#endif // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_TARGET_RESULT_H_

Powered by Google App Engine
This is Rietveld 408576698