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

Unified Diff: chrome/browser/predictors/resource_prefetch_common_unittest.cc

Issue 2815103002: predictors: Enable origin learning with a flag. (Closed)
Patch Set: . Created 3 years, 8 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: chrome/browser/predictors/resource_prefetch_common_unittest.cc
diff --git a/chrome/browser/predictors/resource_prefetch_common_unittest.cc b/chrome/browser/predictors/resource_prefetch_common_unittest.cc
index 56e12037e4129a6fd3d0c8d870f9364816b09bf0..643d3d05461436f0b182b00659bf7846c098b103 100644
--- a/chrome/browser/predictors/resource_prefetch_common_unittest.cc
+++ b/chrome/browser/predictors/resource_prefetch_common_unittest.cc
@@ -73,7 +73,7 @@ class ResourcePrefetchCommonTest : public testing::Test {
EXPECT_FALSE(config.IsSmallDBEnabledForTest());
EXPECT_FALSE(config.is_url_learning_enabled);
EXPECT_FALSE(config.is_manifests_enabled);
- EXPECT_FALSE(config.is_origin_prediction_enabled);
+ EXPECT_FALSE(config.is_origin_learning_enabled);
EXPECT_GT(config.min_resource_hits_to_trigger_prefetch, 1U);
}
@@ -162,6 +162,19 @@ TEST_F(ResourcePrefetchCommonTest, EnableManifests) {
EXPECT_TRUE(config.is_manifests_enabled);
}
+TEST_F(ResourcePrefetchCommonTest, EnableOriginLearning) {
+ variations::testing::VariationParamsManager params_manager(
+ "dummy-trial",
+ {{kModeParamName, kLearningMode},
+ {kEnableOriginLearningParamName, "true"}},
+ {kSpeculativeResourcePrefetchingFeatureName});
+
+ ResourcePrefetchPredictorConfig config;
+ EXPECT_TRUE(IsSpeculativeResourcePrefetchingEnabled(profile_.get(), &config));
+ TestIsPrefetchLearning(config);
+ EXPECT_TRUE(config.is_origin_learning_enabled);
+}
+
// Verifies whether prefetching is disabled according to the network type. But
// learning should not be disabled by network.
TEST_F(ResourcePrefetchCommonTest, RespectsNetworkSettings) {
« no previous file with comments | « chrome/browser/predictors/resource_prefetch_common.cc ('k') | chrome/browser/predictors/resource_prefetch_predictor.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698