diff options
| author | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-09-21 09:15:59 +0000 |
|---|---|---|
| committer | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-09-21 09:15:59 +0000 |
| commit | ec4826a74de56b6f7a26b3cbad1f7a4713b5dda2 (patch) | |
| tree | 14e867d4330b82e245339f64d5996dc736e63dba /chrome/browser/download/save_package.cc | |
| parent | e275e3461a11dd5e325be3a7788ca7258eb7a6ea (diff) | |
| download | chromium_src-ec4826a74de56b6f7a26b3cbad1f7a4713b5dda2.zip chromium_src-ec4826a74de56b6f7a26b3cbad1f7a4713b5dda2.tar.gz chromium_src-ec4826a74de56b6f7a26b3cbad1f7a4713b5dda2.tar.bz2 | |
Regression fix: keep the download shelf visible when multiple sites are saved.
BUG=54149
TEST=see bug
Review URL: http://codereview.chromium.org/3442003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60034 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/download/save_package.cc')
| -rw-r--r-- | chrome/browser/download/save_package.cc | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/chrome/browser/download/save_package.cc b/chrome/browser/download/save_package.cc index 42003fa..9953a4c 100644 --- a/chrome/browser/download/save_package.cc +++ b/chrome/browser/download/save_package.cc @@ -301,14 +301,9 @@ SavePackage::~SavePackage() { STLDeleteValues(&in_progress_items_); STLDeleteValues(&saved_failed_items_); - if (download_) { - // We call this to remove the view from the shelf. It will invoke - // DownloadManager::RemoveDownload, but since the fake DownloadItem is not - // owned by DownloadManager, it will do nothing to our fake item. - download_->Remove(false); - delete download_; - download_ = NULL; - } + // The DownloadItem is owned by DownloadManager. + download_ = NULL; + file_manager_ = NULL; // If there's an outstanding save dialog, make sure it doesn't call us back |