Create LoadingDataCollector class and have observers rely on it instead of ResourcePrefetchPredictor

chrome/browser/net/resource_prefetch_predictor_observer.h

Index: chrome/browser/net/resource_prefetch_predictor_observer.h
diff --git a/chrome/browser/net/resource_prefetch_predictor_observer.h b/chrome/browser/net/resource_prefetch_predictor_observer.h
index 4801540ce9be4e4b8c414fd8be20e694a61fae84..9e56da663991e860c26da5ec706400bd54451274 100644
--- a/chrome/browser/net/resource_prefetch_predictor_observer.h
+++ b/chrome/browser/net/resource_prefetch_predictor_observer.h
@@ -9,6 +9,7 @@
 
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
+#include "chrome/browser/predictors/glowplug_collector.h"
 #include "chrome/browser/predictors/resource_prefetch_predictor.h"

[alexilin, 2017/05/22 13:20:36]

nit:
Do we need this include?

[trevordixon, 2017/05/22 21:29:01]

Gone.

 #include "content/public/browser/resource_request_info.h"
 #include "content/public/common/resource_type.h"
@@ -31,7 +32,7 @@ namespace chrome_browser_net {
 class ResourcePrefetchPredictorObserver {
  public:
   explicit ResourcePrefetchPredictorObserver(
-      predictors::ResourcePrefetchPredictor* predictor);
+      predictors::GlowplugCollector* collector);
   ~ResourcePrefetchPredictorObserver();
 
   // Parts of the ResourceDispatcherHostDelegate that we want to observe.
@@ -72,7 +73,7 @@ class ResourcePrefetchPredictorObserver {
       const base::TimeTicks& creation_time) const;
 
   // Owned by profile.
-  base::WeakPtr<predictors::ResourcePrefetchPredictor> predictor_;
+  base::WeakPtr<predictors::GlowplugCollector> collector_;
 
   DISALLOW_COPY_AND_ASSIGN(ResourcePrefetchPredictorObserver);
 };