Chromium Code Reviews| 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..3448b97fb23aa63ac1e2c114f532536dc39d54a8 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, EnableOriginPrediction) { |
|
alexilin
2017/04/13 09:01:21
nit:
s/EnableOriginPrediction/EnableOriginLearning
Benoit L
2017/04/13 09:15:51
Done.
|
| + variations::testing::VariationParamsManager params_manager( |
| + "dummy-trial", |
| + {{kModeParamName, kLearningMode}, |
| + {kEnableOriginLearningParamName, "true"}}, |
| + {kSpeculativeResourcePrefetchingFeatureName}); |
| + |
| + ResourcePrefetchPredictorConfig config; |
| + EXPECT_TRUE(IsSpeculativeResourcePrefetchingEnabled(profile_.get(), &config)); |
| + EXPECT_TRUE(config.IsLearningEnabled()); |
|
alexilin
2017/04/13 09:01:21
nit:
Change this line with `TestIsPrefetchLearning
Benoit L
2017/04/13 09:15:51
Done.
|
| + 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) { |