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

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

Issue 1363673004: [DRP] Consistently use LoFi for an entire page (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: final comments Created 5 years, 2 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_config_test_utils.cc
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_test_utils.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_test_utils.cc
index f0135e8f83c529894156b9394d98caa07d38e0fc..f2e237cc209dc512d05e3c7ca0adced18f2c03ae 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_test_utils.cc
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_test_utils.cc
@@ -43,8 +43,6 @@ TestDataReductionProxyConfig::TestDataReductionProxyConfig(
config_values.Pass(),
configurator,
event_creator),
- auto_lofi_enabled_group_(false),
- auto_lofi_control_group_(false),
network_quality_prohibitively_slow_(false) {
network_interfaces_.reset(new net::NetworkInterfaceList());
}
@@ -96,25 +94,7 @@ void TestDataReductionProxyConfig::SetStateForTest(
}
void TestDataReductionProxyConfig::ResetLoFiStatusForTest() {
- lofi_status_ = LoFiStatus::LOFI_STATUS_TEMPORARILY_OFF;
-}
-
-bool TestDataReductionProxyConfig::IsIncludedInLoFiEnabledFieldTrial() const {
- return auto_lofi_enabled_group_;
-}
-
-bool TestDataReductionProxyConfig::IsIncludedInLoFiControlFieldTrial() const {
- return auto_lofi_control_group_;
-}
-
-void TestDataReductionProxyConfig::SetIncludedInLoFiEnabledFieldTrial(
- bool auto_lofi_enabled_group) {
- auto_lofi_enabled_group_ = auto_lofi_enabled_group;
-}
-
-void TestDataReductionProxyConfig::SetIncludedInLoFiControlFieldTrial(
- bool auto_lofi_control_group) {
- auto_lofi_control_group_ = auto_lofi_control_group;
+ lofi_off_ = false;
}
void TestDataReductionProxyConfig::SetNetworkProhibitivelySlow(
@@ -143,7 +123,7 @@ void MockDataReductionProxyConfig::UpdateConfigurator(
}
void MockDataReductionProxyConfig::ResetLoFiStatusForTest() {
- lofi_status_ = LoFiStatus::LOFI_STATUS_TEMPORARILY_OFF;
+ lofi_off_ = false;
}
} // namespace data_reduction_proxy

Powered by Google App Engine
This is Rietveld 408576698