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

Unified Diff: components/precache/content/precache_manager_unittest.cc

Issue 2544613003: Disable precache if data_reduction_proxy enabled. (Closed)
Patch Set: Rebase. Created 4 years 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
« no previous file with comments | « components/precache/content/precache_manager.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/precache/content/precache_manager_unittest.cc
diff --git a/components/precache/content/precache_manager_unittest.cc b/components/precache/content/precache_manager_unittest.cc
index b35d1c0c3a30401e6dd4af75f8b5c2b0de11333c..c8d0d0c4fe6682c42c22be71cc556e3faa035d14 100644
--- a/components/precache/content/precache_manager_unittest.cc
+++ b/components/precache/content/precache_manager_unittest.cc
@@ -127,14 +127,18 @@ class TestPrecacheCompletionCallback {
class PrecacheManagerUnderTest : public PrecacheManager {
public:
- PrecacheManagerUnderTest(content::BrowserContext* browser_context,
- const syncer::SyncService* const sync_service,
- const history::HistoryService* const history_service,
- const base::FilePath& db_path,
- std::unique_ptr<PrecacheDatabase> precache_database)
+ PrecacheManagerUnderTest(
+ content::BrowserContext* browser_context,
+ const syncer::SyncService* sync_service,
+ const history::HistoryService* history_service,
+ const data_reduction_proxy::DataReductionProxySettings*
+ data_reduction_proxy_settings,
+ const base::FilePath& db_path,
+ std::unique_ptr<PrecacheDatabase> precache_database)
: PrecacheManager(browser_context,
sync_service,
history_service,
+ data_reduction_proxy_settings,
db_path,
std::move(precache_database)),
control_group_(false) {}
@@ -186,10 +190,10 @@ class PrecacheManagerTest : public testing::Test {
factory_.SetFakeResponse(GURL(kConfigURL), "",
net::HTTP_INTERNAL_SERVER_ERROR,
net::URLRequestStatus::FAILED);
- precache_manager_.reset(
- new PrecacheManagerUnderTest(
- &browser_context_, nullptr /* sync_service */,
- &history_service_, db_path, std::move(precache_database)));
+ precache_manager_.reset(new PrecacheManagerUnderTest(
+ &browser_context_, nullptr /* sync_service */, &history_service_,
+ nullptr /* data_reduction_proxy_settings */, db_path,
+ std::move(precache_database)));
base::RunLoop().RunUntilIdle();
info_.headers = new net::HttpResponseHeaders("");
« no previous file with comments | « components/precache/content/precache_manager.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698