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

Unified Diff: chrome/browser/predictors/loading_stats_collector.h

Issue 2937623007: predictors: Move more methods from ResourcePrefetchPredictor into LoadingDataCollector. (Closed)
Patch Set: Fix browser test Created 3 years, 5 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/loading_stats_collector.h
diff --git a/chrome/browser/predictors/loading_stats_collector.h b/chrome/browser/predictors/loading_stats_collector.h
index c4ad6ed7939662de415bf91b847e38a4f5e70d29..f9b64e789c8892ba0f6e4b5d6458d8b8ce968b3c 100644
--- a/chrome/browser/predictors/loading_stats_collector.h
+++ b/chrome/browser/predictors/loading_stats_collector.h
@@ -12,12 +12,15 @@
#include "base/macros.h"
#include "base/time/time.h"
-#include "chrome/browser/predictors/resource_prefetch_predictor.h"
#include "chrome/browser/predictors/resource_prefetcher.h"
#include "url/gurl.h"
namespace predictors {
+class ResourcePrefetchPredictor;
+struct LoadingPredictorConfig;
+struct PageRequestSummary;
+
namespace internal {
constexpr char kResourcePrefetchPredictorPrecisionHistogram[] =
"ResourcePrefetchPredictor.LearningPrecision";
@@ -56,8 +59,7 @@ class LoadingStatsCollector {
// actions taken for a given page load if any. The summary is compared with a
// prediction by ResourcePrefetchPredictor as well.
// All results are reported to UMA.
- void RecordPageRequestSummary(
- const ResourcePrefetchPredictor::PageRequestSummary& summary);
+ void RecordPageRequestSummary(const PageRequestSummary& summary);
// Evicts all stale stats that are kept in memory. All speculative actions are
// reported and considered as waste.
void CleanupAbandonedStats();
« no previous file with comments | « chrome/browser/predictors/loading_predictor.cc ('k') | chrome/browser/predictors/loading_stats_collector.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698