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

Unified Diff: components/data_reduction_proxy/core/browser/data_reduction_proxy_io_data_unittest.cc

Issue 2626963003: Remove kAllowed and kFallbackAllowed from data reduction proxy (Closed)
Patch Set: Addressed nits Created 3 years, 11 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: components/data_reduction_proxy/core/browser/data_reduction_proxy_io_data_unittest.cc
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_io_data_unittest.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_io_data_unittest.cc
index 0a56765c4fba32129373030b8b2af8df5ededbcb..7b09403fd4cd96758d57a8469eadad496f540741 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_io_data_unittest.cc
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_io_data_unittest.cc
@@ -101,10 +101,10 @@ class DataReductionProxyIODataTest : public testing::Test {
TEST_F(DataReductionProxyIODataTest, TestConstruction) {
std::unique_ptr<DataReductionProxyIOData> io_data(
- new DataReductionProxyIOData(
- Client::UNKNOWN, DataReductionProxyParams::kAllowed, net_log(),
- task_runner(), task_runner(), false /* enabled */,
- std::string() /* user_agent */, std::string() /* channel */));
+ new DataReductionProxyIOData(Client::UNKNOWN, 0, net_log(), task_runner(),
+ task_runner(), false /* enabled */,
+ std::string() /* user_agent */,
+ std::string() /* channel */));
// Check that the SimpleURLRequestContextGetter uses vanilla HTTP.
net::URLRequestContext* request_context =
@@ -150,9 +150,7 @@ TEST_F(DataReductionProxyIODataTest, TestResetBadProxyListOnDisableDataSaver) {
net::TestURLRequestContext context(false);
std::unique_ptr<DataReductionProxyTestContext> drp_test_context =
DataReductionProxyTestContext::Builder()
- .WithParamsFlags(DataReductionProxyParams::kAllowed |
- DataReductionProxyParams::kFallbackAllowed |
- DataReductionProxyParams::kPromoAllowed)
+ .WithParamsFlags(DataReductionProxyParams::kPromoAllowed)
.WithURLRequestContext(&context)
.SkipSettingsInitialization()
.Build();
@@ -192,9 +190,7 @@ TEST_F(DataReductionProxyIODataTest, HoldbackConfiguresProxies) {
net::TestURLRequestContext context(false);
std::unique_ptr<DataReductionProxyTestContext> drp_test_context =
DataReductionProxyTestContext::Builder()
- .WithParamsFlags(DataReductionProxyParams::kAllowed |
- DataReductionProxyParams::kFallbackAllowed |
- DataReductionProxyParams::kPromoAllowed |
+ .WithParamsFlags(DataReductionProxyParams::kPromoAllowed |
DataReductionProxyParams::kHoldback)
.WithURLRequestContext(&context)
.SkipSettingsInitialization()

Powered by Google App Engine
This is Rietveld 408576698