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

Unified Diff: components/previews/core/previews_experiments_unittest.cc

Issue 2760063002: Add support to previews/ for Server LoFi and LitePages (Closed)
Patch Set: Coverging LoFi types Created 3 years, 7 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/previews/core/previews_experiments_unittest.cc
diff --git a/components/previews/core/previews_experiments_unittest.cc b/components/previews/core/previews_experiments_unittest.cc
index be81b3d7587cb07bed2565227aaa18fc2298688a..75c7ef27df1d6d217f73f8d57155ba501308da2e 100644
--- a/components/previews/core/previews_experiments_unittest.cc
+++ b/components/previews/core/previews_experiments_unittest.cc
@@ -58,7 +58,7 @@ TEST(PreviewsExperimentsTest, TestParamsForBlackListAndOffline) {
EXPECT_EQ(base::TimeDelta::FromDays(7),
params::OfflinePreviewFreshnessDuration());
EXPECT_EQ(net::EFFECTIVE_CONNECTION_TYPE_SLOW_2G,
- params::EffectiveConnectionTypeThresholdForOffline());
+ params::EffectiveConnectionTypeThreshold());
EXPECT_EQ(0, params::OfflinePreviewsVersion());
base::FieldTrialList field_trial_list(nullptr);
@@ -94,7 +94,7 @@ TEST(PreviewsExperimentsTest, TestParamsForBlackListAndOffline) {
EXPECT_EQ(base::TimeDelta::FromDays(12),
params::OfflinePreviewFreshnessDuration());
EXPECT_EQ(net::EFFECTIVE_CONNECTION_TYPE_4G,
- params::EffectiveConnectionTypeThresholdForOffline());
+ params::EffectiveConnectionTypeThreshold());
EXPECT_EQ(10, params::OfflinePreviewsVersion());
variations::testing::ClearAllVariationParams();
@@ -125,9 +125,9 @@ TEST(PreviewsExperimentsTest, TestEnableClientLoFiWithDefaultParams) {
base::FieldTrialList::CreateFieldTrial(kClientLoFiFieldTrial, kEnabled));
EXPECT_TRUE(params::IsClientLoFiEnabled());
- EXPECT_EQ(0, params::ClientLoFiVersion());
+ EXPECT_EQ(0, params::LoFiVersion());
EXPECT_EQ(net::EFFECTIVE_CONNECTION_TYPE_2G,
- params::EffectiveConnectionTypeThresholdForClientLoFi());
+ params::EffectiveConnectionTypeThresholdForLoFi());
}
TEST(PreviewsExperimentsTest, TestEnableClientLoFiWithCustomParams) {
@@ -143,9 +143,9 @@ TEST(PreviewsExperimentsTest, TestEnableClientLoFiWithCustomParams) {
base::FieldTrialList::CreateFieldTrial(kClientLoFiFieldTrial, kEnabled));
EXPECT_TRUE(params::IsClientLoFiEnabled());
- EXPECT_EQ(10, params::ClientLoFiVersion());
+ EXPECT_EQ(10, params::LoFiVersion());
EXPECT_EQ(net::EFFECTIVE_CONNECTION_TYPE_3G,
- params::EffectiveConnectionTypeThresholdForClientLoFi());
+ params::EffectiveConnectionTypeThresholdForLoFi());
variations::testing::ClearAllVariationParams();
}

Powered by Google App Engine
This is Rietveld 408576698