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

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

Issue 2183153002: NQE: Move ECT to net:: namespace (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased, also fixed comnpilation error Created 4 years, 5 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_unittest.cc
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_unittest.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_unittest.cc
index 93b03c21dbd9054fc7c091ae2617e87356198bbe..cad437968020d16513c86a5f49463a4a810c82ee 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_unittest.cc
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_unittest.cc
@@ -38,6 +38,7 @@
#include "net/base/net_errors.h"
#include "net/http/http_status_code.h"
#include "net/log/test_net_log.h"
+#include "net/nqe/effective_connection_type.h"
#include "net/nqe/external_estimate_provider.h"
#include "net/nqe/network_quality_estimator.h"
#include "net/proxy/proxy_server.h"
@@ -865,40 +866,35 @@ class TestNetworkQualityEstimator : public net::NetworkQualityEstimator {
: NetworkQualityEstimator(
std::unique_ptr<net::ExternalEstimateProvider>(),
variation_params),
- effective_connection_type_(
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_UNKNOWN),
+ effective_connection_type_(net::EFFECTIVE_CONNECTION_TYPE_UNKNOWN),
recent_effective_connection_type_(
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_UNKNOWN) {}
+ net::EFFECTIVE_CONNECTION_TYPE_UNKNOWN) {}
~TestNetworkQualityEstimator() override {}
- EffectiveConnectionType GetEffectiveConnectionType() const override {
+ net::EffectiveConnectionType GetEffectiveConnectionType() const override {
return effective_connection_type_;
}
- EffectiveConnectionType GetRecentEffectiveConnectionType(
+ net::EffectiveConnectionType GetRecentEffectiveConnectionType(
const base::TimeTicks& start_time) const override {
return recent_effective_connection_type_;
}
void SetEffectiveConnectionType(
- net::NetworkQualityEstimator::EffectiveConnectionType
- effective_connection_type) {
+ net::EffectiveConnectionType effective_connection_type) {
effective_connection_type_ = effective_connection_type;
}
void SetRecentEffectiveConnectionType(
- net::NetworkQualityEstimator::EffectiveConnectionType
- recent_effective_connection_type) {
+ net::EffectiveConnectionType recent_effective_connection_type) {
recent_effective_connection_type_ = recent_effective_connection_type;
}
private:
// Estimate of the quality of the network.
- net::NetworkQualityEstimator::EffectiveConnectionType
- effective_connection_type_;
- net::NetworkQualityEstimator::EffectiveConnectionType
- recent_effective_connection_type_;
+ net::EffectiveConnectionType effective_connection_type_;
+ net::EffectiveConnectionType recent_effective_connection_type_;
base::TimeDelta rtt_since_;
};
@@ -944,9 +940,8 @@ TEST_F(DataReductionProxyConfigTest, AutoLoFiParams) {
};
for (size_t i = 0; i < arraysize(tests); ++i) {
- net::NetworkQualityEstimator::EffectiveConnectionType
- expected_effective_connection_type =
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_SLOW_2G;
+ net::EffectiveConnectionType expected_effective_connection_type =
+ net::EFFECTIVE_CONNECTION_TYPE_SLOW_2G;
int expected_hysteresis_sec = 360;
if (tests[i].lofi_flag_group) {
@@ -954,8 +949,7 @@ TEST_F(DataReductionProxyConfigTest, AutoLoFiParams) {
base::CommandLine::ForCurrentProcess()->AppendSwitchASCII(
switches::kDataReductionProxyLoFi,
switches::kDataReductionProxyLoFiValueSlowConnectionsOnly);
- expected_effective_connection_type =
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_2G;
+ expected_effective_connection_type = net::EFFECTIVE_CONNECTION_TYPE_2G;
expected_hysteresis_sec = 361;
}
@@ -979,7 +973,7 @@ TEST_F(DataReductionProxyConfigTest, AutoLoFiParams) {
// Network quality improved. However, network should still be marked as slow
// because of hysteresis.
test_network_quality_estimator.SetEffectiveConnectionType(
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_BROADBAND);
+ net::EFFECTIVE_CONNECTION_TYPE_BROADBAND);
EXPECT_TRUE(config.IsNetworkQualityProhibitivelySlow(
&test_network_quality_estimator));
@@ -1023,7 +1017,7 @@ TEST_F(DataReductionProxyConfigTest, AutoLoFiMissingParams) {
config.PopulateAutoLoFiParams();
- EXPECT_EQ(net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_SLOW_2G,
+ EXPECT_EQ(net::EFFECTIVE_CONNECTION_TYPE_SLOW_2G,
config.lofi_effective_connection_type_threshold_);
EXPECT_EQ(base::TimeDelta::FromSeconds(60), config.auto_lofi_hysteresis_);
}
@@ -1039,9 +1033,8 @@ TEST_F(DataReductionProxyConfigTest, AutoLoFiParamsSlowConnectionsFlag) {
config.PopulateAutoLoFiParams();
- net::NetworkQualityEstimator::EffectiveConnectionType
- expected_effective_connection_type =
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_SLOW_2G;
+ net::EffectiveConnectionType expected_effective_connection_type =
+ net::EffectiveConnectionType::EFFECTIVE_CONNECTION_TYPE_SLOW_2G;
int hysteresis_sec = 60;
EXPECT_EQ(expected_effective_connection_type,
config.lofi_effective_connection_type_threshold_);
@@ -1054,14 +1047,14 @@ TEST_F(DataReductionProxyConfigTest, AutoLoFiParamsSlowConnectionsFlag) {
// Network is slow.
test_network_quality_estimator.SetEffectiveConnectionType(
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_SLOW_2G);
+ net::EFFECTIVE_CONNECTION_TYPE_SLOW_2G);
EXPECT_TRUE(config.IsNetworkQualityProhibitivelySlow(
&test_network_quality_estimator));
// Network quality improved. However, network should still be marked as slow
// because of hysteresis.
test_network_quality_estimator.SetEffectiveConnectionType(
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_2G);
+ net::EFFECTIVE_CONNECTION_TYPE_2G);
EXPECT_TRUE(config.IsNetworkQualityProhibitivelySlow(
&test_network_quality_estimator));
@@ -1075,7 +1068,7 @@ TEST_F(DataReductionProxyConfigTest, AutoLoFiParamsSlowConnectionsFlag) {
// Changing the network quality has no effect because of hysteresis.
test_network_quality_estimator.SetEffectiveConnectionType(
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_SLOW_2G);
+ net::EFFECTIVE_CONNECTION_TYPE_SLOW_2G);
EXPECT_FALSE(config.IsNetworkQualityProhibitivelySlow(
&test_network_quality_estimator));
@@ -1113,37 +1106,30 @@ TEST_F(DataReductionProxyConfigTest, LoFiAccuracy) {
const struct {
std::string description;
std::string field_trial_group;
- net::NetworkQualityEstimator::EffectiveConnectionType
- effective_connection_type;
- net::NetworkQualityEstimator::EffectiveConnectionType
- recent_effective_connection_type;
+ net::EffectiveConnectionType effective_connection_type;
+ net::EffectiveConnectionType recent_effective_connection_type;
bool expect_network_quality_slow;
uint32_t bucket_to_check;
uint32_t expected_bucket_count;
} tests[] = {
{"Predicted slow, actually slow, Enabled group", "Enabled",
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_SLOW_2G,
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_SLOW_2G, true, 0,
- 1},
+ net::EFFECTIVE_CONNECTION_TYPE_SLOW_2G,
+ net::EFFECTIVE_CONNECTION_TYPE_SLOW_2G, true, 0, 1},
{"Predicted slow, actually slow, Enabled_NoControl group",
- "Enabled_NoControl",
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_SLOW_2G,
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_SLOW_2G, true, 0,
- 1},
+ "Enabled_NoControl", net::EFFECTIVE_CONNECTION_TYPE_SLOW_2G,
+ net::EFFECTIVE_CONNECTION_TYPE_SLOW_2G, true, 0, 1},
{"Predicted slow, actually slow, Control group", "Control",
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_SLOW_2G,
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_SLOW_2G, true, 0,
- 1},
+ net::EFFECTIVE_CONNECTION_TYPE_SLOW_2G,
+ net::EFFECTIVE_CONNECTION_TYPE_SLOW_2G, true, 0, 1},
{"Predicted slow, actually not slow", "Enabled",
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_SLOW_2G,
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_2G, true, 1, 1},
+ net::EFFECTIVE_CONNECTION_TYPE_SLOW_2G,
+ net::EFFECTIVE_CONNECTION_TYPE_2G, true, 1, 1},
{"Predicted not slow, actually slow", "Enabled",
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_2G,
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_SLOW_2G, false,
- 2, 1},
+ net::EFFECTIVE_CONNECTION_TYPE_2G,
+ net::EFFECTIVE_CONNECTION_TYPE_SLOW_2G, false, 2, 1},
{"Predicted not slow, actually not slow", "Enabled",
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_2G,
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_2G, false, 3, 1},
+ net::EFFECTIVE_CONNECTION_TYPE_2G, net::EFFECTIVE_CONNECTION_TYPE_2G,
+ false, 3, 1},
};
for (const auto& test : tests) {
@@ -1220,9 +1206,9 @@ TEST_F(DataReductionProxyConfigTest, LoFiAccuracyNonZeroDelay) {
base::HistogramTester histogram_tester;
// Network was predicted to be slow and actually was slow.
test_network_quality_estimator.SetEffectiveConnectionType(
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_SLOW_2G);
+ net::EFFECTIVE_CONNECTION_TYPE_SLOW_2G);
test_network_quality_estimator.SetRecentEffectiveConnectionType(
- net::NetworkQualityEstimator::EFFECTIVE_CONNECTION_TYPE_SLOW_2G);
+ net::EFFECTIVE_CONNECTION_TYPE_SLOW_2G);
ASSERT_TRUE(config.IsNetworkQualityProhibitivelySlow(
&test_network_quality_estimator));
tick_clock->Advance(base::TimeDelta::FromSeconds(1));
« no previous file with comments | « components/data_reduction_proxy/core/browser/data_reduction_proxy_config.cc ('k') | content/browser/loader/resource_loader.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698