summaryrefslogtreecommitdiffstats
path: root/chrome/browser/download/save_package.cc
diff options
context:
space:
mode:
authorphajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-07-31 22:54:35 +0000
committerphajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-07-31 22:54:35 +0000
commit1a37192a62bb8c6f212705c0df9c3c05bc1bf5e1 (patch)
treeb6c61a05654db7704b843a646c0b0ca060e3b5f7 /chrome/browser/download/save_package.cc
parent0899edb62bc0e042b04beafe8bc64e901c9542f4 (diff)
downloadchromium_src-1a37192a62bb8c6f212705c0df9c3c05bc1bf5e1.zip
chromium_src-1a37192a62bb8c6f212705c0df9c3c05bc1bf5e1.tar.gz
chromium_src-1a37192a62bb8c6f212705c0df9c3c05bc1bf5e1.tar.bz2
Convert parts of SavePage UI tests to more reliable browser tests.
Use a real notification for download completion - that's the main unflakying point. TEST=none http://crbug.com/3791 Review URL: http://codereview.chromium.org/160480 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22212 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/download/save_package.cc')
-rw-r--r--chrome/browser/download/save_package.cc7
1 files changed, 7 insertions, 0 deletions
diff --git a/chrome/browser/download/save_package.cc b/chrome/browser/download/save_package.cc
index dcf8bdb..452a2ba 100644
--- a/chrome/browser/download/save_package.cc
+++ b/chrome/browser/download/save_package.cc
@@ -28,6 +28,8 @@
#include "chrome/browser/tab_contents/tab_contents.h"
#include "chrome/browser/tab_contents/tab_util.h"
#include "chrome/common/chrome_paths.h"
+#include "chrome/common/notification_service.h"
+#include "chrome/common/notification_type.h"
#include "chrome/common/platform_util.h"
#include "chrome/common/pref_names.h"
#include "chrome/common/pref_service.h"
@@ -601,6 +603,11 @@ void SavePackage::Finish() {
save_ids));
download_->Finished(all_save_items_count_);
+
+ NotificationService::current()->Notify(
+ NotificationType::SAVE_PACKAGE_SUCCESSFULLY_FINISHED,
+ Source<SavePackage>(this),
+ Details<GURL>(&page_url_));
}
// Called for updating end state.