diff options
| author | asanka@chromium.org <asanka@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-05-30 23:32:06 +0000 |
|---|---|---|
| committer | asanka@chromium.org <asanka@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-05-30 23:32:06 +0000 |
| commit | 3d833de2590e5bcddec48b51691d088afb3d84e3 (patch) | |
| tree | 66337201944575e32a60e36334076b4bb2be7874 /content/public/browser/download_manager_delegate.h | |
| parent | bb8f10a98c8f3a96025c47100c6473e2ce9645ad (diff) | |
| download | chromium_src-3d833de2590e5bcddec48b51691d088afb3d84e3.zip chromium_src-3d833de2590e5bcddec48b51691d088afb3d84e3.tar.gz chromium_src-3d833de2590e5bcddec48b51691d088afb3d84e3.tar.bz2 | |
Download filename determination refactor (1/3)
- Removes dependency on DownloadStateInfo in chrome/.
- Adds unit tests for ChromeDownloadManagerDelegate.
- Cleanup methods for filename determination in DownloadItem to eliminate setters.
BUG=78085
TEST=unit tests
Review URL: https://chromiumcodereview.appspot.com/10083010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139682 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/public/browser/download_manager_delegate.h')
| -rw-r--r-- | content/public/browser/download_manager_delegate.h | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/content/public/browser/download_manager_delegate.h b/content/public/browser/download_manager_delegate.h index 21b09024..88a8cc5 100644 --- a/content/public/browser/download_manager_delegate.h +++ b/content/public/browser/download_manager_delegate.h @@ -57,8 +57,12 @@ class CONTENT_EXPORT DownloadManagerDelegate { int32 download_id) {} // Allows the embedder to set an intermediate name for the download until it's - // complete. If the embedder doesn't want this return the suggested path. - virtual FilePath GetIntermediatePath(const FilePath& suggested_path); + // complete. The return value is the intermediate path to use. ok_to_overwrite + // should be set to true if the intermediate path should be overwritten if it + // exists. If the embedder doesn't want to set an intermediate path, it should + // return item.GetTargetFilePath(). + virtual FilePath GetIntermediatePath(const DownloadItem& item, + bool* ok_to_overwrite); // Called when the download system wants to alert a WebContents that a // download has started, but the TabConetnts has gone away. This lets an |