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

Unified Diff: chrome/browser/chromeos/drive/drive_url_request_job_unittest.cc

Issue 14197014: Add TestBrowserThreadBundle into RenderViewHostTestHarness. Kill some unnecessary real threads. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: compile fixes. Created 7 years, 6 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
Index: chrome/browser/chromeos/drive/drive_url_request_job_unittest.cc
diff --git a/chrome/browser/chromeos/drive/drive_url_request_job_unittest.cc b/chrome/browser/chromeos/drive/drive_url_request_job_unittest.cc
index c5ca7df33247191ed47ab681a0bb2e33daca3593..43be656a0d06c1de91340671759d9e65fb728c01 100644
--- a/chrome/browser/chromeos/drive/drive_url_request_job_unittest.cc
+++ b/chrome/browser/chromeos/drive/drive_url_request_job_unittest.cc
@@ -18,7 +18,7 @@
#include "chrome/browser/google_apis/task_util.h"
#include "chrome/browser/google_apis/test_util.h"
#include "chrome/common/url_constants.h"
-#include "content/public/test/test_browser_thread.h"
+#include "content/public/test/test_browser_thread_bundle.h"
#include "googleurl/src/gurl.h"
#include "net/base/test_completion_callback.h"
#include "net/http/http_byte_range.h"
@@ -96,23 +96,25 @@ class TestDelegate : public net::TestDelegate {
class DriveURLRequestJobTest : public testing::Test {
protected:
- DriveURLRequestJobTest() : ui_thread_(BrowserThread::UI),
- io_thread_(BrowserThread::IO, &message_loop_) {
+ DriveURLRequestJobTest()
+ : thread_bundle_(content::TestBrowserThreadBundle::IO_MAINLOOP) {
}
virtual ~DriveURLRequestJobTest() {
}
virtual void SetUp() OVERRIDE {
- ui_thread_.Start();
+ // Initialize FakeDriveService.
+ fake_drive_service_.reset(new google_apis::FakeDriveService);
+ ASSERT_TRUE(fake_drive_service_->LoadResourceListForWapi(
+ "chromeos/gdata/root_feed.json"));
+ ASSERT_TRUE(fake_drive_service_->LoadAccountMetadataForWapi(
+ "chromeos/gdata/account_metadata.json"));
- BrowserThread::PostTaskAndReply(
- BrowserThread::UI,
- FROM_HERE,
- base::Bind(&DriveURLRequestJobTest::SetUpOnUIThread,
- base::Unretained(this)),
- base::MessageLoop::QuitClosure());
- message_loop_.Run();
+ // Initialize FakeFileSystem.
+ fake_file_system_.reset(
+ new test_util::FakeFileSystem(fake_drive_service_.get()));
+ ASSERT_TRUE(fake_file_system_->InitializeForTesting());
scoped_refptr<base::SequencedWorkerPool> blocking_pool =
BrowserThread::GetBlockingPool();
@@ -134,30 +136,6 @@ class DriveURLRequestJobTest : public testing::Test {
test_url_request_job_factory_.reset();
test_network_delegate_.reset();
- BrowserThread::PostTaskAndReply(
- BrowserThread::UI,
- FROM_HERE,
- base::Bind(&DriveURLRequestJobTest::TearDownOnUIThread,
- base::Unretained(this)),
- base::MessageLoop::QuitClosure());
- message_loop_.Run();
- }
-
- void SetUpOnUIThread() {
- // Initialize FakeDriveService.
- fake_drive_service_.reset(new google_apis::FakeDriveService);
- ASSERT_TRUE(fake_drive_service_->LoadResourceListForWapi(
- "chromeos/gdata/root_feed.json"));
- ASSERT_TRUE(fake_drive_service_->LoadAccountMetadataForWapi(
- "chromeos/gdata/account_metadata.json"));
-
- // Initialize FakeFileSystem.
- fake_file_system_.reset(
- new test_util::FakeFileSystem(fake_drive_service_.get()));
- ASSERT_TRUE(fake_file_system_->InitializeForTesting());
- }
-
- void TearDownOnUIThread() {
fake_file_system_.reset();
fake_drive_service_.reset();
}
@@ -202,9 +180,7 @@ class DriveURLRequestJobTest : public testing::Test {
return true;
}
- base::MessageLoopForIO message_loop_;
- content::TestBrowserThread ui_thread_;
- content::TestBrowserThread io_thread_;
+ content::TestBrowserThreadBundle thread_bundle_;
scoped_ptr<google_apis::FakeDriveService> fake_drive_service_;
scoped_ptr<test_util::FakeFileSystem> fake_file_system_;

Powered by Google App Engine
This is Rietveld 408576698