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

Unified Diff: chrome/browser/download/download_resource_throttle_unittest.cc

Issue 2014803002: Move DownloadControllerAndroid from content/ to chrome/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix failing tests/bugs Created 4 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/download/download_resource_throttle_unittest.cc
diff --git a/chrome/browser/download/download_resource_throttle_unittest.cc b/chrome/browser/download/download_resource_throttle_unittest.cc
index 7c771f78fdd50e6299bcebda7abafbe62071bf6c..0ce884ccfeff30eab6e79ad38c940e54451ca009 100644
--- a/chrome/browser/download/download_resource_throttle_unittest.cc
+++ b/chrome/browser/download/download_resource_throttle_unittest.cc
@@ -20,7 +20,7 @@
#include "testing/gtest/include/gtest/gtest.h"
#if BUILDFLAG(ANDROID_JAVA_UI)
-#include "chrome/browser/android/download/mock_download_controller_android.h"
+#include "chrome/browser/android/download/mock_download_controller.h"
#endif
namespace {
@@ -64,8 +64,7 @@ class DownloadResourceThrottleTest : public ChromeRenderViewHostTestHarness {
web_contents()->SetDelegate(&delegate_);
run_loop_.reset(new base::RunLoop());
#if BUILDFLAG(ANDROID_JAVA_UI)
- content::DownloadControllerAndroid::SetDownloadControllerAndroid(
- &download_controller_);
+ DownloadControllerBase::SetDownloadControllerBase(&download_controller_);
#endif
}
@@ -73,7 +72,7 @@ class DownloadResourceThrottleTest : public ChromeRenderViewHostTestHarness {
content::BrowserThread::DeleteSoon(content::BrowserThread::IO, FROM_HERE,
throttle_);
#if BUILDFLAG(ANDROID_JAVA_UI)
- content::DownloadControllerAndroid::SetDownloadControllerAndroid(nullptr);
+ DownloadControllerBase::SetDownloadControllerBase(nullptr);
#endif
ChromeRenderViewHostTestHarness::TearDown();
}
@@ -106,7 +105,7 @@ class DownloadResourceThrottleTest : public ChromeRenderViewHostTestHarness {
::testing::NiceMock<MockResourceController> resource_controller_;
std::unique_ptr<base::RunLoop> run_loop_;
#if BUILDFLAG(ANDROID_JAVA_UI)
- chrome::android::MockDownloadControllerAndroid download_controller_;
+ chrome::android::MockDownloadController download_controller_;
#endif
};
@@ -118,7 +117,7 @@ TEST_F(DownloadResourceThrottleTest, StartDownloadThrottle_Basic) {
#if BUILDFLAG(ANDROID_JAVA_UI)
TEST_F(DownloadResourceThrottleTest, DownloadWithFailedFileAcecssRequest) {
- content::DownloadControllerAndroid::Get()
+ DownloadControllerBase::Get()
->SetApproveFileAccessRequestForTesting(false);
EXPECT_CALL(resource_controller_, Cancel())
.WillOnce(QuitLoop(run_loop_->QuitClosure()));
« no previous file with comments | « chrome/browser/download/download_resource_throttle.cc ('k') | chrome/browser/download/download_ui_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698