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

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

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.h
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_test_utils.h b/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_test_utils.h
index d709461213cef2082cedc1b7ee336db9512d023a..59b80422ed9c963b3a019484a0ab47f6d084580c 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_test_utils.h
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_test_utils.h
@@ -70,16 +70,6 @@ class TestDataReductionProxyConfig : public DataReductionProxyConfig {
// Resets the Lo-Fi status to default state.
void ResetLoFiStatusForTest();
- bool IsIncludedInLoFiEnabledFieldTrial() const override;
-
- // Allows tests to set the session as part of Lo-Fi enabled field trial.
- void SetIncludedInLoFiEnabledFieldTrial(bool included_in_lofi_enabled);
-
- bool IsIncludedInLoFiControlFieldTrial() const override;
-
- // Allows tests to set the session as part of Lo-Fi control field trial.
- void SetIncludedInLoFiControlFieldTrial(bool included_in_lofi_control);
-
// Allows tests to mark the network as prohibitively slow.
void SetNetworkProhibitivelySlow(bool network_quality_prohibitively_slow);
@@ -93,12 +83,6 @@ class TestDataReductionProxyConfig : public DataReductionProxyConfig {
private:
scoped_ptr<net::NetworkInterfaceList> network_interfaces_;
- // True if this session is part of Auto Lo-Fi enabled field trial.
- bool auto_lofi_enabled_group_;
-
- // True if this session is part of Auto Lo-Fi control field trial.
- bool auto_lofi_control_group_;
-
// True if network quality is slow enough to turn Auto Lo-Fi ON.
bool network_quality_prohibitively_slow_;
};
@@ -139,8 +123,6 @@ class MockDataReductionProxyConfig : public TestDataReductionProxyConfig {
MOCK_METHOD1(
IsNetworkQualityProhibitivelySlow,
bool(const net::NetworkQualityEstimator* network_quality_estimator));
- MOCK_CONST_METHOD0(IsIncludedInLoFiEnabledFieldTrial, bool());
- MOCK_CONST_METHOD0(IsIncludedInLoFiControlFieldTrial, bool());
void UpdateConfigurator(bool enabled, bool restricted) override;

Powered by Google App Engine
This is Rietveld 408576698