| Index: chrome/browser/component_updater/test/crx_downloader_unittest.cc
|
| diff --git a/chrome/browser/component_updater/test/crx_downloader_unittest.cc b/chrome/browser/component_updater/test/crx_downloader_unittest.cc
|
| index ba4415cea840a0e60c4a038b4da1cbbe8204695d..42a5d2e441445551e3ff4a0a81d4ddf99345a368 100644
|
| --- a/chrome/browser/component_updater/test/crx_downloader_unittest.cc
|
| +++ b/chrome/browser/component_updater/test/crx_downloader_unittest.cc
|
| @@ -104,7 +104,8 @@ void CrxDownloaderTest::TearDown() {
|
| }
|
|
|
| void CrxDownloaderTest::Quit() {
|
| - quit_closure_.Run();
|
| + if (!quit_closure_.is_null())
|
| + quit_closure_.Run();
|
| }
|
|
|
| void CrxDownloaderTest::DownloadComplete(
|
| @@ -133,6 +134,18 @@ void CrxDownloaderTest::RunThreadsUntilIdle() {
|
| base::RunLoop().RunUntilIdle();
|
| }
|
|
|
| +// Tests that starting a download without a url results in an error.
|
| +TEST_F(CrxDownloaderTest, NoUrl) {
|
| + std::vector<GURL> urls;
|
| + crx_downloader_->StartDownload(urls);
|
| +
|
| + RunThreadsUntilIdle();
|
| + EXPECT_EQ(-1, error_);
|
| + EXPECT_EQ(kExpectedContext, crx_context_);
|
| +
|
| + EXPECT_EQ(1, num_calls_);
|
| +}
|
| +
|
| // Tests that downloading from one url is successful.
|
| TEST_F(CrxDownloaderTest, OneUrl) {
|
| const GURL expected_crx_url =
|
|
|