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

Unified Diff: chrome/browser/predictors/resource_prefetch_common.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.cc
diff --git a/chrome/browser/predictors/resource_prefetch_common.cc b/chrome/browser/predictors/resource_prefetch_common.cc
index 1c45d1ce095f5788c850e145c9a8cccf82e2ed1e..ca7a9a64a4c243842909b75812f971f970278e5e 100644
--- a/chrome/browser/predictors/resource_prefetch_common.cc
+++ b/chrome/browser/predictors/resource_prefetch_common.cc
@@ -25,6 +25,7 @@ const char kExternalPrefetchingMode[] = "external-prefetching";
const char kPrefetchingMode[] = "prefetching";
const char kEnableUrlLearningParamName[] = "enable-url-learning";
const char kEnableManifestsParamName[] = "enable-manifests";
+const char kEnableOriginLearningParamName[] = "enable-origin-learning";
const base::Feature kSpeculativeResourcePrefetchingFeature =
base::Feature(kSpeculativeResourcePrefetchingFeatureName,
@@ -71,6 +72,11 @@ bool IsSpeculativeResourcePrefetchingEnabled(
if (enable_manifests_value == "true")
config->is_manifests_enabled = true;
+ bool enable_origin_learning = base::GetFieldTrialParamValueByFeature(
+ kSpeculativeResourcePrefetchingFeature,
+ kEnableOriginLearningParamName) == "true";
+ config->is_origin_learning_enabled = enable_origin_learning;
+
// Ensure that a resource that was only seen once is never prefetched. This
// prevents the easy mistake of trying to prefetch an ephemeral url.
DCHECK_GT(config->min_resource_hits_to_trigger_prefetch, 1U);
@@ -145,7 +151,7 @@ ResourcePrefetchPredictorConfig::ResourcePrefetchPredictorConfig()
max_prefetches_inflight_per_host_per_navigation(3),
is_url_learning_enabled(false),
is_manifests_enabled(false),
- is_origin_prediction_enabled(false) {}
+ is_origin_learning_enabled(false) {}
ResourcePrefetchPredictorConfig::ResourcePrefetchPredictorConfig(
const ResourcePrefetchPredictorConfig& other) = default;
« no previous file with comments | « chrome/browser/predictors/resource_prefetch_common.h ('k') | chrome/browser/predictors/resource_prefetch_common_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698