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

Unified Diff: content/public/browser/download_item.h

Issue 12286020: Replace FilePath with base::FilePath. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 10 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
« no previous file with comments | « content/public/browser/devtools_http_handler_delegate.h ('k') | content/public/browser/download_manager.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/public/browser/download_item.h
diff --git a/content/public/browser/download_item.h b/content/public/browser/download_item.h
index 3e29c07543799aebbb7e436c7af1387aab2ccf55..e3e8d3276da9b5c6ed0974a742da629bf70de165 100644
--- a/content/public/browser/download_item.h
+++ b/content/public/browser/download_item.h
@@ -201,27 +201,27 @@ class CONTENT_EXPORT DownloadItem : public base::SupportsUserData {
// physical file, if one exists. It should be considered a hint; changes to
// this value and renames of the file on disk are not atomic with each other.
// May be empty if the in-progress path hasn't been determined yet.
- virtual const FilePath& GetFullPath() const = 0;
+ virtual const base::FilePath& GetFullPath() const = 0;
// Target path of an in-progress download. We may be downloading to a
// temporary or intermediate file (specified by GetFullPath()); this is the
// name we will use once the download completes.
// May be empty if the target path hasn't yet been determined.
- virtual const FilePath& GetTargetFilePath() const = 0;
+ virtual const base::FilePath& GetTargetFilePath() const = 0;
// If the download forced a path rather than requesting name determination,
// return the path requested.
- virtual const FilePath& GetForcedFilePath() const = 0;
+ virtual const base::FilePath& GetForcedFilePath() const = 0;
// Returns the user-verified target file path for the download.
// This returns the same path as GetTargetFilePath() for safe downloads
// but does not for dangerous downloads until the name is verified.
- virtual FilePath GetUserVerifiedFilePath() const = 0;
+ virtual base::FilePath GetUserVerifiedFilePath() const = 0;
// Returns the file-name that should be reported to the user. If a display
// name has been explicitly set using SetDisplayName(), this function returns
// that display name. Otherwise returns the final target filename.
- virtual FilePath GetFileNameToReportUser() const = 0;
+ virtual base::FilePath GetFileNameToReportUser() const = 0;
virtual TargetDisposition GetTargetDisposition() const = 0;
@@ -314,7 +314,7 @@ class CONTENT_EXPORT DownloadItem : public base::SupportsUserData {
// Set a display name for the download that will be independent of the target
// filename. If |name| is not empty, then GetFileNameToReportUser() will
// return |name|. Has no effect on the final target filename.
- virtual void SetDisplayName(const FilePath& name) = 0;
+ virtual void SetDisplayName(const base::FilePath& name) = 0;
// Debug/testing -------------------------------------------------------------
virtual std::string DebugString(bool verbose) const = 0;
« no previous file with comments | « content/public/browser/devtools_http_handler_delegate.h ('k') | content/public/browser/download_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698