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

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

Issue 2847183002: predictors: Introduce GlowplugPredictor. (Closed)
Patch Set: Address comments. Created 3 years, 7 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_tab_helper.cc
diff --git a/chrome/browser/predictors/resource_prefetch_predictor_tab_helper.cc b/chrome/browser/predictors/resource_prefetch_predictor_tab_helper.cc
index d6d2dd7e075ceeb0930d558302f76a8a370262f6..412f59711fae1f91acbed91ca05d402440f44e4b 100644
--- a/chrome/browser/predictors/resource_prefetch_predictor_tab_helper.cc
+++ b/chrome/browser/predictors/resource_prefetch_predictor_tab_helper.cc
@@ -4,8 +4,10 @@
#include "chrome/browser/predictors/resource_prefetch_predictor_tab_helper.h"
-#include "chrome/browser/predictors/resource_prefetch_predictor.h"
-#include "chrome/browser/predictors/resource_prefetch_predictor_factory.h"
+#include <string>
+
+#include "chrome/browser/predictors/glowplug_predictor.h"
+#include "chrome/browser/predictors/glowplug_predictor_factory.h"
#include "chrome/browser/profiles/profile.h"
#include "content/public/browser/browser_thread.h"
@@ -27,14 +29,14 @@ ResourcePrefetchPredictorTabHelper::~ResourcePrefetchPredictorTabHelper() {
void ResourcePrefetchPredictorTabHelper::DocumentOnLoadCompletedInMainFrame() {
DCHECK_CURRENTLY_ON(BrowserThread::UI);
- ResourcePrefetchPredictor* predictor =
- ResourcePrefetchPredictorFactory::GetForProfile(
- web_contents()->GetBrowserContext());
+ auto* predictor = GlowplugPredictorFactory::GetForProfile(
+ Profile::FromBrowserContext(web_contents()->GetBrowserContext()));
if (!predictor)
return;
+ auto* resource_prefetch_predictor = predictor->resource_prefetch_predictor();
NavigationID navigation_id(web_contents());
- predictor->RecordMainFrameLoadComplete(navigation_id);
+ resource_prefetch_predictor->RecordMainFrameLoadComplete(navigation_id);
}
void ResourcePrefetchPredictorTabHelper::DidLoadResourceFromMemoryCache(
@@ -43,9 +45,8 @@ void ResourcePrefetchPredictorTabHelper::DidLoadResourceFromMemoryCache(
content::ResourceType resource_type) {
DCHECK_CURRENTLY_ON(BrowserThread::UI);
- ResourcePrefetchPredictor* predictor =
- ResourcePrefetchPredictorFactory::GetForProfile(
- web_contents()->GetBrowserContext());
+ auto* predictor = GlowplugPredictorFactory::GetForProfile(
+ Profile::FromBrowserContext(web_contents()->GetBrowserContext()));
if (!predictor)
return;
@@ -57,7 +58,8 @@ void ResourcePrefetchPredictorTabHelper::DidLoadResourceFromMemoryCache(
ResourcePrefetchPredictor::GetResourceTypeFromMimeType(
mime_type, resource_type);
summary.was_cached = true;
- predictor->RecordURLResponse(summary);
+ auto* resource_prefetch_predictor = predictor->resource_prefetch_predictor();
+ resource_prefetch_predictor->RecordURLResponse(summary);
}
} // namespace predictors

Powered by Google App Engine
This is Rietveld 408576698