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 = |