| Index: chrome/browser/extensions/api/downloads/downloads_api_unittest.cc
|
| diff --git a/chrome/browser/download/download_extension_test.cc b/chrome/browser/extensions/api/downloads/downloads_api_unittest.cc
|
| similarity index 96%
|
| rename from chrome/browser/download/download_extension_test.cc
|
| rename to chrome/browser/extensions/api/downloads/downloads_api_unittest.cc
|
| index 02e2c4e6638cff2882cde3ca50d40a1eef817c10..ee35b4a430ef95d3c9ae1ff470b9106b54321685 100644
|
| --- a/chrome/browser/download/download_extension_test.cc
|
| +++ b/chrome/browser/extensions/api/downloads/downloads_api_unittest.cc
|
| @@ -7,11 +7,11 @@
|
| #include "base/file_util.h"
|
| #include "base/scoped_temp_dir.h"
|
| #include "base/stringprintf.h"
|
| -#include "chrome/browser/download/download_extension_api.h"
|
| #include "chrome/browser/download/download_file_icon_extractor.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/extensions/api/downloads/downloads_api.h"
|
| #include "chrome/browser/extensions/extension_function_test_utils.h"
|
| #include "chrome/browser/net/url_request_mock_util.h"
|
| #include "chrome/browser/prefs/pref_service.h"
|
| @@ -379,7 +379,8 @@ class ScopedItemVectorCanceller {
|
|
|
| } // namespace
|
|
|
| -IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_PauseResumeCancel) {
|
| +IN_PROC_BROWSER_TEST_F(
|
| + DownloadExtensionTest, DownloadExtensionTest_PauseResumeCancel) {
|
| DownloadItem* download_item = CreateSlowTestDownload();
|
| ASSERT_TRUE(download_item);
|
|
|
| @@ -437,7 +438,8 @@ IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_PauseResumeCancel) {
|
|
|
| // Test downloads.getFileIcon() on in-progress, finished, cancelled and deleted
|
| // download items.
|
| -IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_FileIcon_Active) {
|
| +IN_PROC_BROWSER_TEST_F(DownloadExtensionTest,
|
| + DownloadExtensionTest_FileIcon_Active) {
|
| DownloadItem* download_item = CreateSlowTestDownload();
|
| ASSERT_TRUE(download_item);
|
|
|
| @@ -536,7 +538,8 @@ IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_FileIcon_Active) {
|
| // whether they exist or not. If the file doesn't exist we should receive a
|
| // generic icon from the OS/toolkit that may or may not be specific to the file
|
| // type.
|
| -IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_FileIcon_History) {
|
| +IN_PROC_BROWSER_TEST_F(DownloadExtensionTest,
|
| + DownloadExtensionTest_FileIcon_History) {
|
| const HistoryDownloadInfo kHistoryInfo[] = {
|
| { FILE_PATH_LITERAL("real.txt"),
|
| DownloadItem::COMPLETE,
|
| @@ -584,7 +587,8 @@ IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_FileIcon_History) {
|
| // The temporary files should be cleaned up when the ScopedTempDir is removed.
|
| }
|
|
|
| -IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchEmptyQuery) {
|
| +IN_PROC_BROWSER_TEST_F(DownloadExtensionTest,
|
| + DownloadExtensionTest_SearchEmptyQuery) {
|
| ScopedCancellingItem item(CreateSlowTestDownload());
|
| ASSERT_TRUE(item.get());
|
|
|
| @@ -597,7 +601,7 @@ IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchEmptyQuery) {
|
| }
|
|
|
| IN_PROC_BROWSER_TEST_F(DownloadExtensionTest,
|
| - DownloadsApi_SearchFilenameRegex) {
|
| + DownloadExtensionTest_SearchFilenameRegex) {
|
| const HistoryDownloadInfo kHistoryInfo[] = {
|
| { FILE_PATH_LITERAL("foobar"),
|
| DownloadItem::COMPLETE,
|
| @@ -623,7 +627,7 @@ IN_PROC_BROWSER_TEST_F(DownloadExtensionTest,
|
| ASSERT_EQ(0, item_id);
|
| }
|
|
|
| -IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchId) {
|
| +IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadExtensionTest_SearchId) {
|
| DownloadManager::DownloadVector items;
|
| CreateSlowTestDownloads(2, &items);
|
| ScopedItemVectorCanceller delete_items(&items);
|
| @@ -642,7 +646,7 @@ IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchId) {
|
| }
|
|
|
| IN_PROC_BROWSER_TEST_F(DownloadExtensionTest,
|
| - DownloadsApi_SearchIdAndFilename) {
|
| + DownloadExtensionTest_SearchIdAndFilename) {
|
| DownloadManager::DownloadVector items;
|
| CreateSlowTestDownloads(2, &items);
|
| ScopedItemVectorCanceller delete_items(&items);
|
| @@ -655,7 +659,8 @@ IN_PROC_BROWSER_TEST_F(DownloadExtensionTest,
|
| ASSERT_EQ(0UL, result_list->GetSize());
|
| }
|
|
|
| -IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchOrderBy) {
|
| +IN_PROC_BROWSER_TEST_F(DownloadExtensionTest,
|
| + DownloadExtensionTest_SearchOrderBy) {
|
| const HistoryDownloadInfo kHistoryInfo[] = {
|
| { FILE_PATH_LITERAL("zzz"),
|
| DownloadItem::COMPLETE,
|
| @@ -685,7 +690,8 @@ IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchOrderBy) {
|
| ASSERT_LT(item0_name, item1_name);
|
| }
|
|
|
| -IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchOrderByEmpty) {
|
| +IN_PROC_BROWSER_TEST_F(DownloadExtensionTest,
|
| + DownloadExtensionTest_SearchOrderByEmpty) {
|
| const HistoryDownloadInfo kHistoryInfo[] = {
|
| { FILE_PATH_LITERAL("zzz"),
|
| DownloadItem::COMPLETE,
|
| @@ -715,7 +721,8 @@ IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchOrderByEmpty) {
|
| ASSERT_GT(item0_name, item1_name);
|
| }
|
|
|
| -IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchDanger) {
|
| +IN_PROC_BROWSER_TEST_F(DownloadExtensionTest,
|
| + DownloadExtensionTest_SearchDanger) {
|
| const HistoryDownloadInfo kHistoryInfo[] = {
|
| { FILE_PATH_LITERAL("zzz"),
|
| DownloadItem::COMPLETE,
|
| @@ -736,7 +743,8 @@ IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchDanger) {
|
| ASSERT_EQ(1UL, result_list->GetSize());
|
| }
|
|
|
| -IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchState) {
|
| +IN_PROC_BROWSER_TEST_F(DownloadExtensionTest,
|
| + DownloadExtensionTest_SearchState) {
|
| DownloadManager::DownloadVector items;
|
| CreateSlowTestDownloads(2, &items);
|
| ScopedItemVectorCanceller delete_items(&items);
|
| @@ -751,7 +759,8 @@ IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchState) {
|
| ASSERT_EQ(1UL, result_list->GetSize());
|
| }
|
|
|
| -IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchLimit) {
|
| +IN_PROC_BROWSER_TEST_F(DownloadExtensionTest,
|
| + DownloadExtensionTest_SearchLimit) {
|
| DownloadManager::DownloadVector items;
|
| CreateSlowTestDownloads(2, &items);
|
| ScopedItemVectorCanceller delete_items(&items);
|
| @@ -764,7 +773,8 @@ IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchLimit) {
|
| ASSERT_EQ(1UL, result_list->GetSize());
|
| }
|
|
|
| -IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchInvalid) {
|
| +IN_PROC_BROWSER_TEST_F(DownloadExtensionTest,
|
| + DownloadExtensionTest_SearchInvalid) {
|
| std::string error = RunFunctionAndReturnError(
|
| new DownloadsSearchFunction(), "[{\"filenameRegex\": \"(\"}]");
|
| EXPECT_STREQ(download_extension_errors::kInvalidFilterError,
|
| @@ -787,7 +797,8 @@ IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchInvalid) {
|
| error.c_str());
|
| }
|
|
|
| -IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchPlural) {
|
| +IN_PROC_BROWSER_TEST_F(DownloadExtensionTest,
|
| + DownloadExtensionTest_SearchPlural) {
|
| const HistoryDownloadInfo kHistoryInfo[] = {
|
| { FILE_PATH_LITERAL("aaa"),
|
| DownloadItem::CANCELLED,
|
| @@ -821,7 +832,7 @@ IN_PROC_BROWSER_TEST_F(DownloadExtensionTest, DownloadsApi_SearchPlural) {
|
| }
|
|
|
| IN_PROC_BROWSER_TEST_F(DownloadExtensionTestIncognito,
|
| - DownloadsApi_SearchIncognito) {
|
| + DownloadExtensionTest_SearchPauseResumeCancelGetFileIconIncognito) {
|
| scoped_ptr<base::Value> result_value;
|
| base::ListValue* result_list = NULL;
|
| base::DictionaryValue* result_dict = NULL;
|
|
|