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

Unified Diff: chrome/browser/ui/browser_close_browsertest.cc

Issue 10855116: Move DownloadTestObserver and friends down into content. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Incorporated comments. Created 8 years, 4 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 | « chrome/browser/extensions/crx_installer_browsertest.cc ('k') | chrome/chrome_tests.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/browser_close_browsertest.cc
diff --git a/chrome/browser/ui/browser_close_browsertest.cc b/chrome/browser/ui/browser_close_browsertest.cc
index 7002a3f532b6147fac614cd421eebe808d4259e2..1fb8ae922a173b3a1ed2f4b7e3bfe5f765298efb 100644
--- a/chrome/browser/ui/browser_close_browsertest.cc
+++ b/chrome/browser/ui/browser_close_browsertest.cc
@@ -9,7 +9,7 @@
#include "chrome/browser/browser_process.h"
#include "chrome/browser/download/download_service.h"
#include "chrome/browser/download/download_service_factory.h"
-#include "chrome/browser/download/download_test_observer.h"
+#include "chrome/browser/download/download_test_file_chooser_observer.h"
#include "chrome/browser/net/url_request_mock_util.h"
#include "chrome/browser/prefs/pref_service.h"
#include "chrome/browser/profiles/profile.h"
@@ -26,6 +26,7 @@
#include "content/public/browser/download_item.h"
#include "content/public/common/page_transition_types.h"
#include "content/public/test/browser_test_utils.h"
+#include "content/public/test/download_test_observer.h"
#include "content/test/net/url_request_slow_download_job.h"
using content::BrowserContext;
@@ -117,9 +118,9 @@ class BrowserCloseTest : public InProcessBrowserTest {
// to get to IN_PROGRESS.
DownloadManager* download_manager =
BrowserContext::GetDownloadManager(browser->profile());
- scoped_ptr<DownloadTestObserver> observer(
- new DownloadTestObserverInProgress(download_manager,
- num_downloads));
+ scoped_ptr<content::DownloadTestObserver> observer(
+ new content::DownloadTestObserverInProgress(download_manager,
+ num_downloads));
// Set of that number of downloads.
size_t count_downloads = num_downloads;
@@ -150,8 +151,8 @@ class BrowserCloseTest : public InProcessBrowserTest {
DownloadServiceFactory::GetForProfile(*pit);
if (download_service->HasCreatedDownloadManager()) {
DownloadManager *mgr = BrowserContext::GetDownloadManager(*pit);
- scoped_refptr<DownloadTestFlushObserver> observer(
- new DownloadTestFlushObserver(mgr));
+ scoped_refptr<content::DownloadTestFlushObserver> observer(
+ new content::DownloadTestFlushObserver(mgr));
observer->WaitForFlush();
}
if ((*pit)->HasOffTheRecordProfile()) {
@@ -161,8 +162,8 @@ class BrowserCloseTest : public InProcessBrowserTest {
if (incognito_download_service->HasCreatedDownloadManager()) {
DownloadManager *mgr = BrowserContext::GetDownloadManager(
(*pit)->GetOffTheRecordProfile());
- scoped_refptr<DownloadTestFlushObserver> observer(
- new DownloadTestFlushObserver(mgr));
+ scoped_refptr<content::DownloadTestFlushObserver> observer(
+ new content::DownloadTestFlushObserver(mgr));
observer->WaitForFlush();
}
}
« no previous file with comments | « chrome/browser/extensions/crx_installer_browsertest.cc ('k') | chrome/chrome_tests.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698