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

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

Issue 2729563002: predictors: Disable URL-based learning by default. (Closed)
Patch Set: Created 3 years, 10 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_predictor_unittest.cc
diff --git a/chrome/browser/predictors/resource_prefetch_predictor_unittest.cc b/chrome/browser/predictors/resource_prefetch_predictor_unittest.cc
index 460c10af7e5d2f969131db7dfa416fe44739a0a9..e01dcf3f1b851a2ebac48816b56568905a3b4be6 100644
--- a/chrome/browser/predictors/resource_prefetch_predictor_unittest.cc
+++ b/chrome/browser/predictors/resource_prefetch_predictor_unittest.cc
@@ -232,8 +232,8 @@ class ResourcePrefetchPredictorTest : public testing::Test {
config.max_consecutive_misses = 2;
config.min_resource_confidence_to_trigger_prefetch = 0.5;
- // TODO(shishir): Enable the prefetching mode in the tests.
- config.mode |= ResourcePrefetchPredictorConfig::LEARNING;
+ config.mode |= ResourcePrefetchPredictorConfig::URL_LEARNING |
+ ResourcePrefetchPredictorConfig::HOST_LEARNING;
predictor_.reset(new ResourcePrefetchPredictor(config, profile_.get()));
predictor_->set_mock_tables(mock_tables_);
}

Powered by Google App Engine
This is Rietveld 408576698