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

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

Issue 2743373002: predictors: Highlight prefetchable resources in webui. (Closed)
Patch Set: Created 3 years, 9 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.cc
diff --git a/chrome/browser/predictors/resource_prefetch_predictor.cc b/chrome/browser/predictors/resource_prefetch_predictor.cc
index f25f57d7fa2905a2d799962ab483c2e2245bb39b..87635c7c19c9052ac61da25ea96881ebbc3b5d5b 100644
--- a/chrome/browser/predictors/resource_prefetch_predictor.cc
+++ b/chrome/browser/predictors/resource_prefetch_predictor.cc
@@ -641,6 +641,15 @@ bool ResourcePrefetchPredictor::IsUrlPrefetchable(const GURL& main_frame_url) {
return GetPrefetchData(main_frame_url, nullptr);
}
+bool ResourcePrefetchPredictor::IsResourcePrefetchable(
+ const ResourceData& resource) const {
+ float confidence = static_cast<float>(resource.number_of_hits()) /
+ (resource.number_of_hits() + resource.number_of_misses());
+ return confidence >= config_.min_resource_confidence_to_trigger_prefetch &&
+ resource.number_of_hits() >=
+ config_.min_resource_hits_to_trigger_prefetch;
+}
+
void ResourcePrefetchPredictor::SetObserverForTesting(TestObserver* observer) {
observer_ = observer;
}
@@ -1320,15 +1329,6 @@ void ResourcePrefetchPredictor::LearnRedirect(const std::string& key,
}
}
-bool ResourcePrefetchPredictor::IsResourcePrefetchable(
- const ResourceData& resource) const {
- float confidence = static_cast<float>(resource.number_of_hits()) /
- (resource.number_of_hits() + resource.number_of_misses());
- return confidence >= config_.min_resource_confidence_to_trigger_prefetch &&
- resource.number_of_hits() >=
- config_.min_resource_hits_to_trigger_prefetch;
-}
-
void ResourcePrefetchPredictor::ReportDatabaseReadiness(
const history::TopHostsList& top_hosts) const {
DCHECK_CURRENTLY_ON(BrowserThread::UI);

Powered by Google App Engine
This is Rietveld 408576698